From 760bd16776b111c006ba209545abfc5c849702b4 Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Tue, 24 Apr 2018 01:21:32 +0200 Subject: src: simplify netlink_get_setelems() and rename it to netlink_list_setelems() This is called from cache population path, remove netlink_io_error() call since this is not needed. Rename it for consistency with similar netlink_list_*() NLM_F_DUMP functions. Get rid of location parameter. Signed-off-by: Pablo Neira Ayuso --- src/netlink.c | 15 ++++++--------- src/rule.c | 3 +-- 2 files changed, 7 insertions(+), 11 deletions(-) (limited to 'src') diff --git a/src/netlink.c b/src/netlink.c index 9e11af5d..372caaa9 100644 --- a/src/netlink.c +++ b/src/netlink.c @@ -1294,8 +1294,8 @@ static int list_setelem_cb(struct nftnl_set_elem *nlse, void *arg) return netlink_delinearize_setelem(nlse, ctx->set, ctx->cache); } -int netlink_get_setelems(struct netlink_ctx *ctx, const struct handle *h, - const struct location *loc, struct set *set) +int netlink_list_setelems(struct netlink_ctx *ctx, const struct handle *h, + struct set *set) { struct nftnl_set *nls; int err; @@ -1308,11 +1308,11 @@ int netlink_get_setelems(struct netlink_ctx *ctx, const struct handle *h, if (errno == EINTR) return -1; - goto out; + return 0; } ctx->set = set; - set->init = set_expr_alloc(loc, set); + set->init = set_expr_alloc(&internal_location, set); nftnl_set_elem_foreach(nls, list_setelem_cb, ctx); if (!(set->flags & NFT_SET_INTERVAL)) @@ -1323,11 +1323,8 @@ int netlink_get_setelems(struct netlink_ctx *ctx, const struct handle *h, if (set->flags & NFT_SET_INTERVAL) interval_map_decompose(set->init); -out: - if (err < 0) - netlink_io_error(ctx, loc, "Could not receive set elements: %s", - strerror(errno)); - return err; + + return 0; } int netlink_get_setelem(struct netlink_ctx *ctx, const struct handle *h, diff --git a/src/rule.c b/src/rule.c index 2ad7f23e..1f69afd1 100644 --- a/src/rule.c +++ b/src/rule.c @@ -84,8 +84,7 @@ static int cache_init_objects(struct netlink_ctx *ctx, enum cmd_ops cmd) return -1; list_for_each_entry(set, &table->sets, list) { - ret = netlink_get_setelems(ctx, &set->handle, - &internal_location, set); + ret = netlink_list_setelems(ctx, &set->handle, set); if (ret < 0) return -1; } -- cgit v1.2.3