lkml.org 
[lkml]   [2008]   [May]   [27]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 27 May 2008 10:17:53 +0300
FromMarkku Savela <>
SubjectRe: Bug/misfeature of "securityfs"
I should not really reply from hip in the morning, ignore more
previous message. The "file exists" definitely comes from the module
init failing, because it finds the directory existing.

Here is some more detailed information and a test module
(unfortunately, just pasted in this message):

*shell1*
	*shell2*

         > cd /sys/kernel/security/
         > ls
         apparmor
 > sudo insmod foobar.ko
        > ls
        apparmor  foobar
 > sudo rmmod foobar.ko
        > ls
        apparmor
// so far fine, directory comes and goes...

 > sudo insmod foobar.ko
        > ls
        apparmor  foobar
        > cd foobar/
 > sudo rmmod foobar.ko
 > ls /sys/kernel/security/
apparmor  foobar
        > cd ..
 > ls /sys/kernel/security/
apparmor

// so far fine, directory goes when
// other shell exits it.

 > sudo insmod foobar.ko
        > cd foobar/
 > sudo rmmod foobar.ko
 > ls /sys/kernel/security/
apparmor  foobar
 > sudo insmod foobar.ko
insmod: error inserting 'foobar.ko': -1 File exists
        > cd ..
        > ls
        apparmor  foobar
// 'foobar' still exists, while module unloaded
// and shell also is out.

 > ls /sys/kernel/security/
apparmor  foobar
 > ls /sys/kernel/security/
apparmor  foobar
 > sudo insmod foobar.ko
insmod: error inserting 'foobar.ko': -1 File exists

// And, now only reboot seems to remove the directory"

-------------------------------------------------------------------
tail /var/log/syslog
May 27 09:49:13 maja kernel: [  192.771424] foobar: Module 'foobar' loaded
May 27 09:49:28 maja kernel: [  207.637838] foobar: Module 'foobar' unloaded
May 27 09:49:35 maja kernel: [  214.890241] foobar: Module 'foobar' loaded
May 27 09:49:46 maja kernel: [  225.719495] foobar: Module 'foobar' unloaded
May 27 09:50:19 maja kernel: [  258.782263] foobar: Module 'foobar' loaded
May 27 09:50:27 maja kernel: [  266.641655] foobar: Module 'foobar' unloaded
May 27 09:50:48 maja kernel: [  288.383145] foobar: Foobar FS create failed
May 27 09:51:11 maja kernel: [  311.323851] foobar: Foobar FS create failed

------------------------------------------------------------------
#include <linux/init.h>
#include <linux/module.h>
#include <linux/security.h>
#define FOOBAR_NAME "foobar"

#undef DBG
#undef DBGG
#undef INFO
#undef ERR

#ifdef FOOBAR_DEBUG
#	define DBG(args...) printk(KERN_DEBUG FOOBAR_NAME ": " args)
#else
#	define DBG(args...)
#endif
#define DBGG(args...)
#define INFO(args...) printk(KERN_INFO FOOBAR_NAME ": " args)
#define ERR(args...) printk(KERN_ERR FOOBAR_NAME ": " args)

static struct dentry* foobar_fs;

static void foobar_cleanup(void)
	{
	if (foobar_fs && !IS_ERR(foobar_fs))
		{
		securityfs_remove(foobar_fs);
		};
	}
static int __init foobar_init(void)
	{
	DBG("foobar_init\n");
	if (foobar_fs)
		{
		ERR("Foobar FS already exists\n");
		return -EEXIST;
		}
	foobar_fs = securityfs_create_dir(FOOBAR_NAME, NULL);
	if (IS_ERR(foobar_fs))
		{
		ERR("Foobar FS create failed\n");
		foobar_cleanup();
		return PTR_ERR(foobar_fs);
		}
	INFO("Module 'foobar' loaded\n");
	return 0;
	}
static void __exit foobar_exit(void)
	{
	DBG("foobar_exit\n");
	foobar_cleanup();
	INFO("Module 'foobar' unloaded\n");
	}
module_init(foobar_init);
module_exit(foobar_exit);

MODULE_LICENSE("GPL");
MODULE_AUTHOR("Markku Savela");




\
 
 \ /
  Last update: 2008-05-27 09:21    [from the cache]
©2003-2008