summaryrefslogtreecommitdiffstats
path: root/drivers/net/tehuti.c
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2008-04-29 11:45:16 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2008-04-29 11:45:16 -0700
commit6203554207728f43cfb9fd48585cd6500da73d42 (patch)
treea1e486f288c3ef0d29c4367a058fe906ece763e3 /drivers/net/tehuti.c
parent9781db7b345b5dfe93787aaaf310c861db7c1ede (diff)
downloadop-kernel-dev-6203554207728f43cfb9fd48585cd6500da73d42.zip
op-kernel-dev-6203554207728f43cfb9fd48585cd6500da73d42.tar.gz
drivers/net/tehuti: use proper capability check for raw IO access
Yeah, in practice they both mean "root", but Alan correctly points out that anybody who gets to do raw IO space accesses should really be using CAP_SYS_RAWIO rather than CAP_NET_ADMIN. Pointed-out-by: Alan Cox <alan@lxorguk.ukuu.org.uk> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/net/tehuti.c')
-rw-r--r--drivers/net/tehuti.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/tehuti.c b/drivers/net/tehuti.c
index e83b166..432e837 100644
--- a/drivers/net/tehuti.c
+++ b/drivers/net/tehuti.c
@@ -649,7 +649,7 @@ static int bdx_ioctl_priv(struct net_device *ndev, struct ifreq *ifr, int cmd)
DBG("%d 0x%x 0x%x\n", data[0], data[1], data[2]);
}
- if (!capable(CAP_NET_ADMIN))
+ if (!capable(CAP_SYS_RAWIO))
return -EPERM;
switch (data[0]) {
OpenPOWER on IntegriCloud