From f64f2b78abd6b0b0dd556bd3535f8701d6a9898e Mon Sep 17 00:00:00 2001 From: folkert van heusden Date: Sun, 28 Apr 2024 01:22:38 +0200 Subject: [PATCH] coverity: in theory, socket() can return -1 --- log.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/log.cpp b/log.cpp index d250081..62ae93f 100644 --- a/log.cpp +++ b/log.cpp @@ -83,8 +83,10 @@ void send_syslog(const int ll, const std::string & what) std::string msg = format("<%d>%s", 16 * 8 + ll, what.c_str()); int s = socket(AF_INET, SOCK_DGRAM, 0); - (void)sendto(s, msg.c_str(), msg.size(), 0, reinterpret_cast(&syslog_ip_addr), sizeof syslog_ip_addr); - close(s); + if (s != -1) { + (void)sendto(s, msg.c_str(), msg.size(), 0, reinterpret_cast(&syslog_ip_addr), sizeof syslog_ip_addr); + close(s); + } } void closelog()