Seokmann Ju <[email protected]> writes:
> This patch fixes incorrect irq data type in the driver which led driver
> initialization failure in some cases.
> The problem was reported by Eric @. Biederman <[email protected]>.
We have a race going on, I read your first message then this one pops up.
But I will reply again for wider distribution.
Nacked-by: Eric Biederman.
The problem is that struct mcontroller is exported to user space
and is a packed structure so it has no free bytes.
Therefore you will break user space when you add bytes in the
middle of your structure. We can't do that. You either need
to add an extra field at the end (while keeping the structure size the
same) or you need to add a new ioctl that gets this information correct,
or you need to give up entirely on the idea of exporting the irq
number to user space this way.
What you need not to do is break the kernel<->user space ABI. That
is just confusing and couples the kernel and user space in unpleasant
ways.
I believe struct mraid_hba_info has the same problem but I could not
see where it was exported to user space.
Eric
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
