[PATCH] dix: die if we can't activate or init the VCP/VCK.

Dan Nicholson dbn.lists at gmail.com
Tue Feb 3 16:46:05 PST 2009


On Tue, Feb 3, 2009 at 4:30 PM, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> If we have a busted xkb setup, the XKB initialization on the core devices
> fails and leaves us with dev->key->xkbInfo == NULL. This in turn causes
> segfaults lateron.
>
> Return BadValue when the XKB configuration for a master device failed, and if
> that happens for the VCP/VCK, die semi-gracefully.

Seems reasonable to me. I'd much rather die early than scratch my head
on some later crash. Would it also be a good idea to return BadValue
from CorePointerProc when InitPointerDeviceStruct fails? I don't know
what you'd put in the log in that situation, though.

--
Dan



More information about the xorg mailing list