diff options
author | John Fastabend <john.r.fastabend@intel.com> | 2011-06-21 03:35:04 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-06-21 19:06:12 -0400 |
commit | 4003b65871c101eb5ce8f37a325feac54aa5c681 (patch) | |
tree | 67f7f335bbc5dc1043f69bde2ba7cd54ee468fe7 /net/dcb | |
parent | ab6baf980b095c70a56c5eb2f58166aef8a0edc8 (diff) |
dcb: Add missing error check in dcb_ieee_set()
Missing error checking before nla_parse_nested().
Reported-by: Mark Rustad <mark.d.rustad@intel.com>
Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dcb')
-rw-r--r-- | net/dcb/dcbnl.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/dcb/dcbnl.c b/net/dcb/dcbnl.c index f54c784e2b5b..e954d4c6590c 100644 --- a/net/dcb/dcbnl.c +++ b/net/dcb/dcbnl.c | |||
@@ -1368,6 +1368,9 @@ static int dcbnl_ieee_set(struct net_device *netdev, struct nlattr **tb, | |||
1368 | if (!ops) | 1368 | if (!ops) |
1369 | return err; | 1369 | return err; |
1370 | 1370 | ||
1371 | if (!tb[DCB_ATTR_IEEE]) | ||
1372 | return -EINVAL; | ||
1373 | |||
1371 | err = nla_parse_nested(ieee, DCB_ATTR_IEEE_MAX, | 1374 | err = nla_parse_nested(ieee, DCB_ATTR_IEEE_MAX, |
1372 | tb[DCB_ATTR_IEEE], dcbnl_ieee_policy); | 1375 | tb[DCB_ATTR_IEEE], dcbnl_ieee_policy); |
1373 | if (err) | 1376 | if (err) |