linux-kernel - Re: [PATCH] change inotifyfs magic as the same magic is used for futexfs (v2)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20070924080808.83be8eed.randy.dunlap@oracle.com>
Date:	Mon, 24 Sep 2007 08:08:08 -0700
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	Andrey Mirkin <major@...nvz.org>
Cc:	John McCutchan <ttb@...tacle.dhs.org>,
	Robert Love <linux-kernel@...r.kernel.org>,
	Ingo Molnar <mingo@...hat.com>, Andrew Morton <akpm@...l.org>,
	Jan Engelhardt <jengelh@...putergmbh.de>, devel@...nvz.org
Subject: Re: [PATCH] change inotifyfs magic as the same magic is used for
 futexfs (v2)

On Mon, 24 Sep 2007 12:46:06 +0400 Andrey Mirkin wrote:

> From: Andrey Mirkin <major@...nvz.org>
> 
> Right now futexfs and inotifyfs have one magic 0xBAD1DEA, that looks a little 
> bit confusing.
> Use 0xBAD1DEA as magic for futexfs and 0x2BAD1DEA as magic for inotifyfs.
> 
> Signed-off-by: Andrey Mirkin <major@...nvz.org>

Acked-by: Randy Dunlap <randy.dunlap@...cle.com>

> ----
> 
>  fs/inotify_user.c     |    4 +++-
>  include/linux/magic.h |    3 +++
>  kernel/futex.c        |    3 ++-
>  3 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/inotify_user.c b/fs/inotify_user.c
> index 9bf2f6c..5e00933 100644
> --- a/fs/inotify_user.c
> +++ b/fs/inotify_user.c
> @@ -31,6 +31,7 @@ #include <linux/init.h>
>  #include <linux/list.h>
>  #include <linux/inotify.h>
>  #include <linux/syscalls.h>
> +#include <linux/magic.h>
>  
>  #include <asm/ioctls.h>
>  
> @@ -684,7 +685,8 @@ static int
>  inotify_get_sb(struct file_system_type *fs_type, int flags,
>  	       const char *dev_name, void *data, struct vfsmount *mnt)
>  {
> -	return get_sb_pseudo(fs_type, "inotify", NULL, 0xBAD1DEA, mnt);
> +	return get_sb_pseudo(fs_type, "inotify", NULL,
> +			INOTIFYFS_SUPER_MAGIC, mnt);
>  }
>  
>  static struct file_system_type inotify_fs_type = {
> diff --git a/include/linux/futex.h b/include/linux/futex.h
> diff --git a/include/linux/inotify.h b/include/linux/inotify.h
> diff --git a/include/linux/magic.h b/include/linux/magic.h
> index 36cc20d..722d475 100644
> --- a/include/linux/magic.h
> +++ b/include/linux/magic.h
> @@ -38,4 +38,7 @@ #define REISER2FS_JR_SUPER_MAGIC_STRING	
>  #define SMB_SUPER_MAGIC		0x517B
>  #define USBDEVICE_SUPER_MAGIC	0x9fa2
>  
> +#define FUTEXFS_SUPER_MAGIC	0xBAD1DEA
> +#define INOTIFYFS_SUPER_MAGIC	0x2BAD1DEA
> +
>  #endif /* __LINUX_MAGIC_H__ */
> diff --git a/kernel/futex.c b/kernel/futex.c
> index e8935b1..856dfba 100644
> --- a/kernel/futex.c
> +++ b/kernel/futex.c
> @@ -52,6 +52,7 @@ #include <linux/pagemap.h>
>  #include <linux/syscalls.h>
>  #include <linux/signal.h>
>  #include <linux/module.h>
> +#include <linux/magic.h>
>  #include <asm/futex.h>
>  
>  #include "rtmutex_common.h"
> @@ -2074,7 +2075,7 @@ static int futexfs_get_sb(struct file_sy
>  			  int flags, const char *dev_name, void *data,
>  			  struct vfsmount *mnt)
>  {
> -	return get_sb_pseudo(fs_type, "futex", NULL, 0xBAD1DEA, mnt);
> +	return get_sb_pseudo(fs_type, "futex", NULL, FUTEXFS_SUPER_MAGIC, mnt);
>  }
>  
>  static struct file_system_type futex_fs_type = {
> -

---
~Randy
Phaedrus says that Quality is about caring.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/