From: Michal Privoznik <mprivozn@redhat.com>
Date: Fri, 4 Jul 2025 11:49:12 +0000 (+0200)
Subject: nss: Promote debug message to proper error when time() fails
X-Git-Tag: v11.6.0-rc1~81
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c5015c435b79f07397f6770d225457e6c1a04cb2;p=thirdparty%2Flibvirt.git

nss: Promote debug message to proper error when time() fails

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>
---

diff --git a/tools/nss/libvirt_nss.c b/tools/nss/libvirt_nss.c
index 6ee328a8df..52b769d760 100644
--- a/tools/nss/libvirt_nss.c
+++ b/tools/nss/libvirt_nss.c
@@ -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;
     }