On Sun, Mar 29, 2015 at 03:45:50PM +0200, Silvan Jegen wrote: > Heyho! > > On Wed, Mar 25, 2015 at 07:15:08PM +0000, non...@inventati.org wrote: > > There are two issues with buffer_alloc function(s). > > > > First function is located in buffer.c: > > http://repo.or.cz/w/vis.git/blob/HEAD:/buffer.c > > > > The problem is on line 15: > > http://repo.or.cz/w/vis.git/blob/HEAD:/buffer.c#l15 > > > > If realloc fails, buf->data is set to NULL, but original buf->data is > > not freed, see realloc(3). The result of realloc should be stored in > > separate variable and buffer_free should be called to free buf->data and > > set size and len to 0 if realloc fails. > > This is true. In a memory-overcommitting system like Linux, however, > this should not be a problem in practice since realloc should never fail.
Then NULL check is a dead code and should be removed.
