[pacman-dev] [PATCH 1/2] Attempted to free lock on failure in alpm_db_update()
Allan McRae
allan at archlinux.org
Thu Dec 19 01:17:38 UTC 2019
On 17/12/19 5:13 pm, Andrew Gregory wrote:
> s/Attempted to//?
>
> On 12/17/19 at 01:04am, Allan McRae wrote:
>> Also fixes a memory leak under an error condition.
>>
>> Signed-off-by: Allan McRae <allan at archlinux.org>
>> ---
>> lib/libalpm/be_sync.c | 18 ++++++++++--------
>> 1 file changed, 10 insertions(+), 8 deletions(-)
>>
>> diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c
>> index 041b2266..5502d92d 100644
>> --- a/lib/libalpm/be_sync.c
>> +++ b/lib/libalpm/be_sync.c
> ...
>> @@ -324,6 +325,7 @@ int SYMEXPORT alpm_db_update(int force, alpm_db_t *db)
>> }
>> }
>>
>> +cleanup:
>
> Shouldn't this be before the previous if(updated){...} block?
It could be... No need to drop the cache and reset validation if we
are restoring the old database, but it does not hurt.
>> if(ret == -1) {
>> /* pm_errno was set by the download code */
>> _alpm_log(handle, ALPM_LOG_DEBUG, "failed to sync db: %s\n",
> .
>
More information about the pacman-dev
mailing list