chiark / gitweb /
fix wrong parameter size on ioctl FIONREAD
authorAndrew Church <gentoo4@achurch.org>
Thu, 24 Sep 2009 17:51:12 +0000 (10:51 -0700)
committerKay Sievers <kay.sievers@vrfy.org>
Thu, 24 Sep 2009 17:53:10 +0000 (10:53 -0700)
commit4daa146bf71cea174271371a0eb3cf22719a550b
treecbc5cce64745d6eaaeca4d78c82257bb7275a366
parent49c3a01d444052169363030dfd996fc7fd6a4fad
fix wrong parameter size on ioctl FIONREAD

On Wed, Sep 23, 2009 at 23:11, Matthias Schwarzott <zzam@gentoo.org> wrote:
> It is about ioctl failures on amd64:
>   http://bugs.gentoo.org/show_bug.cgi?id=286041
>
> A bad parameter type to an ioctl() call causes udev-146 to generate "error
> getting buffer for inotify" messages in syslog.  The offending code is
> roughly:
>
>    ssize_t nbytes, pos;
>    // ...
>    ioctl(fd, FIONREAD, &nbytes);
>
> where ssize_t is 64 bits on amd64, but the kernel code for FIONREAD (at least
> through gentoo-sources-2.6.31) uses type int:
>
>    p = (void __user *) arg;
>    switch (cmd) {
>    case FIONREAD:
>        // ...
>        ret = put_user(send_len, (int __user *) p);
>
> so the upper 32 bits of "nbytes" are left uninitialized, and the subsequent
> malloc(nbytes) fails unless those 32 bits happen to be zero (or the system has
> a LOT of memory).
udev/udevd.c