On Tue, Jun 28, 2016 at 11:48:58 +1000, Peter Hutterer wrote:

> Regression introduce by ac164e58870d which calls
> XkbUpdateAllDeviceIndicators() with two NULL arguments. A few layers down into
> the stack and we triggered a NULL-pointer dereference. In theory a NULL cause
> is acceptable since we don't actually change modifier state here. Instead of
> updating all places to check for NULL just set the cause to the client
> request and go to the pub.
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=96384
> 
> Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
> ---
>  xkb/xkb.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
Going to the pub seems like a great plan.

Reviewed-by: Julien Cristau <jcris...@debian.org>

Cheers,
Julien
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to