[pacman-dev] [PATCH] libalpm: set ret in download files

morganamilo morganamilo at archlinux.org
Mon Nov 30 18:21:26 UTC 2020


download_files never set ret on failiure, so even when downloading
fails, the transaction goes on to commit and error out.

:: Retrieving packages...
 python-packaging-20.4-4-any.pkg.tar.zst failed to download
error: failed retrieving file 'python-packaging-20.4-4-any.pkg.tar.zst' from mirror.oldsql.cc : The requested URL returned error: 404
warning: failed to retrieve some files
(1/1) checking keys in keyring
(1/1) checking package integrity
error: failed to commit transaction (wrong or NULL argument passed)
Errors occurred, no packages were upgraded.

Also make the ret checking more consistent.

diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c
index 601f1d69..5d8652a5 100644
--- a/lib/libalpm/sync.c
+++ b/lib/libalpm/sync.c
@@ -769,7 +769,7 @@ static int download_files(alpm_handle_t *handle)
 	}
 
 	ret = find_dl_candidates(handle, &files);
-	if(ret) {
+	if(ret != 0) {
 		goto finish;
 	}
 
@@ -818,7 +818,9 @@ static int download_files(alpm_handle_t *handle)
 
 			payloads = alpm_list_add(payloads, payload);
 		}
-		if(_alpm_download(handle, payloads, cachedir) == -1) {
+
+		ret = _alpm_download(handle, payloads, cachedir);
+		if(ret != 0) {
 			event.type = ALPM_EVENT_PKG_RETRIEVE_FAILED;
 			EVENT(handle, &event);
 			_alpm_log(handle, ALPM_LOG_WARNING, _("failed to retrieve some files\n"));
-- 
2.29.2


More information about the pacman-dev mailing list