[pacman-dev] [PATCH] libalpm, fix leak if malloc fails in be_sync

Andrew Gregory andrew.gregory.8 at gmail.com
Mon Jan 4 09:55:18 UTC 2016


On 01/04/16 at 02:54pm, Allan McRae wrote:
> On 11/11/15 05:52, Rikard Falkeborn wrote:
> > ---
> >  lib/libalpm/be_sync.c | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> 
> This needs reworked due to other changes made to that file.  But I seem
> to recall Andrew had a comment on this...
> 
> Andrew - did I recall correctly?

I think my only real issue with the patch itself was that it moved
_alpm_handle_unlock when it shouldn't have.  At this point this patch
should just remove RET_ERR, manually set pm_errno, and break or
continue the loop.

apg


More information about the pacman-dev mailing list