netfilter: synproxy: fix BUG_ON triggered by corrupt TCP packets
authorPatrick McHardy <kaber@trash.net>
Mon, 30 Sep 2013 07:51:46 +0000 (08:51 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 30 Sep 2013 10:44:38 +0000 (12:44 +0200)
TCP packets hitting the SYN proxy through the SYNPROXY target are not
validated by TCP conntrack. When th->doff is below 5, an underflow happens
when calculating the options length, causing skb_header_pointer() to
return NULL and triggering the BUG_ON().

Handle this case gracefully by checking for NULL instead of using BUG_ON().

Reported-by: Martin Topholm <mph@one.com>
Tested-by: Martin Topholm <mph@one.com>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>

No differences found