brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

xwayland: Prevent some boolean confusion

Browse Source 
The original code in Weston that this was ported from returned an errno,
not a boolean, so we were inadvertently returning TRUE here during an
error path. Fix that up.
This commit is contained in:
Jasper St. Pierre 2015-03-21 18:25:02 -07:00
parent a13f906ed1
commit a86368dcb1
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/wayland/meta-xwayland.c
View File

@ -445,14 +445,14 @@ meta_xwayland_start (MetaXWaylandManager *manager,
{ {
g_warning ("xwayland_client_fd socketpair failed\n"); g_warning ("xwayland_client_fd socketpair failed\n");
unlink (manager->lockfile); unlink (manager->lockfile);
return 1; return FALSE;
} }
if (socketpair (AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0, displayfd) < 0) if (socketpair (AF_UNIX, SOCK_STREAM | SOCK_CLOEXEC, 0, displayfd) < 0)
{ {
g_warning ("displayfd socketpair failed\n"); g_warning ("displayfd socketpair failed\n");
unlink (manager->lockfile); unlink (manager->lockfile);
return 1; return FALSE;
} }
manager->pid = fork (); manager->pid = fork ();