Re: [PATCH] mq_open() honor leading slash

From: Chris Wright
Date: Thu Apr 15 2004 - 15:42:18 EST


* Manfred Spraul (manfred@xxxxxxxxxxxxxxxx) wrote:
> Chris Wright wrote:
> >Patch below simply eats all leading slashes before passing name to
> >lookup_one_len() in mq_open() and mq_unlink().
> >
> Why should we do that in kernel space?
> The kernel interface is "no slash at all". User space can add a SuS
> compatible layer on top of the kernel interface (i.e. fail with -EINVAL
> if the first character is not a /, then skip the slash and pass "name+1"
> to sys_mq_open()).

Jakub said similar, and I missed the fact that glibc was sanitizing
before calling kernel. I agree, it's fine as is.

thanks,
-chris
--
Linux Security Modules http://lsm.immunix.org http://lsm.bkbits.net
-
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/