diff options
| author | Patrick McHardy | 2015-03-03 21:04:19 +0100 |
|---|---|---|
| committer | Pablo Neira Ayuso | 2015-03-04 18:46:05 +0100 |
| commit | 9889840f5988ecfd43b00c9abb83c1804e21406b (patch) | |
| tree | 9124100fd1cb08ea518f56d01b7f0907fd362fe7 /net/netfilter/nf_tables_api.c | |
| parent | netfilter: nf_tables: fix transaction race condition (diff) | |
| download | kernel-qcow2-linux-9889840f5988ecfd43b00c9abb83c1804e21406b.tar.gz kernel-qcow2-linux-9889840f5988ecfd43b00c9abb83c1804e21406b.tar.xz kernel-qcow2-linux-9889840f5988ecfd43b00c9abb83c1804e21406b.zip | |
netfilter: nf_tables: check for overflow of rule dlen field
Check that the space required for the expressions doesn't exceed the
size of the dlen field, which would lead to the iterators crashing.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/netfilter/nf_tables_api.c')
| -rw-r--r-- | net/netfilter/nf_tables_api.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/netfilter/nf_tables_api.c b/net/netfilter/nf_tables_api.c index 6fb532bf0fdb..7baafd5ab520 100644 --- a/net/netfilter/nf_tables_api.c +++ b/net/netfilter/nf_tables_api.c @@ -1968,6 +1968,10 @@ static int nf_tables_newrule(struct sock *nlsk, struct sk_buff *skb, n++; } } + /* Check for overflow of dlen field */ + err = -EFBIG; + if (size >= 1 << 12) + goto err1; if (nla[NFTA_RULE_USERDATA]) ulen = nla_len(nla[NFTA_RULE_USERDATA]); |