diff options
author | 2019-06-10 12:50:00 +0200 | |
---|---|---|
committer | 2019-06-10 12:50:00 +0200 | |
commit | 13c6ba1f855415cf3b9c58ea926ae8858050ec1c (patch) | |
tree | 81809930b937d2fa61f5550b83c69bc06a9b53dc /net/netfilter/ipset/ip_set_core.c | |
parent | netfilter: ipset: fix a missing check of nla_parse (diff) | |
download | linux-13c6ba1f855415cf3b9c58ea926ae8858050ec1c.tar.gz linux-13c6ba1f855415cf3b9c58ea926ae8858050ec1c.tar.bz2 linux-13c6ba1f855415cf3b9c58ea926ae8858050ec1c.zip |
netfilter: ipset: Fix the last missing check of nla_parse_deprecated()
In dump_init() the outdated comment was incorrect and we had a missing
validation check of nla_parse_deprecated().
Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Diffstat (limited to 'net/netfilter/ipset/ip_set_core.c')
-rw-r--r-- | net/netfilter/ipset/ip_set_core.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/net/netfilter/ipset/ip_set_core.c b/net/netfilter/ipset/ip_set_core.c index d0f4c627ff91..039892cd2b7d 100644 --- a/net/netfilter/ipset/ip_set_core.c +++ b/net/netfilter/ipset/ip_set_core.c @@ -1293,11 +1293,13 @@ dump_init(struct netlink_callback *cb, struct ip_set_net *inst) struct nlattr *attr = (void *)nlh + min_len; u32 dump_type; ip_set_id_t index; + int ret; - /* Second pass, so parser can't fail */ - nla_parse_deprecated(cda, IPSET_ATTR_CMD_MAX, attr, - nlh->nlmsg_len - min_len, ip_set_setname_policy, - NULL); + ret = nla_parse_deprecated(cda, IPSET_ATTR_CMD_MAX, attr, + nlh->nlmsg_len - min_len, + ip_set_setname_policy, NULL); + if (ret) + return ret; cb->args[IPSET_CB_PROTO] = nla_get_u8(cda[IPSET_ATTR_PROTOCOL]); if (cda[IPSET_ATTR_SETNAME]) { |