[PATCH 01/32] [libx11] Using freed pointer "e"

Matt Turner mattst88 at gmail.com
Sat Feb 12 16:22:44 PST 2011


On Sat, Feb 12, 2011 at 11:00 PM, Mark Kettenis <mark.kettenis at xs4all.nl> wrote:
>> From: =?UTF-8?q?Erkki=20Sepp=C3=A4l=C3=A4?= <erkki.seppala at vincit.fi>
>> Date: Thu, 27 Jan 2011 09:53:44 +0200
>>
>> Reordered code to first to do the comparison and then to release data
>
> Your static analysis tool isn't all that good.  Using the value of a
> pointer after freeing the storage it points to is totally fine.

While you're right, this is a bit strange. Other patches doing this
have already been accepted, so I don't see any problem.

Matt


More information about the xorg-devel mailing list