Unverified Commit 42640b04 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!1713 netfilter: nf_tables: disallow rule addition to bound chain via NFTA_RULE_CHAIN_ID 

Merge Pull Request from: @ci-robot 
 
PR sync from: Lu Wei <luwei32@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/SGMAG5EQTBM4R4SFB5KR5GIRFLPCZIQP/ 
 
https://gitee.com/src-openeuler/kernel/issues/I7QG0U 
 
Link:https://gitee.com/openeuler/kernel/pulls/1713

 

Reviewed-by: default avatarYue Haibing <yuehaibing@huawei.com>
Reviewed-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
Signed-off-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
parents c0a502be a39007a6

net/netfilter/nf_tables_api.c

+3 −2
Original line number Diff line number Diff line
@@ -3255,8 +3255,6 @@ static int nf_tables_newrule(struct net *net, struct sock *nlsk, 
			NL_SET_BAD_ATTR(extack, nla[NFTA_RULE_CHAIN]);

			return PTR_ERR(chain);

		}

		if (nft_chain_is_bound(chain))

			return -EOPNOTSUPP;



	} else if (nla[NFTA_RULE_CHAIN_ID]) {

		chain = nft_chain_lookup_byid(net, table, nla[NFTA_RULE_CHAIN_ID],
@@ -3269,6 +3267,9 @@ static int nf_tables_newrule(struct net *net, struct sock *nlsk, 
		return -EINVAL;

	}



	if (nft_chain_is_bound(chain))

		return -EOPNOTSUPP;



	if (nla[NFTA_RULE_HANDLE]) {

		handle = be64_to_cpu(nla_get_be64(nla[NFTA_RULE_HANDLE]));

		rule = __nft_rule_lookup(chain, handle);