David Miller schreef op 21/05/2014 22:18:
From: Bart De Schuymer <bdschuym@xxxxxxxxxx>
There's no reason why they should overlap in the cb: it's 48 bytes
big, so big enough to hold both struct br_input_skb_cb and struct
inet_skb_parm. The original problem was introduced when
BR_INPUT_SKB_CB was introduced (around Feb 27, 2010), so fixing
BR_INPUT_SKB_CB seems most appropriate to me.
So you are suggesting the patch below will fix everything?
Assuming:
- David Newall's worries about IPCB are incorrect
- you also revert the commit mentioned by David
(462fb2af9788a82a534f8184abfde31574e1cfa0 (bridge :
Sanitize skb before it enters the IP stack))
Then I give it a good chance the regression will be gone with your patch.
We never should have added bridging netfilter to the tree in the
first place, I wish I had better judgment back then.
Feel free to deprecate it. This is my last spare-time involvement.
Please apply following patch:
diff --git a/MAINTAINERS b/MAINTAINERS
index f5de16e..2369bae 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3163,10 +3163,9 @@ S: Maintained
F: drivers/scsi/eata_pio.*
EBTABLES
-M: Bart De Schuymer <bart.de.schuymer@xxxxxxxxxx>
L: netfilter-devel@xxxxxxxxxxxxxxx
W: http://ebtables.sourceforge.net/
-S: Maintained
+S: Orphan
F: include/linux/netfilter_bridge/ebt_*.h
F: include/uapi/linux/netfilter_bridge/ebt_*.h
F: net/bridge/netfilter/ebt*.c
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
