summaryrefslogtreecommitdiffstats
path: root/net/bridge
diff options
context:
space:
mode:
authorWilson Kok <wkok@cumulusnetworks.com>2015-05-25 06:39:31 -0700
committerDavid S. Miller <davem@davemloft.net>2015-05-25 20:29:54 -0400
commiteb8d7baae21584a7b27e79c2deca3eee72376ff0 (patch)
treed8bbfc35fd7d7021a31fa0cc941bbaec8ef74ab8 /net/bridge
parentd4969581452be430135ba5040c5b6c9023c1d461 (diff)
downloadop-kernel-dev-eb8d7baae21584a7b27e79c2deca3eee72376ff0.zip
op-kernel-dev-eb8d7baae21584a7b27e79c2deca3eee72376ff0.tar.gz
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/bridge')
-rw-r--r--net/bridge/br_fdb.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c
index e0670d7..7896cf1 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))
OpenPOWER on IntegriCloud