From 2cd2d2c47593941f002ed45f44f0e0c1072738d7 Mon Sep 17 00:00:00 2001
From: Joseph Hunkeler <jhunkeler@gmail.com>
Date: Thu, 12 Feb 2026 09:53:21 -0500
Subject: errmsg argument must be a valid pointer

* Remove redundant status check already performed by HTTP_ERROR()
---
 src/lib/core/artifactory.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/lib/core/artifactory.c b/src/lib/core/artifactory.c
index eedaf43..d5457e7 100644
--- a/src/lib/core/artifactory.c
+++ b/src/lib/core/artifactory.c
@@ -61,9 +61,10 @@ int artifactory_download_cli(char *dest,
     }
 
     sprintf(path + strlen(path), "/%s", remote_filename);
-    long fetch_status = download(url, path, NULL);
-    if (HTTP_ERROR(fetch_status) || fetch_status < 0) {
-        fprintf(stderr, "%s: download failed: %s\n", __FUNCTION__, url);
+    char *errmsg = NULL;
+    long fetch_status = download(url, path, &errmsg);
+    if (HTTP_ERROR(fetch_status)) {
+        SYSERROR("download failed: %s: %s\n", errmsg, url);
         return -1;
     }
     chmod(path, 0755);
-- 
cgit