diff options
author | Roopa Prabhu <roopa@cumulusnetworks.com> | 2015-02-05 22:24:45 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-02-07 22:49:47 -0800 |
commit | 1fd0bddb618aa970b87e7907a5703682aa119a6f (patch) | |
tree | d8ed10105a874cdee1f6a7aa290ccc655c22d257 /net/can | |
parent | d78f802f18ed100c53f331cd59791cd82ccb9438 (diff) | |
download | linux-1fd0bddb618aa970b87e7907a5703682aa119a6f.tar.gz linux-1fd0bddb618aa970b87e7907a5703682aa119a6f.tar.bz2 linux-1fd0bddb618aa970b87e7907a5703682aa119a6f.zip |
bridge: add missing bridge port check for offloads
This patch fixes a missing bridge port check caught by smatch.
setlink/dellink of attributes like vlans can come for a bridge device
and there is no need to offload those today. So, this patch adds a bridge
port check. (In these cases however, the BRIDGE_SELF flags will always be set
and we may not hit a problem with the current code).
smatch complaint:
The patch 68e331c785b8: "bridge: offload bridge port attributes to
switch asic if feature flag set" from Jan 29, 2015, leads to the
following Smatch complaint:
net/bridge/br_netlink.c:552 br_setlink()
error: we previously assumed 'p' could be null (see line 518)
net/bridge/br_netlink.c
517
518 if (p && protinfo) {
^
Check for NULL.
Reported-By: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/can')
0 files changed, 0 insertions, 0 deletions