On Sun, 2017-01-22 at 15:18 +0100, Tobias Stoeckmann wrote:

> @@ -286,6 +286,7 @@ XineramaQueryScreens(
>      if (!_XReply (dpy, (xReply *) &rep, 0, xFalse)) {
>       UnlockDisplay (dpy);
>       SyncHandle ();
> +     *number = 0;

Not that any caller has likely made this mistake, but you want an if
(number) before this, otherwise you turn a protocol error into a
segfault.

- ajax
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to