forked from luck/tmp_suning_uos_patched
[NET]: Nicer WARN_ON in netstat_show
The if (statement) WARN_ON(1); looks much better as WARN_ON(statement); Signed-off-by: Pavel Emelyanov <xemul@openvz.org> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
c7dc89c0ac
commit
df1b86c53d
@ -247,9 +247,8 @@ static ssize_t netstat_show(const struct device *d,
|
||||
struct net_device_stats *stats;
|
||||
ssize_t ret = -EINVAL;
|
||||
|
||||
if (offset > sizeof(struct net_device_stats) ||
|
||||
offset % sizeof(unsigned long) != 0)
|
||||
WARN_ON(1);
|
||||
WARN_ON(offset > sizeof(struct net_device_stats) ||
|
||||
offset % sizeof(unsigned long) != 0);
|
||||
|
||||
read_lock(&dev_base_lock);
|
||||
if (dev_isalive(dev) && dev->get_stats &&
|
||||
|
Loading…
Reference in New Issue
Block a user