[PATCH 1/2] shm: Fix use-after-free in ShmDestroyPixmap
Michel Dänzer
michel at daenzer.net
Sun Feb 15 18:28:57 PST 2015
On 15.02.2015 06:30, Chris Wilson wrote:
> On Sat, Feb 14, 2015 at 08:33:18PM +0900, Michel Dänzer wrote:
>> On 14.02.2015 19:36, Chris Wilson wrote:
>>> /*ARGSUSED*/ static int
>>> ShmDetachSegment(void *value, /* must conform to DeleteType */
>>> - XID shmseg)
>>> + XID unused)
>>> {
>>> ShmDescPtr shmdesc = (ShmDescPtr) value;
>>> ShmDescPtr *prev;
>>>
>>
>> It's a static function, so you can just remove the unused parameter.
>
> We want to keep it around so that it matches DeleteType as it is passed
> to CreateNewResourceType as the cleanup handler for the resource. The
> comment is telling the truth for once!
Gotcha, sorry for the noise.
Reviewed-by: Michel Dänzer <michel at daenzer.net>
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
More information about the xorg-devel
mailing list