summaryrefslogtreecommitdiff
path: root/drivers/net/ethernet/tehuti/tehuti.c
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-07-19 20:46:59 +0300
committerDavid S. Miller <davem@davemloft.net>2017-07-20 08:48:02 +0300
commit1e6c22aef28364dcc5f03c04a05ec463bc2b3431 (patch)
tree1ca42b7dbc6fc046a66b106936bcb63f4d330ad3 /drivers/net/ethernet/tehuti/tehuti.c
parent3753654e541938717b13f2b25791c3171a3a06aa (diff)
downloadlinux-1e6c22aef28364dcc5f03c04a05ec463bc2b3431.tar.xz
net: tehuti: don't process data if it has not been copied from userspace
The array data is only populated with valid information from userspace if cmd != SIOCDEVPRIVATE, other cases the array contains garbage on the stack. The subsequent switch statement acts on a subcommand in data[0] which could be any garbage value if cmd is SIOCDEVPRIVATE which seems incorrect to me. Instead, just return EOPNOTSUPP for the case where cmd == SIOCDEVPRIVATE to avoid this issue. As a side note, I suspect that the original intention of the code was for this ioctl to work just for cmd == SIOCDEVPRIVATE (and the current logic is reversed). However, I don't wont to change the current semantics in case any userspace code relies on this existing behaviour. Detected by CoverityScan, CID#139647 ("Uninitialized scalar variable") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/tehuti/tehuti.c')
-rw-r--r--drivers/net/ethernet/tehuti/tehuti.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/ethernet/tehuti/tehuti.c b/drivers/net/ethernet/tehuti/tehuti.c
index 711fbbbc4b1f..163d8d16bc24 100644
--- a/drivers/net/ethernet/tehuti/tehuti.c
+++ b/drivers/net/ethernet/tehuti/tehuti.c
@@ -654,6 +654,8 @@ static int bdx_ioctl_priv(struct net_device *ndev, struct ifreq *ifr, int cmd)
RET(-EFAULT);
}
DBG("%d 0x%x 0x%x\n", data[0], data[1], data[2]);
+ } else {
+ return -EOPNOTSUPP;
}
if (!capable(CAP_SYS_RAWIO))