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@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", .