Re: [patch -mm] utsname namespace : fix unshare when CONFIG_UTS_NS is not set

[Serge E. Hallyn]

Quoting Cedric Le Goater (clg@xxxxxxxxxx):
> 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@xxxxxxxxxx>

Acked-by: Serge E. Hallyn <serue@xxxxxxxxxx>

thanks,
-serge

> Cc: Andrew Morton <akpm@xxxxxxxx>
> Cc: Serge E. Hallyn <serue@xxxxxxxxxx>
> Cc: Linux Containers <containers@xxxxxxxxxxxxxx>
> 
> ---
>  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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/