[PATCH 02/54] Don't make failure to -nolisten fatal
Alan Coopersmith
alan.coopersmith at oracle.com
Mon Jul 9 19:32:19 PDT 2012
On 07/ 9/12 06:02 PM, Daniel Stone wrote:
> If failing to disable a protocol specified by -nolisten failed, we'd
> throw a FatalError and bomb startup entirely. From poking at xtrans, it
> looks like the only way we can get a failure here is because we've
> specified a protocol name which doesn't exist, which probably doesn't
> constitute a security risk.
>
> And it makes it possible to start gdm even though you've built with
> --disable-tcp-transport.
>
> Signed-off-by: Daniel Stone <daniel at fooishbar.org>
> ---
> os/utils.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/os/utils.c b/os/utils.c
> index 2537934..b00d38b 100644
> --- a/os/utils.c
> +++ b/os/utils.c
> @@ -757,8 +757,8 @@ ProcessCommandLine(int argc, char *argv[])
> else if (strcmp(argv[i], "-nolisten") == 0) {
> if (++i < argc) {
> if (_XSERVTransNoListen(argv[i]))
> - FatalError("Failed to disable listen for %s transport",
> - argv[i]);
> + ErrorF("Failed to disable listen for %s transport",
> + argv[i]);
> }
> else
> UseMsg();
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the xorg-devel
mailing list