<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/net/bridge, branch tmp/leds/core</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=tmp%2Fleds%2Fcore</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=tmp%2Fleds%2Fcore'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2009-01-13T05:18:35Z</updated>
<entry>
<title>netfilter 05/09: ebtables: fix inversion in match code</title>
<updated>2009-01-13T05:18:35Z</updated>
<author>
<name>Jan Engelhardt</name>
<email>jengelh@medozas.de</email>
</author>
<published>2009-01-12T00:06:06Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=d61ba9fd55b52a10b8e0ffd39bbc33587d3bfc8d'/>
<id>urn:sha1:d61ba9fd55b52a10b8e0ffd39bbc33587d3bfc8d</id>
<content type='text'>
Commit 8cc784ee (netfilter: change return types of match functions
for ebtables extensions) broke ebtables matches by inverting the
sense of match/nomatch.

Reported-by: Matt Cross &lt;matthltc@us.ibm.com&gt;
Signed-off-by: Jan Engelhardt &lt;jengelh@medozas.de&gt;
Signed-off-by: Patrick McHardy &lt;kaber@trash.net&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>netfilter 03/09: bridge: Disable PPPOE/VLAN processing by default</title>
<updated>2009-01-13T05:18:34Z</updated>
<author>
<name>Herbert Xu</name>
<email>herbert@gondor.apana.org.au</email>
</author>
<published>2009-01-12T00:06:03Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=47e0e1ca13d64eeeb687995fbe4e239e743d7544'/>
<id>urn:sha1:47e0e1ca13d64eeeb687995fbe4e239e743d7544</id>
<content type='text'>
The PPPOE/VLAN processing code in the bridge netfilter is broken
by design.  The VLAN tag and the PPPOE session ID are an integral
part of the packet flow information, yet they're completely
ignored by the bridge netfilter.  This is potentially a security
hole as it treats all VLANs and PPPOE sessions as the same.

What's more, it's actually broken for PPPOE as the bridge netfilter
tries to trim the packets to the IP length without adjusting the
PPPOE header (and adjusting the PPPOE header isn't much better
since the PPPOE peer may require the padding to be present).

Therefore we should disable this by default.

It does mean that people relying on this feature may lose networking
depending on how their bridge netfilter rules are configured.
However, IMHO the problems this code causes are serious enough to
warrant this.

Signed-off-by: Herbert Xu &lt;herbert@gondor.apana.org.au&gt;
Signed-off-by: Patrick McHardy &lt;kaber@trash.net&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>netfilter 02/09: bridge: Fix handling of non-IP packets in FORWARD/POST_ROUTING</title>
<updated>2009-01-13T05:18:33Z</updated>
<author>
<name>Herbert Xu</name>
<email>herbert@gondor.apana.org.au</email>
</author>
<published>2009-01-12T00:06:02Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a2bd40ad3151d4d346fd167e01fb84b06f7247fc'/>
<id>urn:sha1:a2bd40ad3151d4d346fd167e01fb84b06f7247fc</id>
<content type='text'>
Currently the bridge FORWARD/POST_ROUTING chains treats all
non-IPv4 packets as IPv6.  This packet fixes that by returning
NF_ACCEPT on non-IP packets instead, just as is done in PRE_ROUTING.

Signed-off-by: Herbert Xu &lt;herbert@gondor.apana.org.au&gt;
Signed-off-by: Patrick McHardy &lt;kaber@trash.net&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>net/bridge/netfilter: move a dereference below a NULL test</title>
<updated>2009-01-11T08:06:33Z</updated>
<author>
<name>Julia Lawall</name>
<email>julia@diku.dk</email>
</author>
<published>2009-01-09T10:22:22Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=f3d8b2e467da7a9237a45248ff03b56b6a7c3df7'/>
<id>urn:sha1:f3d8b2e467da7a9237a45248ff03b56b6a7c3df7</id>
<content type='text'>
In each case, if the NULL test is necessary, then the dereference should be
moved below the NULL test.

The semantic patch that makes this change is as follows:
(http://www.emn.fr/x-info/coccinelle/)

// &lt;smpl&gt;
@@
type T;
expression E;
identifier i,fld;
statement S;
@@

- T i = E-&gt;fld;
+ T i;
  ... when != E
      when != i
  if (E == NULL) S
+ i = E-&gt;fld;
// &lt;/smpl&gt;

Signed-off-by: Julia Lawall &lt;julia@diku.dk&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/kaber/nf-next-2.6</title>
<updated>2008-11-28T10:19:15Z</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2008-11-28T10:19:15Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ed77a89c30fa03dcb234a84ddea710b3fb7b62da'/>
<id>urn:sha1:ed77a89c30fa03dcb234a84ddea710b3fb7b62da</id>
<content type='text'>
Conflicts:

	net/netfilter/nf_conntrack_netlink.c
</content>
</entry>
<entry>
<title>Merge branch 'master' of master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6</title>
<updated>2008-11-27T07:48:40Z</updated>
<author>
<name>David S. Miller</name>
<email>davem@davemloft.net</email>
</author>
<published>2008-11-27T07:48:40Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=5b9ab2ec04ec1e1e53939768805612ac191d7ba2'/>
<id>urn:sha1:5b9ab2ec04ec1e1e53939768805612ac191d7ba2</id>
<content type='text'>
Conflicts:

	drivers/net/hp-plus.c
	drivers/net/wireless/ath5k/base.c
	drivers/net/wireless/ath9k/recv.c
	net/wireless/reg.c
</content>
</entry>
<entry>
<title>bridge: netfilter: fix update_pmtu crash with GRE</title>
<updated>2008-11-25T00:06:50Z</updated>
<author>
<name>Herbert Xu</name>
<email>herbert@gondor.apana.org.au</email>
</author>
<published>2008-11-25T00:06:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=631339f1e544a4d39a63cfe6708c5bddcd5a2c48'/>
<id>urn:sha1:631339f1e544a4d39a63cfe6708c5bddcd5a2c48</id>
<content type='text'>
As GRE tries to call the update_pmtu function on skb-&gt;dst and
bridge supplies an skb-&gt;dst that has a NULL ops field, all is
not well.

This patch fixes this by giving the bridge device an ops field
with an update_pmtu function.  For the moment I've left all
other fields blank but we can fill them in later should the
need arise.

Based on report and patch by Philip Craig.

Signed-off-by: Herbert Xu &lt;herbert@gondor.apana.org.au&gt;
Signed-off-by: Patrick McHardy &lt;kaber@trash.net&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>netdev: add more functions to netdevice ops</title>
<updated>2008-11-21T04:14:53Z</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@vyatta.com</email>
</author>
<published>2008-11-21T04:14:53Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=008298231abbeb91bc7be9e8b078607b816d1a4a'/>
<id>urn:sha1:008298231abbeb91bc7be9e8b078607b816d1a4a</id>
<content type='text'>
This patch moves neigh_setup and hard_start_xmit into the network device ops
structure. For bisection, fix all the previously converted drivers as well.
Bonding driver took the biggest hit on this.

Added a prefetch of the hard_start_xmit in the fast path to try and reduce
any impact this would have.

Signed-off-by: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>bridge: convert to net_device_ops</title>
<updated>2008-11-20T06:42:38Z</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@vyatta.com</email>
</author>
<published>2008-11-20T05:49:00Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a2dbb88210b9877f1c53d3798fd5d717a4d45256'/>
<id>urn:sha1:a2dbb88210b9877f1c53d3798fd5d717a4d45256</id>
<content type='text'>
Convert to net_device_ops function table.

Signed-off-by: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>netdevice: safe convert to netdev_priv() #part-4</title>
<updated>2008-11-13T07:39:10Z</updated>
<author>
<name>Wang Chen</name>
<email>wangchen@cn.fujitsu.com</email>
</author>
<published>2008-11-13T07:39:10Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=524ad0a79126efabf58d0a49eace6155ab5b4549'/>
<id>urn:sha1:524ad0a79126efabf58d0a49eace6155ab5b4549</id>
<content type='text'>
We have some reasons to kill netdev-&gt;priv:
1. netdev-&gt;priv is equal to netdev_priv().
2. netdev_priv() wraps the calculation of netdev-&gt;priv's offset, obviously
   netdev_priv() is more flexible than netdev-&gt;priv.
But we cann't kill netdev-&gt;priv, because so many drivers reference to it
directly.

This patch is a safe convert for netdev-&gt;priv to netdev_priv(netdev).
Since all of the netdev-&gt;priv is only for read.
But it is too big to be sent in one mail.
I split it to 4 parts and make every part smaller than 100,000 bytes,
which is max size allowed by vger.

Signed-off-by: Wang Chen &lt;wangchen@cn.fujitsu.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
</feed>
