mirrors/weston
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

xwayland: Fix X11 lock file size confusion

Browse Source 
The X11 lock file was somewhat opaque. Into a sized array of 16
characters, we previously read 11 bytes. 61beda653b fixed the parsing of
this input to ensure that we only considered the first 10 bytes: this
has the effect of culling a LF byte at the end of the string.

This commit more explicitly NULLs the entire string before reading, and
trims trailing LF characters only.

It also adds some documentation by way of resizing pid, an explicit size
check on snprintf's return, and comments.

Verified manually that it emits lock files with a trailing \n, as Xorg
does. Also verified manually that it ignores misformatted lock files,
but accepts either \n or \0 in the trailing position.

Related Mutter issue: https://bugzilla.gnome.org/show_bug.cgi?id=774613

Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
This commit is contained in:
Daniel Stone 2016-11-16 16:37:05 +00:00
parent 97863d6f0d
commit 08a35d307f
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
xwayland/launcher.c
View File

@ -148,7 +148,9 @@ bind_to_unix_socket(int display)
static int
create_lockfile(int display, char *lockfile, size_t lsize)
{
char pid[16];
/* 10 decimal characters, trailing LF and NUL byte; see comment
* at end of function. */
char pid[11];
int fd, size;
pid_t other;
@ -166,7 +168,7 @@ create_lockfile(int display, char *lockfile, size_t lsize)
return -1;
}
/* Trim the newline, ensure terminated string. */
/* Trim the trailing LF, or at least ensure it's NULL. */
pid[10] = '\0';
if (!safe_strtoint(pid, &other)) {
@ -199,10 +201,12 @@ create_lockfile(int display, char *lockfile, size_t lsize)
return -1;
}
/* Subtle detail: we use the pid of the wayland
* compositor, not the xserver in the lock file. */
size = snprintf(pid, sizeof pid, "%10d\n", getpid());
if (write(fd, pid, size) != size) {
/* Subtle detail: we use the pid of the wayland compositor, not the
* xserver in the lock file.
* Also subtle is that we don't emit a trailing NUL to the file, so
* our size here is 11 rather than 12. */
size = dprintf(fd, "%10d\n", getpid());
if (size != 11) {
unlink(lockfile);
close(fd);
return -1;