[PATCH 1/4] fb: fix screen pixmap leak on server reset

Dave Airlie airlied at gmail.com
Thu Jun 26 14:20:13 PDT 2014


On 27 June 2014 07:15, Keith Packard <keithp at keithp.com> wrote:
> Julien Cristau <jcristau at debian.org> writes:
>
>> This breaks Xvfb:
>> https://bugs.freedesktop.org/show_bug.cgi?id=80313
>>
>> pScreen->devPrivate is NULL, so we crash trying to dereference it in
>> FreePixmap(), whereas the free() was a no-op.
>
> How about we have fb check pScreen->devPrivate != NULL before calling
> FreePixmap?

is pScreen->devPrivate guaranteed to be a pixmap, or is that just a convention?

Do we have anything to say when it is?

I think I'm also confusing this with the screwed up miScreenInit
private ptr hiding stuff.

Dave.


More information about the xorg-devel mailing list