Preparing for proper multipart dump handling, pass the object reference directly instead of individual fields. Keep passing the 'family' field though so caller does not have to extract that value from netlink header again. Signed-off-by: Phil Sutter --- net/netfilter/nfnetlink_hook.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/net/netfilter/nfnetlink_hook.c b/net/netfilter/nfnetlink_hook.c index 95005e9a6066..e47a2add4d5b 100644 --- a/net/netfilter/nfnetlink_hook.c +++ b/net/netfilter/nfnetlink_hook.c @@ -338,12 +338,12 @@ nfnl_hook_entries_head(u8 pf, unsigned int hook, struct net *net, const char *de } static int nfnl_hook_dump_nat(struct sk_buff *nlskb, - const struct nfnl_dump_hook_data *ctx, - const struct nf_hook_ops *ops, - int family, unsigned int seq) + struct netlink_callback *cb, + const struct nf_hook_ops *ops, int family) { struct nf_nat_lookup_hook_priv *priv = ops->priv; struct nf_hook_entries *e = rcu_dereference(priv->entries); + struct nfnl_dump_hook_data *ctx = cb->data; struct nf_hook_ops **nat_ops; int i, err; @@ -354,7 +354,8 @@ static int nfnl_hook_dump_nat(struct sk_buff *nlskb, for (i = 0; i < e->num_hook_entries; i++) { err = nfnl_hook_dump_one(nlskb, ctx, nat_ops[i], - ops->priority, family, seq); + ops->priority, family, + cb->nlh->nlmsg_seq); if (err) return err; } @@ -390,8 +391,7 @@ static int nfnl_hook_dump(struct sk_buff *nlskb, for (; i < e->num_hook_entries; i++) { if (ops[i]->hook_ops_type == NF_HOOK_OP_NAT) - err = nfnl_hook_dump_nat(nlskb, ctx, ops[i], family, - cb->nlh->nlmsg_seq); + err = nfnl_hook_dump_nat(nlskb, cb, ops[i], family); else err = nfnl_hook_dump_one(nlskb, ctx, ops[i], ops[i]->priority, family, -- 2.54.0