]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
nss: Promote debug message to proper error when time() fails
authorMichal Privoznik <mprivozn@redhat.com>
Fri, 4 Jul 2025 11:49:12 +0000 (13:49 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 18 Jul 2025 13:05:40 +0000 (15:05 +0200)
The difference between DEBUG() and ERROR() macros is that the
latter also prints (stringified) errno. Inside of findLease()
there's one line where time() is called which is documented as:

  On error, ((time_t) -1) is returned, and errno is set to
  indicate the error.

Therefore, don't just print DEBUG() but ERROR() so that the cause
for the error is printed too.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
tools/nss/libvirt_nss.c

index 6ee328a8df005c3a9b32b08a4eff11a874a4b798..52b769d760377b87b43efd3b3c3c797fc2262d63 100644 (file)
@@ -174,7 +174,7 @@ findLease(const char *name,
 #endif
 
     if ((now = time(NULL)) == (time_t)-1) {
-        DEBUG("Failed to get time");
+        ERROR("Failed to get time");
         goto cleanup;
     }