Commit 16f3051b authored by Adrian Bunk's avatar Adrian Bunk Committed by David S. Miller

[VLAN] net/8021q/vlanproc.c: fix check-after-use

The Coverity checker spotted that we'd have already oops'ed if
"vlandev" was NULL.
Signed-off-by: default avatarAdrian Bunk <bunk@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6f93b9c2
...@@ -319,7 +319,7 @@ static int vlandev_seq_show(struct seq_file *seq, void *offset) ...@@ -319,7 +319,7 @@ static int vlandev_seq_show(struct seq_file *seq, void *offset)
static const char fmt[] = "%30s %12lu\n"; static const char fmt[] = "%30s %12lu\n";
int i; int i;
if ((vlandev == NULL) || (!(vlandev->priv_flags & IFF_802_1Q_VLAN))) if (!(vlandev->priv_flags & IFF_802_1Q_VLAN))
return 0; return 0;
seq_printf(seq, "%s VID: %d REORDER_HDR: %i dev->priv_flags: %hx\n", seq_printf(seq, "%s VID: %d REORDER_HDR: %i dev->priv_flags: %hx\n",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment