Re: [PATCH] xserver: Don't crash if Xv is not initialized

2011-02-14 Thread Dave Airlie
On Tue, Feb 8, 2011 at 8:39 PM, Mauro Carvalho Chehab mche...@infradead.org wrote: The xf86-video-v4l video driver calls xf86XVQueryOffscreenImages() function in order to probe for the Xv FOURCC formats supported for PutVideo ops. However, as this support is deprecated on most of the modern

[PATCH] xserver: Don't crash if Xv is not initialized

2011-02-14 Thread Mauro Carvalho Chehab
The xf86-video-v4l video driver calls xf86XVQueryOffscreenImages() function in order to probe for the Xv FOURCC formats supported for PutVideo ops. However, as this support is deprecated on most of the modern drivers, a call to this method will cause a crash: X: ../../../include/privates.h:115:

Re: [PATCH] xserver: Don't crash if Xv is not initialized

2011-02-14 Thread Keith Packard
+if (!OffscreenPrivateKey-initialized) { + *num = 0; + return NULL; +} Please use dixPrivateKeyRegistered instead of looking inside the structure. -- keith.pack...@intel.com pgpVm8QakooVK.pgp Description: PGP signature ___

[PATCH] xserver: Don't crash if Xv is not initialized

2011-02-08 Thread Mauro Carvalho Chehab
The xf86-video-v4l video driver calls xf86XVQueryOffscreenImages() function in order to probe for the Xv FOURCC formats supported for PutVideo ops. However, as this support is deprecated on most of the modern drivers, a call to this method will cause a crash: X: ../../../include/privates.h:115: