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 | |
| 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')
| -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) |
