Can we make sure this gets backported to 1.1, too?

> -----Original Message-----
> From: Eoff, Ullysses A
> Sent: Monday, June 03, 2013 4:23 PM
> To: [email protected]
> Cc: Eoff, Ullysses A
> Subject: [PATCH weston v2] use _exit instead of exit if client fails to exec
> 
> From: "U. Artie Eoff" <[email protected]>
> 
> exit() calls atexit() handlers and C++ destructors (e.g. a C++
> weston module) which could destroy state that the main process
> depends on (e.g. ioctl's, tmpfiles, sockets, etc...).  If an exec
> fails, call _exit() instead of exit().
> 
> v2: prefer _exit over _Exit
> 
> Signed-off-by: U. Artie Eoff <[email protected]>
> ---
>  src/compositor.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/compositor.c b/src/compositor.c
> index f382340..37fce0a 100644
> --- a/src/compositor.c
> +++ b/src/compositor.c
> @@ -221,7 +221,7 @@ weston_client_launch(struct weston_compositor *compositor,
> 
>       if (pid == 0) {
>               child_client_exec(sv[1], path);
> -             exit(-1);
> +             _exit(-1);
>       }
> 
>       close(sv[1]);
> --
> 1.7.11.7

_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to