Revision e7c243c925f6d9dcb898504ff24d6650b5cbb3b1 authored by Evgeniy Polyakov on 25 August 2007, 06:36:29 UTC, committed by David S. Miller on 27 August 2007, 01:35:47 UTC
I tried to preserve bridging code as it was before, but logic is quite
strange - I think we should free skb on error, since it is already
unshared and thus will just leak.

Herbert Xu states:

> +	if ((skb = skb_share_check(skb, GFP_ATOMIC)) == NULL)
> +		goto out;

If this happens it'll be a double-free on skb since we'll
return NF_DROP which makes the caller free it too.

We could return NF_STOLEN to prevent that but I'm not sure
whether that's correct netfilter semantics.  Patrick, could
you please make a call on this?

Patrick McHardy states:

NF_STOLEN should work fine here.

Signed-off-by: Evgeniy Polyakov <johnpol@2ka.mipt.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 7c8347a
History
File Mode Size
9p
bluetooth
irda
iucv
netfilter
sctp
tc_act
tipc
act_api.h -rw-r--r-- 4.1 KB
addrconf.h -rw-r--r-- 6.8 KB
af_rxrpc.h -rw-r--r-- 2.0 KB
af_unix.h -rw-r--r-- 1.8 KB
ah.h -rw-r--r-- 766 bytes
arp.h -rw-r--r-- 989 bytes
atmclip.h -rw-r--r-- 1.6 KB
ax25.h -rw-r--r-- 14.8 KB
ax88796.h -rw-r--r-- 782 bytes
cfg80211.h -rw-r--r-- 2.3 KB
checksum.h -rw-r--r-- 2.4 KB
cipso_ipv4.h -rw-r--r-- 6.2 KB
compat.h -rw-r--r-- 1.4 KB
datalink.h -rw-r--r-- 482 bytes
dn.h -rw-r--r-- 7.0 KB
dn_dev.h -rw-r--r-- 5.4 KB
dn_fib.h -rw-r--r-- 4.8 KB
dn_neigh.h -rw-r--r-- 824 bytes
dn_nsp.h -rw-r--r-- 6.1 KB
dn_route.h -rw-r--r-- 4.1 KB
dsfield.h -rw-r--r-- 1.1 KB
dst.h -rw-r--r-- 6.0 KB
esp.h -rw-r--r-- 1.5 KB
fib_rules.h -rw-r--r-- 2.5 KB
flow.h -rw-r--r-- 2.0 KB
gen_stats.h -rw-r--r-- 1.5 KB
genetlink.h -rw-r--r-- 6.7 KB
icmp.h -rw-r--r-- 2.0 KB
ieee80211.h -rw-r--r-- 38.1 KB
ieee80211_crypt.h -rw-r--r-- 3.6 KB
ieee80211_radiotap.h -rw-r--r-- 9.5 KB
ieee80211softmac.h -rw-r--r-- 12.1 KB
ieee80211softmac_wx.h -rw-r--r-- 3.1 KB
if_inet6.h -rw-r--r-- 6.3 KB
inet6_connection_sock.h -rw-r--r-- 1.2 KB
inet6_hashtables.h -rw-r--r-- 2.9 KB
inet_common.h -rw-r--r-- 1.4 KB
inet_connection_sock.h -rw-r--r-- 10.6 KB
inet_ecn.h -rw-r--r-- 2.9 KB
inet_hashtables.h -rw-r--r-- 13.6 KB
inet_sock.h -rw-r--r-- 4.9 KB
inet_timewait_sock.h -rw-r--r-- 6.5 KB
inetpeer.h -rw-r--r-- 1.4 KB
ip.h -rw-r--r-- 11.3 KB
ip6_checksum.h -rw-r--r-- 2.3 KB
ip6_fib.h -rw-r--r-- 4.7 KB
ip6_route.h -rw-r--r-- 4.1 KB
ip6_tunnel.h -rw-r--r-- 1.0 KB
ip_fib.h -rw-r--r-- 6.6 KB
ip_vs.h -rw-r--r-- 29.9 KB
ipcomp.h -rw-r--r-- 212 bytes
ipconfig.h -rw-r--r-- 833 bytes
ipip.h -rw-r--r-- 1.2 KB
ipv6.h -rw-r--r-- 16.5 KB
ipx.h -rw-r--r-- 3.8 KB
iw_handler.h -rw-r--r-- 23.5 KB
lapb.h -rw-r--r-- 4.7 KB
llc.h -rw-r--r-- 3.7 KB
llc_c_ac.h -rw-r--r-- 10.1 KB
llc_c_ev.h -rw-r--r-- 11.6 KB
llc_c_st.h -rw-r--r-- 1.7 KB
llc_conn.h -rw-r--r-- 4.0 KB
llc_if.h -rw-r--r-- 3.1 KB
llc_pdu.h -rw-r--r-- 14.3 KB
llc_s_ac.h -rw-r--r-- 1.6 KB
llc_s_ev.h -rw-r--r-- 2.3 KB
llc_s_st.h -rw-r--r-- 941 bytes
llc_sap.h -rw-r--r-- 1.1 KB
mac80211.h -rw-r--r-- 40.0 KB
mip6.h -rw-r--r-- 1.7 KB
ndisc.h -rw-r--r-- 2.9 KB
neighbour.h -rw-r--r-- 9.5 KB
netdma.h -rw-r--r-- 1.4 KB
netevent.h -rw-r--r-- 784 bytes
netlabel.h -rw-r--r-- 10.8 KB
netlink.h -rw-r--r-- 32.3 KB
netrom.h -rw-r--r-- 7.9 KB
nexthop.h -rw-r--r-- 821 bytes
p8022.h -rw-r--r-- 441 bytes
pkt_cls.h -rw-r--r-- 9.0 KB
pkt_sched.h -rw-r--r-- 2.9 KB
protocol.h -rw-r--r-- 3.6 KB
psnap.h -rw-r--r-- 296 bytes
raw.h -rw-r--r-- 1.4 KB
rawv6.h -rw-r--r-- 954 bytes
red.h -rw-r--r-- 7.5 KB
request_sock.h -rw-r--r-- 6.7 KB
rose.h -rw-r--r-- 7.7 KB
route.h -rw-r--r-- 5.4 KB
rtnetlink.h -rw-r--r-- 2.7 KB
sch_generic.h -rw-r--r-- 7.5 KB
scm.h -rw-r--r-- 2.7 KB
slhc_vj.h -rw-r--r-- 6.6 KB
snmp.h -rw-r--r-- 3.9 KB
sock.h -rw-r--r-- 39.1 KB
syncppp.h -rw-r--r-- 3.4 KB
tcp.h -rw-r--r-- 38.3 KB
tcp_ecn.h -rw-r--r-- 3.2 KB
tcp_states.h -rw-r--r-- 1.2 KB
timewait_sock.h -rw-r--r-- 1.2 KB
transp_v6.h -rw-r--r-- 1.2 KB
udp.h -rw-r--r-- 4.9 KB
udplite.h -rw-r--r-- 3.5 KB
wext.h -rw-r--r-- 468 bytes
wireless.h -rw-r--r-- 3.3 KB
x25.h -rw-r--r-- 9.3 KB
x25device.h -rw-r--r-- 322 bytes
xfrm.h -rw-r--r-- 31.6 KB

back to top