Prepare for calling netlink_io_error() which needs the context pointer. Trade this in for the cache pointer since no caller uses a special one. No functional change intended. Signed-off-by: Phil Sutter <phil@xxxxxx> --- include/netlink.h | 6 +++--- src/monitor.c | 7 +++---- src/netlink.c | 11 ++++++----- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/include/netlink.h b/include/netlink.h index e9667a24b0d11..c7da6f9e3bcbb 100644 --- a/include/netlink.h +++ b/include/netlink.h @@ -172,9 +172,9 @@ extern int netlink_list_setelems(struct netlink_ctx *ctx, extern int netlink_get_setelem(struct netlink_ctx *ctx, const struct handle *h, const struct location *loc, struct set *cache_set, struct set *set, struct expr *init, bool reset); -extern int netlink_delinearize_setelem(struct nftnl_set_elem *nlse, - struct set *set, - struct nft_cache *cache); +extern int netlink_delinearize_setelem(struct netlink_ctx *ctx, + struct nftnl_set_elem *nlse, + struct set *set); extern int netlink_list_objs(struct netlink_ctx *ctx, const struct handle *h); extern struct obj *netlink_delinearize_obj(struct netlink_ctx *ctx, diff --git a/src/monitor.c b/src/monitor.c index 0db40895babf9..7556dfd350e4e 100644 --- a/src/monitor.c +++ b/src/monitor.c @@ -472,8 +472,8 @@ static int netlink_events_setelem_cb(const struct nlmsghdr *nlh, int type, nftnl_set_elems_iter_destroy(nlsei); goto out; } - if (netlink_delinearize_setelem(nlse, dummyset, - &monh->ctx->nft->cache) < 0) { + if (netlink_delinearize_setelem(monh->ctx, + nlse, dummyset) < 0) { set_free(dummyset); nftnl_set_elems_iter_destroy(nlsei); goto out; @@ -829,8 +829,7 @@ static void netlink_events_cache_addsetelem(struct netlink_mon_handler *monh, nlse = nftnl_set_elems_iter_next(nlsei); while (nlse != NULL) { - if (netlink_delinearize_setelem(nlse, set, - &monh->ctx->nft->cache) < 0) { + if (netlink_delinearize_setelem(monh->ctx, nlse, set) < 0) { fprintf(stderr, "W: Unable to cache set_elem. " "Delinearize failed.\n"); diff --git a/src/netlink.c b/src/netlink.c index fe02120def035..1222919458bae 100644 --- a/src/netlink.c +++ b/src/netlink.c @@ -1435,12 +1435,13 @@ static void set_elem_parse_udata(struct nftnl_set_elem *nlse, } } -int netlink_delinearize_setelem(struct nftnl_set_elem *nlse, - struct set *set, struct nft_cache *cache) +int netlink_delinearize_setelem(struct netlink_ctx *ctx, + struct nftnl_set_elem *nlse, + struct set *set) { struct setelem_parse_ctx setelem_parse_ctx = { .set = set, - .cache = cache, + .cache = &ctx->nft->cache, }; struct nft_data_delinearize nld; struct expr *expr, *key, *data; @@ -1498,7 +1499,7 @@ int netlink_delinearize_setelem(struct nftnl_set_elem *nlse, struct stmt *stmt; nle = nftnl_set_elem_get(nlse, NFTNL_SET_ELEM_EXPR, NULL); - stmt = netlink_parse_set_expr(set, cache, nle); + stmt = netlink_parse_set_expr(set, &ctx->nft->cache, nle); list_add_tail(&stmt->list, &setelem_parse_ctx.stmt_list); } else if (nftnl_set_elem_is_set(nlse, NFTNL_SET_ELEM_EXPRESSIONS)) { nftnl_set_elem_expr_foreach(nlse, set_elem_parse_expressions, @@ -1575,7 +1576,7 @@ int netlink_delinearize_setelem(struct nftnl_set_elem *nlse, static int list_setelem_cb(struct nftnl_set_elem *nlse, void *arg) { struct netlink_ctx *ctx = arg; - return netlink_delinearize_setelem(nlse, ctx->set, &ctx->nft->cache); + return netlink_delinearize_setelem(ctx, nlse, ctx->set); } static int list_setelem_debug_cb(struct nftnl_set_elem *nlse, void *arg) -- 2.49.0