From c5015c435b79f07397f6770d225457e6c1a04cb2 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Fri, 4 Jul 2025 13:49:12 +0200 Subject: [PATCH] nss: Promote debug message to proper error when time() fails MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 Reviewed-by: Ján Tomko --- tools/nss/libvirt_nss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; } -- 2.47.2