<div>Hi.</div><div> </div><div>I added error handling code in write_rotate().</div><div>I think that it is better to check tf->logfd before doing truncate()/write() and getting error.</div><div> </div><div> </div><div> </div><div> </div><div> </div><div> </div><div>From 5e24e65d3abfd78e5405b81690d6ada4260cca4e Mon Sep 17 00:00:00 2001<br>From: Hyejin Kim <<a href="mailto:hj8296@gmail.com">hj8296@gmail.com</a>><br>Date: Mon, 2 Mar 2015 14:27:00 +0900<br>Subject: [PATCH] syslogd : check return value when opening file</div><div>---<br> toys/pending/syslogd.c | 4 ++++<br> 1 file changed, 4 insertions(+)</div><div>diff --git a/toys/pending/syslogd.c b/toys/pending/syslogd.c<br>index 7fb297f..41d655d 100644<br>--- a/toys/pending/syslogd.c<br>+++ b/toys/pending/syslogd.c<br>@@ -300,6 +300,10 @@ static int write_rotate(struct logfile *tf, int len)<br>         unlink(tf->filename);<br>         close(tf->logfd);<br>         tf->logfd = open(tf->filename, O_CREAT | O_WRONLY | O_APPEND, 0666);<br>+  if (tf->logfd < 0) {<br>+          perror_msg("can't open %s", tf->filename);<br>+          return -1;<br>+        }<br>       }<br>       ftruncate(tf->logfd, 0);<br>     }<br>-- <br>1.9.1</div>