[giflib-devel] [PATCH] lib/dgif_lib.c: Initialize Private->RunningBits

[Simon T. <fo...@c-...>]

Hello,

When opening a gif file using giflib the RunningBits member of
GifFilePrivateType is never initialized before access in
DGifSetupDecompress (called from DGifGetImageDesc) causing the function
to occasionally set GifFile->Error = D_GIF_ERR_READ_FAILED and return
GIF_ERROR even though no actual error occurred.

I've attached a patch that initializes RunningBits to 0 in
DGifOpenFileHandle, but it might be safer to memset Private to 0 after
allocating with malloc as there might other variables that are accessed
without initialization that I haven't hit.

Since I am not currently subscribed to the mailing list, please CC me in
replies to this mail (though I will try and check the mailing list
archives periodically).

-- 
Simon Thelen