[PATCH weston] xwayland: Fix X11 lock file size confusion

Daniel Stone daniel at fooishbar.org
Thu Nov 17 11:51:37 UTC 2016


On 17 November 2016 at 11:35, Daniel Stone <daniels at collabora.com> wrote:
> 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.
>
> Related Mutter issue: https://bugzilla.gnome.org/show_bug.cgi?id=774613

For those of you playing along at home, this meant that before Pekka's
patch, Weston would fail to parse its own lock files and Xorg's, as
the trailing \n made safe_strtol explode. It would, however, parse
Mutter's lock files, because Mutter was off by one in snprintf and
would thus print "%10d\0" to the file instead of the "%10d\n". Great
stuff.

Cheers,
Daniel


More information about the wayland-devel mailing list