Skip to content

Commit c00e858

Browse files
committed
Merge git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf
Pablo Neira Ayuso says: ==================== Netfilter fixes for net The following patchset contains Netfilter fixes for net: 1) Use kvfree() to release vmalloc()'ed areas in ipset, from Eric Dumazet. 2) UAF in nfnetlink_queue from the nf_conntrack_update() path. ==================== Signed-off-by: David S. Miller <[email protected]>
2 parents 4d57254 + d005fbb commit c00e858

File tree

5 files changed

+7
-5
lines changed

5 files changed

+7
-5
lines changed

net/netfilter/ipset/ip_set_bitmap_ip.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ bitmap_ip_create(struct net *net, struct ip_set *set, struct nlattr *tb[],
326326
set->variant = &bitmap_ip;
327327
if (!init_map_ip(set, map, first_ip, last_ip,
328328
elements, hosts, netmask)) {
329-
kfree(map);
329+
ip_set_free(map);
330330
return -ENOMEM;
331331
}
332332
if (tb[IPSET_ATTR_TIMEOUT]) {

net/netfilter/ipset/ip_set_bitmap_ipmac.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ bitmap_ipmac_create(struct net *net, struct ip_set *set, struct nlattr *tb[],
363363
map->memsize = BITS_TO_LONGS(elements) * sizeof(unsigned long);
364364
set->variant = &bitmap_ipmac;
365365
if (!init_map_ipmac(set, map, first_ip, last_ip, elements)) {
366-
kfree(map);
366+
ip_set_free(map);
367367
return -ENOMEM;
368368
}
369369
if (tb[IPSET_ATTR_TIMEOUT]) {

net/netfilter/ipset/ip_set_bitmap_port.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ bitmap_port_create(struct net *net, struct ip_set *set, struct nlattr *tb[],
274274
map->memsize = BITS_TO_LONGS(elements) * sizeof(unsigned long);
275275
set->variant = &bitmap_port;
276276
if (!init_map_port(set, map, first_port, last_port)) {
277-
kfree(map);
277+
ip_set_free(map);
278278
return -ENOMEM;
279279
}
280280
if (tb[IPSET_ATTR_TIMEOUT]) {

net/netfilter/ipset/ip_set_hash_gen.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -682,7 +682,7 @@ mtype_resize(struct ip_set *set, bool retried)
682682
}
683683
t->hregion = ip_set_alloc(ahash_sizeof_regions(htable_bits));
684684
if (!t->hregion) {
685-
kfree(t);
685+
ip_set_free(t);
686686
ret = -ENOMEM;
687687
goto out;
688688
}
@@ -1533,7 +1533,7 @@ IPSET_TOKEN(HTYPE, _create)(struct net *net, struct ip_set *set,
15331533
}
15341534
t->hregion = ip_set_alloc(ahash_sizeof_regions(hbits));
15351535
if (!t->hregion) {
1536-
kfree(t);
1536+
ip_set_free(t);
15371537
kfree(h);
15381538
return -ENOMEM;
15391539
}

net/netfilter/nf_conntrack_core.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2158,6 +2158,8 @@ static int nf_conntrack_update(struct net *net, struct sk_buff *skb)
21582158
err = __nf_conntrack_update(net, skb, ct, ctinfo);
21592159
if (err < 0)
21602160
return err;
2161+
2162+
ct = nf_ct_get(skb, &ctinfo);
21612163
}
21622164

21632165
return nf_confirm_cthelper(skb, ct, ctinfo);

0 commit comments

Comments
 (0)