[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20060914155118.GA4337@sergelap.austin.ibm.com>
Date: Thu, 14 Sep 2006 10:51:18 -0500
From: "Serge E. Hallyn" <serue@...ibm.com>
To: Cedric Le Goater <clg@...ibm.com>
Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...l.org>,
"Serge E. Hallyn" <serue@...ibm.com>,
Linux Containers <containers@...ts.osdl.org>
Subject: Re: [patch -mm] utsname namespace : fix unshare when CONFIG_UTS_NS is not set
Quoting Cedric Le Goater (clg@...ibm.com):
> If the kernel is not configured with the CONFIG_UTS_NS, unshare of
> ipc namespace will fail and return -EINVAL.
>
> The patch changes the dummy unshare_utsname() to check the clone flags
> before returning.
>
> Signed-off-by: Cedric Le Goater <clg@...ibm.com>
Acked-by: Serge E. Hallyn <serue@...ibm.com>
thanks,
-serge
> Cc: Andrew Morton <akpm@...l.org>
> Cc: Serge E. Hallyn <serue@...ibm.com>
> Cc: Linux Containers <containers@...ts.osdl.org>
>
> ---
> include/linux/utsname.h | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> Index: 2.6.18-rc6-mm2/include/linux/utsname.h
> ===================================================================
> --- 2.6.18-rc6-mm2.orig/include/linux/utsname.h
> +++ 2.6.18-rc6-mm2/include/linux/utsname.h
> @@ -60,8 +60,12 @@ static inline void put_uts_ns(struct uts
> static inline int unshare_utsname(unsigned long unshare_flags,
> struct uts_namespace **new_uts)
> {
> - return -EINVAL;
> + if (unshare_flags & CLONE_NEWUTS)
> + return -EINVAL;
> +
> + return 0;
> }
> +
> static inline int copy_utsname(int flags, struct task_struct *tsk)
> {
> return 0;
-
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/
Powered by blists - more mailing lists