projects / cascardo / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: abb1cf3) | patch
Revert "[PATCH] usb: drivers/usb/core/devio.c dereferences a userspace pointer"
authorLinus Torvalds <torvalds@g5.osdl.org>
Sun, 25 Jun 2006 00:47:09 +0000 (17:47 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Sun, 25 Jun 2006 00:47:09 +0000 (17:47 -0700)
commit83626b01275d0228516b4d97da008328fc37c934
tree7414f0b4ee877e10e40671df03425e362dd43c43tree | snapshot
parentabb1cf3cb9532d824c8bda00a111adb106dd457dcommit | diff
Revert "[PATCH] usb: drivers/usb/core/devio.c dereferences a userspace pointer"

This reverts commit 786dc1d3d7333f269e17d742886eac2188a2d9cc.

As Al so eloquently points out, the patch is crap. The old code was fine,
the new code was bogus.

It never dereferenced a user pointer, the "->" operator was to an array
member, which gives the _address_ of the member (in user space), not an
actual dereference at all.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/usb/core/devio.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.