[pacman-dev] [PATCH] Add check for NULL data parameter in sync commit

Allan McRae mcrae_allan at hotmail.com
Mon Dec 24 07:56:02 EST 2007


Nagy Gabor wrote:
> Some little comments: If you don't keep the result, you should free
> deps list with FREELIST().
>
> And the same for the allocated errormsg strings <- here you can put the
> whole {calloc, snprintf, alpm_list_add} block to if(data).
>
> Bye
>   
Updated patch attached that fixes the above issues.

Allan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Allow-NULL-parameter-in-alpm_trans_commit.patch
Type: text/x-patch
Size: 3367 bytes
Desc: not available
URL: <http://archlinux.org/pipermail/pacman-dev/attachments/20071224/99bc1bf7/attachment.bin>


More information about the pacman-dev mailing list