[PATCH weston v2] use _exit instead of exit if client fails to exec
U. Artie Eoff
ullysses.a.eoff at intel.com
Mon Jun 3 16:22:31 PDT 2013
From: "U. Artie Eoff" <ullysses.a.eoff at intel.com>
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 <ullysses.a.eoff at intel.com>
---
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
More information about the wayland-devel
mailing list