Commit 8ed7ec13 authored by Andrew Lunn's avatar Andrew Lunn Committed by Jakub Kicinski

drivers: net: xen-netfront: Fixed W=1 set but unused warnings

drivers/net/xen-netfront.c:2416:16: warning: variable ‘target’ set but not used [-Wunused-but-set-variable]
 2416 |  unsigned long target;

Remove target and just discard the return value from simple_strtoul().

This patch does give a checkpatch warning, but the warning was there
before anyway, as this file has lots of checkpatch warnings.
Signed-off-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/20201031180435.1081127-1-andrew@lunn.chSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent ce3f9521
...@@ -2413,12 +2413,11 @@ static ssize_t store_rxbuf(struct device *dev, ...@@ -2413,12 +2413,11 @@ static ssize_t store_rxbuf(struct device *dev,
const char *buf, size_t len) const char *buf, size_t len)
{ {
char *endp; char *endp;
unsigned long target;
if (!capable(CAP_NET_ADMIN)) if (!capable(CAP_NET_ADMIN))
return -EPERM; return -EPERM;
target = simple_strtoul(buf, &endp, 0); simple_strtoul(buf, &endp, 0);
if (endp == buf) if (endp == buf)
return -EBADMSG; return -EBADMSG;
......
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