summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Weißschuh <linux@weissschuh.net>2025-10-28 10:15:42 +0100
committerJohannes Berg <johannes.berg@intel.com>2025-11-06 13:02:33 +0100
commit880f615bf96eff89eab88b4f9aacf527cf55a714 (patch)
treeb2f0e58d93a24e684d093ed7f706e12371a4ab50
parent78fdfc9fc4215add97fe331aff7c64e4a423a104 (diff)
um: Remove redundant range check from __access_ok_vsyscall()
The only caller __access_ok() is already doing the same check through __addr_range_nowrap(). Remove the redundant check. Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> Link: https://patch.msgid.link/20251028-uml-remove-32bit-pseudo-vdso-v1-7-e930063eff5f@weissschuh.net Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r--arch/um/include/asm/uaccess.h3
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/um/include/asm/uaccess.h b/arch/um/include/asm/uaccess.h
index 1c6e0ae41b0c..3770bdeee100 100644
--- a/arch/um/include/asm/uaccess.h
+++ b/arch/um/include/asm/uaccess.h
@@ -17,8 +17,7 @@
#define __access_ok_vsyscall(addr, size) \
(((unsigned long) (addr) >= FIXADDR_USER_START) && \
- ((unsigned long) (addr) + (size) <= FIXADDR_USER_END) && \
- ((unsigned long) (addr) + (size) >= (unsigned long)(addr)))
+ ((unsigned long) (addr) + (size) <= FIXADDR_USER_END))
#define __addr_range_nowrap(addr, size) \
((unsigned long) (addr) <= ((unsigned long) (addr) + (size)))