diff options
| author |   Wilson Kok <wkok@cumulusnetworks.com> | 2015-05-25 06:39:31 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-05-25 20:29:54 -0400 |
| commit | eb8d7baae21584a7b27e79c2deca3eee72376ff0 (patch) | |
| tree | d8bbfc35fd7d7021a31fa0cc941bbaec8ef74ab8 /net | |
| parent | pktgen: remove one sparse error (diff) | |
| download | linux-dev-eb8d7baae21584a7b27e79c2deca3eee72376ff0.tar.xz linux-dev-eb8d7baae21584a7b27e79c2deca3eee72376ff0.zip | |
bridge: skip fdb add if the port shouldn't learn
Check in fdb_add_entry() if the source port should learn, similar
check is used in br_fdb_update.
Note that new fdb entries which are added manually or
as local ones are still permitted.
This patch has been tested by running traffic via a bridge port and
switching the port's state, also by manually adding/removing entries
from the bridge's fdb.
Signed-off-by: Wilson Kok <wkok@cumulusnetworks.com>
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/bridge/br_fdb.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c index e0670d7054f9..7896cf143045 100644 --- a/net/bridge/br_fdb.c +++ b/net/bridge/br_fdb.c @@ -736,6 +736,12 @@ static int fdb_add_entry(struct net_bridge_port *source, const __u8 *addr, struct net_bridge_fdb_entry *fdb; bool modified = false; + /* If the port cannot learn allow only local and static entries */ + if (!(state & NUD_PERMANENT) && !(state & NUD_NOARP) && + !(source->state == BR_STATE_LEARNING || + source->state == BR_STATE_FORWARDING)) + return -EPERM; + fdb = fdb_find(head, addr, vid); if (fdb == NULL) { if (!(flags & NLM_F_CREATE)) |