|
@@ -22,7 +22,7 @@
|
|
#include <asm/uaccess.h>
|
|
#include <asm/uaccess.h>
|
|
#include "br_private.h"
|
|
#include "br_private.h"
|
|
|
|
|
|
-/* net device transmit always called with no BH (preempt_disabled) */
|
|
|
|
|
|
+/* net device transmit always called with BH disabled */
|
|
netdev_tx_t br_dev_xmit(struct sk_buff *skb, struct net_device *dev)
|
|
netdev_tx_t br_dev_xmit(struct sk_buff *skb, struct net_device *dev)
|
|
{
|
|
{
|
|
struct net_bridge *br = netdev_priv(dev);
|
|
struct net_bridge *br = netdev_priv(dev);
|
|
@@ -48,13 +48,16 @@ netdev_tx_t br_dev_xmit(struct sk_buff *skb, struct net_device *dev)
|
|
skb_reset_mac_header(skb);
|
|
skb_reset_mac_header(skb);
|
|
skb_pull(skb, ETH_HLEN);
|
|
skb_pull(skb, ETH_HLEN);
|
|
|
|
|
|
|
|
+ rcu_read_lock();
|
|
if (is_multicast_ether_addr(dest)) {
|
|
if (is_multicast_ether_addr(dest)) {
|
|
if (unlikely(netpoll_tx_running(dev))) {
|
|
if (unlikely(netpoll_tx_running(dev))) {
|
|
br_flood_deliver(br, skb);
|
|
br_flood_deliver(br, skb);
|
|
goto out;
|
|
goto out;
|
|
}
|
|
}
|
|
- if (br_multicast_rcv(br, NULL, skb))
|
|
|
|
|
|
+ if (br_multicast_rcv(br, NULL, skb)) {
|
|
|
|
+ kfree_skb(skb);
|
|
goto out;
|
|
goto out;
|
|
|
|
+ }
|
|
|
|
|
|
mdst = br_mdb_get(br, skb);
|
|
mdst = br_mdb_get(br, skb);
|
|
if (mdst || BR_INPUT_SKB_CB_MROUTERS_ONLY(skb))
|
|
if (mdst || BR_INPUT_SKB_CB_MROUTERS_ONLY(skb))
|
|
@@ -67,6 +70,7 @@ netdev_tx_t br_dev_xmit(struct sk_buff *skb, struct net_device *dev)
|
|
br_flood_deliver(br, skb);
|
|
br_flood_deliver(br, skb);
|
|
|
|
|
|
out:
|
|
out:
|
|
|
|
+ rcu_read_unlock();
|
|
return NETDEV_TX_OK;
|
|
return NETDEV_TX_OK;
|
|
}
|
|
}
|
|
|
|
|