[PATCH 1/2] libX11: Fix potential memory leaks during font set loading.

Kim Woelders kim at woelders.dk
Mon Nov 23 09:43:19 PST 2009


On Mon, 23 Nov 2009 03:03:01 +0100, Alan Coopersmith  
<Alan.Coopersmith at sun.com> wrote:

> Kim Woelders wrote:
>> -    char *pattern = NULL, *ptr = NULL;
>> +    char *pattern, *ptr;
>
> Why remove the initialization to NULL?
>
They are useless assignments. There are other useless initializations  
(pixel_size, field_num, len) but they are not related to the fix, which is  
about properly freeing the pattern buffer, so it didn't feel right to  
change those.

Anyway, removing the initializations or not makes no difference in the end.

/Kim



More information about the xorg-devel mailing list