[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