Re: [hackers] [sent] Bail out before allocating slides if file is empty

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Wed, 18 Nov 2015 23:29:16 +0100

Quentin Rameau wrote:
> In load() we allocated slides before checking if we actually read anything
> from the FILE fp and then continue with an allocated but “empty” space wich
> would lead to errors.

Heyho,

of course that is another fix for the problem related to memory sanitazion. I
merged it, thanks. I still leave my fix (checking for slidecount instead of
checking pointers), since it is more verbose than the old version: If no slides
were read, die with usage message.

--Markus
Received on Wed Nov 18 2015 - 23:29:16 CET