Skip to contentSkip to conversation

lkdtm: Add tests for put_user/get_user

PATCH New Laura Abbott 1 participant 1 message Nov 1, 2017 Char and misc drivers

lore
Share & tools
lore

Laura Abbott

author
#0 lore raw

PATCH 1/1 lkdtm: Add tests for put_user/get_user

{get,put}_user are designed to be 'safe' access and verify the address passed in whereas __{get,put}_user are supposed to be 'unsafe'. Add a test to verify this behavior.

Signed-off-by Laura Abbott <[email protected]>

---
Inspired by the paranoid __{get,put}_user checks for arm64.

---
 drivers/misc/lkdtm.h          |   2 +
 drivers/misc/lkdtm_core.c     |   2 +
 drivers/misc/lkdtm_usercopy.c | 107 ++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 111 insertions(+)
drivers/misc/lkdtm.h +2 -0
index bfb6c45b6130..c80236198c96 100644
@@ -81,5 +81,7 @@ void lkdtm_USERCOPY_STACK_FRAME_TO(void);
 void lkdtm_USERCOPY_STACK_FRAME_FROM(void);
 void lkdtm_USERCOPY_STACK_BEYOND(void);
 void lkdtm_USERCOPY_KERNEL(void);
+void lkdtm_PUT_USER(void);
+void lkdtm_GET_USER(void);
 
 #endif
drivers/misc/lkdtm_core.c +2 -0
index 981b3ef71e47..f0a99ccc2a5a 100644
@@ -251,6 +251,8 @@ struct crashtype crashtypes[] = {
 	CRASHTYPE(USERCOPY_STACK_FRAME_FROM),
 	CRASHTYPE(USERCOPY_STACK_BEYOND),
 	CRASHTYPE(USERCOPY_KERNEL),
+	CRASHTYPE(PUT_USER),
+	CRASHTYPE(GET_USER),
 };
drivers/misc/lkdtm_usercopy.c +107 -0

113 lines not on this page · raw message

Signature · 1 line
2.13.5