[PATCH 3/7] Use FreePixmap to free the fbPixmap in case the driver forgot

Alan Coopersmith alan.coopersmith at oracle.com
Wed May 12 15:53:34 PDT 2010


Keith Packard wrote:
>      free(pScreen->devPrivate);
> +    if (pScreen->devPrivate) {
> +	FreePixmap(pScreen->devPrivate);
> +	pScreen->devPrivate = NULL;
> +    }

Shouldn't you delete the initial free then, so you're not referencing
the pointer you just free'd ?

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list