[Bug 2683] arch/sparc64/kernel/sunos_ioctl32.c: sys_ioctl wrong?

Adrian Bunk <[email protected]>
Newsgroups gmane.linux.ports.ultrapenguin
Message-ID <[email protected]>
Kernel Bugzilla #2683 contains the following report:

<--  snip  -->

Problem Description:
The potential error is detected by a static analysis tool in
/arch/sparc64/kernel/sunos_ioctl32.c:163-168

163   case _IOW('i', 21, struct ifreq): /* SIOCSIFMTU */
164         ret = sys_ioctl(fd, SIOCSIFMTU, arg);
165         goto out;
166   case _IOWR('i', 22, struct ifreq): /* SIOCGIFMTU */
167         ret = sys_ioctl(fd, SIOCGIFMTU, arg);
168         goto out;


May be changed to:
163   case _IOW('i', 21, struct ifreq32): /* SIOCSIFMTU */
164         ret = compat_sys_ioctl(fd, SIOCSIFMTU, arg);
165         goto out;
166   case _IOWR('i', 22, struct ifreq32): /* SIOCGIFMTU */
167         ret = compat_sys_ioctl(fd, SIOCGIFMTU, arg);
168         goto out;


Can anyone confirm whether it is a bug or not?

<--  snip  -->

Please review this issue.

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe ultralinux" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.