[PATCH libXrandr] Avoid out of boundary accesses on illegal responses
Julien Cristau
jcristau at debian.org
Sat Jan 28 14:47:13 UTC 2017
On Sat, Jan 7, 2017 at 19:15:42 +0100, Tobias Stoeckmann wrote:
> Hi Julien,
>
> On Sat, Jan 07, 2017 at 07:03:17PM +0100, Julien Cristau wrote:
> > It looks like we're leaking 'attr' on these error paths?
>
> confirmed. That is what I get for copying the error handling of the
> attr == NULL case...
>
Pushed as
https://cgit.freedesktop.org/xorg/lib/libXrandr/commit/?id=87227e5fc79750d3eccc3c3482a3c5b3f2af2e90
I kind of wonder if those error paths shouldn't just IOError, if we're
talking to a broken and/or malicious X server what's the point trying to
recover...
Cheers,
Julien
More information about the xorg-devel
mailing list