From: Gunnar Beutner <gunnar.beutner@netways.de>
Date: Thu, 26 Jun 2014 12:12:20 +0000 (+0200)
Subject: Fix build warnings
X-Git-Tag: v2.0.1~30
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3ece0cff4d93d56c28cad5c578214f4611f5a792;p=icinga2

Fix build warnings

fixes #6488
---

diff --git a/lib/base/process.cpp b/lib/base/process.cpp
index e978803d9..40fbfba5b 100644
--- a/lib/base/process.cpp
+++ b/lib/base/process.cpp
@@ -228,7 +228,8 @@ void Process::IOThreadProc(int tid)
 #else /* _WIN32 */
 			if (pfds[0].revents & (POLLIN | POLLHUP | POLLERR)) {
 				char buffer[512];
-				(void)read(l_EventFDs[tid][0], buffer, sizeof(buffer));
+				if (read(l_EventFDs[tid][0], buffer, sizeof(buffer)) < 0)
+					Log(LogCritical, "base", "Read from event FD failed.");
 			}
 #endif /* _WIN32 */
 
@@ -512,7 +513,8 @@ void Process::Run(const boost::function<void(const ProcessResult&)>& callback)
 		(void)close(fds[0]);
 		(void)close(fds[1]);
 
-		(void)nice(5);
+		if (nice(5) < 0)
+			Log(LogWarning, "base", "Failed to renice child process.");
 
 		if (icinga2_execvpe(argv[0], argv, envp) < 0) {
 			char errmsg[512];
@@ -570,7 +572,8 @@ void Process::Run(const boost::function<void(const ProcessResult&)>& callback)
 #ifdef _WIN32
 	SetEvent(l_Events[tid]);
 #else /* _WIN32 */
-	(void)write(l_EventFDs[tid][1], "T", 1);
+	if (write(l_EventFDs[tid][1], "T", 1) < 0 && errno != EINTR && errno != EAGAIN)
+		Log(LogCritical, "base", "Write to event FD failed.");
 #endif /* _WIN32 */
 }