diff options
author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2007-10-30 23:40:45 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-31 00:29:42 -0400 |
commit | be48be08a829db09a4f786f44a1872ef0f533c85 (patch) | |
tree | f9d32e4c00eb8d08ac0a765182a64d7fc62e2971 | |
parent | 51c739d1f484b2562040a3e496dc8e1670d4e279 (diff) |
[COMPAT]: Fix new dev_ifname32 returning -EFAULT
A stray semicolon slipped in the patch that updated dev_ifname32 to
not be inline, causing it to always return -EFAULT. This fixes it.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | fs/compat_ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c index a4284ccac1f9..bd26e4cbb994 100644 --- a/fs/compat_ioctl.c +++ b/fs/compat_ioctl.c | |||
@@ -322,7 +322,7 @@ static int dev_ifname32(unsigned int fd, unsigned int cmd, unsigned long arg) | |||
322 | int err; | 322 | int err; |
323 | 323 | ||
324 | uifr = compat_alloc_user_space(sizeof(struct ifreq)); | 324 | uifr = compat_alloc_user_space(sizeof(struct ifreq)); |
325 | if (copy_in_user(uifr, compat_ptr(arg), sizeof(struct ifreq32))); | 325 | if (copy_in_user(uifr, compat_ptr(arg), sizeof(struct ifreq32))) |
326 | return -EFAULT; | 326 | return -EFAULT; |
327 | 327 | ||
328 | err = sys_ioctl(fd, SIOCGIFNAME, (unsigned long)uifr); | 328 | err = sys_ioctl(fd, SIOCGIFNAME, (unsigned long)uifr); |