On Thu, Jun 26, 2014 at 14:15:41 -0700, Keith Packard wrote:

> From: Keith Packard <[email protected]>
> Date: Thu, 26 Jun 2014 14:12:24 -0700
> Subject: [PATCH] fb: Don't free NULL pixmap in fbCloseScreen. Bug #80313
> 
> We fixed fbCloseScreen to use the FreePixmap function so that the
> private counts would be updated correctly during CloseScreen. Xvfb
> calls FreePixmap and sets devPrivate to NULL before fbCloseScreen is
> called; not checking devPrivate before calling would result in a NULL
> pointer dereference.
> 
> Signed-off-by: Keith Packard <[email protected]>
> ---
>  fb/fbscreen.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
Reviewed-by: Julien Cristau <[email protected]>

Cheers,
Julien
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to