lkdtm: silence warnings about function declarations
authorKees Cook <keescook@chromium.org>
Fri, 15 Jul 2016 23:04:39 +0000 (16:04 -0700)
committerKees Cook <keescook@chromium.org>
Fri, 15 Jul 2016 23:14:45 +0000 (16:14 -0700)
When building under W=1, the lack of lkdtm.h in lkdtm_usercopy.c and
lkdtm_rodata.c was discovered. This fixes the issue and consolidates
the common header and the pr_fmt macro for simplicity and regularity
across each test source file.

Signed-off-by: Kees Cook <keescook@chromium.org>
drivers/misc/lkdtm.h
drivers/misc/lkdtm_bugs.c
drivers/misc/lkdtm_core.c
drivers/misc/lkdtm_heap.c
drivers/misc/lkdtm_perms.c
drivers/misc/lkdtm_rodata.c
drivers/misc/lkdtm_usercopy.c

index d70a41741bb3ac7473b5881c7d8d348979233674..fdf954c2107f935a36ca719ee6cd760a3d6e9182 100644 (file)
@@ -1,6 +1,10 @@
 #ifndef __LKDTM_H
 #define __LKDTM_H
 
+#define pr_fmt(fmt) "lkdtm: " fmt
+
+#include <linux/kernel.h>
+
 /* lkdtm_bugs.c */
 void __init lkdtm_bugs_init(int *recur_param);
 void lkdtm_PANIC(void);
@@ -53,5 +57,4 @@ void lkdtm_USERCOPY_STACK_FRAME_FROM(void);
 void lkdtm_USERCOPY_STACK_BEYOND(void);
 void lkdtm_USERCOPY_KERNEL(void);
 
-
 #endif
index e87071f9c003fa22f1d7c86fc6c82f4feedaea8a..182ae1894b328861323c42767aadd5f2588318b9 100644 (file)
@@ -4,12 +4,8 @@
  * lockups) along with other things that don't fit well into existing LKDTM
  * test source files.
  */
-#define pr_fmt(fmt) "lkdtm: " fmt
-
-#include <linux/kernel.h>
-#include <linux/sched.h>
-
 #include "lkdtm.h"
+#include <linux/sched.h>
 
 /*
  * Make sure our attempts to over run the kernel stack doesn't trigger
index 717aad6d374b26f0968942a0289fed309aa3e94d..f9154b8d67f6974bd6a7f27c24be9e6b0b985ebe 100644 (file)
@@ -30,9 +30,7 @@
  *
  * See Documentation/fault-injection/provoke-crashes.txt for instructions
  */
-#define pr_fmt(fmt) "lkdtm: " fmt
-
-#include <linux/kernel.h>
+#include "lkdtm.h"
 #include <linux/fs.h>
 #include <linux/module.h>
 #include <linux/buffer_head.h>
@@ -49,8 +47,6 @@
 #include <linux/ide.h>
 #endif
 
-#include "lkdtm.h"
-
 #define DEFAULT_COUNT 10
 
 static int lkdtm_debugfs_open(struct inode *inode, struct file *file);
index 12f50e8dcbfe0be294b23a9dd987c5d11d8557a8..0f1581664c1c78e28149159f09f1c76e966fcc5d 100644 (file)
@@ -2,12 +2,8 @@
  * This is for all the tests relating directly to heap memory, including
  * page allocation and slab allocations.
  */
-#define pr_fmt(fmt) "lkdtm: " fmt
-
-#include <linux/kernel.h>
-#include <linux/slab.h>
-
 #include "lkdtm.h"
+#include <linux/slab.h>
 
 /*
  * This tries to stay within the next largest power-of-2 kmalloc cache
index 8201006502e24900dc3a5f9b04dcade27c13f227..45f1c0f966123c7b3870e4f42d724270f4f75e5f 100644 (file)
@@ -3,17 +3,13 @@
  * permissions: non-executable regions, non-writable regions, and
  * even non-readable regions.
  */
-#define pr_fmt(fmt) "lkdtm: " fmt
-
-#include <linux/kernel.h>
+#include "lkdtm.h"
 #include <linux/slab.h>
 #include <linux/vmalloc.h>
 #include <linux/mman.h>
 #include <linux/uaccess.h>
 #include <asm/cacheflush.h>
 
-#include "lkdtm.h"
-
 /* Whether or not to fill the target memory area with do_nothing(). */
 #define CODE_WRITE     true
 #define CODE_AS_IS     false
index 4d0d851f02b937e91242bc6cdadbffcf4e240a32..166b1db3969fbc1f74fed6882bae0995a5768922 100644 (file)
@@ -2,7 +2,7 @@
  * This includes functions that are meant to live entirely in .rodata
  * (via objcopy tricks), to validate the non-executability of .rodata.
  */
-#include <linux/kernel.h>
+#include "lkdtm.h"
 
 void lkdtm_rodata_do_nothing(void)
 {
index 9c748e819a35de0a47662229adf65fd8da0a1ad3..5a3fd76eec27b226ec4a9a9dd9655d0a2ae95759 100644 (file)
@@ -2,9 +2,7 @@
  * This is for all the tests related to copy_to_user() and copy_from_user()
  * hardening.
  */
-#define pr_fmt(fmt) "lkdtm: " fmt
-
-#include <linux/kernel.h>
+#include "lkdtm.h"
 #include <linux/slab.h>
 #include <linux/vmalloc.h>
 #include <linux/mman.h>