Skip to contentSkip to conversation

pramfs: test module

PATCH Superseded Marco Stornelli 4 participants 5 messages Oct 10, 2010 – Oct 13, 2010 VFS

v1 of 5 Superseded Latest is v5 →

lore
Share & tools
lore
#0 lore raw

PATCH 15/16 pramfs: test module

From: Marco Stornelli <[email protected]>

Test module.

---

fs/pramfs/pramfs_test.c +49 -0
diff -Nurp linux-2.6.36-orig/fs/pramfs/pramfs_test.c linux-2.6.36/fs/pramfs/pramfs_test.c
@@ -0,0 +1,49 @@
+/*
+ * FILE NAME fs/pramfs/namei.c
+ *
+ * BRIEF DESCRIPTION
+ *
+ * Pramfs test module.
+ *
+ * Copyright 2009-2010 Marco Stornelli <[email protected]>
+ * Copyright 2003 Sony Corporation
+ * Copyright 2003 Matsushita Electric Industrial Co., Ltd.
+ * 2003-2004 (c) MontaVista Software, Inc. , Steve Longerbeam
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+#include <linux/module.h>
+#include <linux/version.h>
+#include <linux/init.h>
+#include <linux/fs.h>
+#include "pram.h"
+
+int __init test_pramfs_write(void)
+{
+	struct pram_super_block *psb;
+
+	psb = get_pram_super();
+	if (!psb) {
+		printk(KERN_ERR
+		"%s: PRAMFS super block not found (not mounted?)\n",
+		__func__);
+		return 1;
+	}
+
+	/*
+	 * Attempt an unprotected clear of checksum information in the
+	 * superblock, this should cause a kernel page protection fault.
+	 */
+	printk("%s: writing to kernel VA %p\n", __func__, psb);
+	psb->s_sum = 0;
+
+	return 0;
+}
+
+void test_pramfs_write_cleanup(void) {}
+
+/* Module information */
+MODULE_LICENSE("GPL");
+module_init(test_pramfs_write);
+module_exit(test_pramfs_write_cleanup);
#1 lore raw
Quoted 30 lines · On Sun, 10 Oct 2010 18:37:49 +0200 Marco Stornelli wrote:

From: Marco Stornelli <[email protected]>

Test module.

---
diff -Nurp linux-2.6.36-orig/fs/pramfs/pramfs_test.c linux-2.6.36/fs/pramfs/pramfs_test.c

fs/pramfs/pramfs_test.c +20 -0
@@ -0,0 +1,49 @@
+/*
+ * FILE NAME fs/pramfs/namei.c
+ *
+ * BRIEF DESCRIPTION
+ *
+ * Pramfs test module.
+ *
+ * Copyright 2009-2010 Marco Stornelli <[email protected]>
+ * Copyright 2003 Sony Corporation
+ * Copyright 2003 Matsushita Electric Industrial Co., Ltd.
+ * 2003-2004 (c) MontaVista Software, Inc. , Steve Longerbeam
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+#include <linux/module.h>
+#include <linux/version.h>
+#include <linux/init.h>
+#include <linux/fs.h>
+#include "pram.h"

fs/pramfs/pramfs_test.c line 30

Quoted 10 lines
fs/pramfs/pramfs_test.c +10 -0
+
+int __init test_pramfs_write(void)
+{
+	struct pram_super_block *psb;
+
+	psb = get_pram_super();
+	if (!psb) {
+		printk(KERN_ERR
+		"%s: PRAMFS super block not found (not mounted?)\n",
+		__func__);

Above 2 lines need to indented more.

Quoted 9 lines
fs/pramfs/pramfs_test.c +9 -0
+		return 1;
+	}
+
+	/*
+	 * Attempt an unprotected clear of checksum information in the
+	 * superblock, this should cause a kernel page protection fault.
+	 */
+	printk("%s: writing to kernel VA %p\n", __func__, psb);
+	psb->s_sum = 0;

fs/pramfs/pramfs_test.c line 49

Quoted 10 lines
fs/pramfs/pramfs_test.c +10 -0
+
+	return 0;
+}
+
+void test_pramfs_write_cleanup(void) {}
+
+/* Module information */
+MODULE_LICENSE("GPL");
+module_init(test_pramfs_write);
+module_exit(test_pramfs_write_cleanup);

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

#2 lore raw

2010/10/10 Randy Dunlap <[email protected]>:

Quoted 4 lines · On Sun, 10 Oct 2010 18:37:49 +0200 Marco Stornelli wrote:

Above 2 lines need to indented more.

Ack.

Quoted 11 lines
Quoted 11 lines

+             return 1;
+     }

fs/pramfs/pramfs_test.c +9 -0
+
+     /*
+      * Attempt an unprotected clear of checksum information in the
+      * superblock, this should cause a kernel page protection fault.
+      */
+     printk("%s: writing to kernel VA %p\n", __func__, psb);
+     psb->s_sum = 0;
+
+     return 0;
Quoted 9 lines

fs/pramfs/pramfs_test.c line 49

Quoted 8 lines
fs/pramfs/pramfs_test.c +8 -0
+}
+
+void test_pramfs_write_cleanup(void) {}
+
+/* Module information */
+MODULE_LICENSE("GPL");
+module_init(test_pramfs_write);
+module_exit(test_pramfs_write_cleanup);

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#4 lore raw
Quoted 4 lines · Kieran Bingham

fs/pramfs/pramfs_test.c line 2

Quoted 3 lines
fs/pramfs/pramfs_test.c +2 -0
@@ -0,0 +1,49 @@
+/*
+ * FILE NAME fs/pramfs/namei.c

FILE NAME != namei.c

Yes, that's why you should never do such filename comments which are not only utterly prone to be out of data, but also 100% useless.