The function only has one caller. It's not clear how to extend this in a useful way, so that it makes sense to keep the initialization in a separate function. Simplify the code, by inlining and dropping the static function nft_ctx_netlink_init(). There was only one caller. Signed-off-by: Thomas Haller <thaller@xxxxxxxxxx> --- src/libnftables.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/libnftables.c b/src/libnftables.c index 5b3eb2dc3df4..79dfdfc7c6ec 100644 --- a/src/libnftables.c +++ b/src/libnftables.c @@ -186,11 +186,6 @@ void nft_ctx_clear_include_paths(struct nft_ctx *ctx) ctx->include_paths = NULL; } -static void nft_ctx_netlink_init(struct nft_ctx *ctx) -{ - ctx->nf_sock = nft_mnl_socket_open(); -} - EXPORT_SYMBOL(nft_ctx_new); struct nft_ctx *nft_ctx_new(uint32_t flags) { @@ -218,7 +213,7 @@ struct nft_ctx *nft_ctx_new(uint32_t flags) ctx->output.error_fp = stderr; init_list_head(&ctx->vars_ctx.indesc_list); - nft_ctx_netlink_init(ctx); + ctx->nf_sock = nft_mnl_socket_open(); return ctx; } -- 2.41.0