On 27 June 2014 07:15, Keith Packard <[email protected]> wrote: > Julien Cristau <[email protected]> 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. _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
