Commit a3c74c52570c0c4ac90c9a0216de800c39089ba7
Committed by
Thomas Gleixner
1 parent
1dcc41bb03
Exists in
master
and in
39 other branches
futex: Mark restart_block.futex.uaddr[2] __user
@uaddr and @uaddr2 fields in restart_block.futex are user pointers. Add __user and remove unnecessary casts. Signed-off-by: Namhyung Kim <namhyung@gmail.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Darren Hart <dvhltc@us.ibm.com> LKML-Reference: <1284468228-8723-2-git-send-email-namhyung@gmail.com> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Showing 2 changed files with 4 additions and 4 deletions Side-by-side Diff
include/linux/thread_info.h
... | ... | @@ -23,12 +23,12 @@ |
23 | 23 | }; |
24 | 24 | /* For futex_wait and futex_wait_requeue_pi */ |
25 | 25 | struct { |
26 | - u32 *uaddr; | |
26 | + u32 __user *uaddr; | |
27 | 27 | u32 val; |
28 | 28 | u32 flags; |
29 | 29 | u32 bitset; |
30 | 30 | u64 time; |
31 | - u32 *uaddr2; | |
31 | + u32 __user *uaddr2; | |
32 | 32 | } futex; |
33 | 33 | /* For nanosleep */ |
34 | 34 | struct { |
kernel/futex.c
... | ... | @@ -1843,7 +1843,7 @@ |
1843 | 1843 | |
1844 | 1844 | restart = ¤t_thread_info()->restart_block; |
1845 | 1845 | restart->fn = futex_wait_restart; |
1846 | - restart->futex.uaddr = (u32 *)uaddr; | |
1846 | + restart->futex.uaddr = uaddr; | |
1847 | 1847 | restart->futex.val = val; |
1848 | 1848 | restart->futex.time = abs_time->tv64; |
1849 | 1849 | restart->futex.bitset = bitset; |
... | ... | @@ -1869,7 +1869,7 @@ |
1869 | 1869 | |
1870 | 1870 | static long futex_wait_restart(struct restart_block *restart) |
1871 | 1871 | { |
1872 | - u32 __user *uaddr = (u32 __user *)restart->futex.uaddr; | |
1872 | + u32 __user *uaddr = restart->futex.uaddr; | |
1873 | 1873 | int fshared = 0; |
1874 | 1874 | ktime_t t, *tp = NULL; |
1875 | 1875 |