git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/3] daemon: handle freopen() failure

From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
Date
Dec 26, 2008, 09:46 UTC
Message-ID
<1230284785.6728.26.camel@ubuntu.ubuntu-domain>

Die if stderr couldn't be sent to /dev/null when operating in inetd mode and report the error message from the OS.

This fixes a compiler warning about the return value of freopen() being ignored on Ubuntu 8.10.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
---
 daemon.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/daemon.c b/daemon.c
index 1cef309..8c317be 100644
--- a/daemon.c
+++ b/daemon.c
@@ -1118,7 +1118,9 @@ int main(int argc, char **argv)
 		struct sockaddr *peer = (struct sockaddr *)&ss;
 		socklen_t slen = sizeof(ss);
 
-		freopen("/dev/null", "w", stderr);
+		if (!freopen("/dev/null", "w", stderr))
+			die("failed to redirect stderr to /dev/null: %s",
+			    strerror(errno));
 
 		if (getpeername(0, peer, &slen))
 			peer = NULL;
-- 
1.6.1
Next: René Scharfe
Message 1 of 4 in “daemon: handle freopen() failure”
  1. 1/3 daemon: handle freopen() failureRené Scharfe, Dec 26, 2008
  2. 2/3 daemon: cleanup: replace loop with ifRené Scharfe, Dec 26, 2008
  3. 3/3 daemon: cleanup: factor out xstrdup_tolower()René Scharfe, Dec 26, 2008
  4. merge-file: handle freopen() failureRené Scharfe, Dec 26, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.