Commit b9aa3328 authored by antirez's avatar antirez
Browse files

Check write(2) return value to avoid warnings, because in this context failing...

Check write(2) return value to avoid warnings, because in this context failing write is not critical.
parent 3f64694e
...@@ -316,14 +316,15 @@ void redisLogFromHandler(int level, const char *msg) { ...@@ -316,14 +316,15 @@ void redisLogFromHandler(int level, const char *msg) {
STDOUT_FILENO; STDOUT_FILENO;
if (fd == -1) return; if (fd == -1) return;
ll2string(buf,sizeof(buf),getpid()); ll2string(buf,sizeof(buf),getpid());
write(fd,"[",1); if (write(fd,"[",1) == -1) goto err;
write(fd,buf,strlen(buf)); if (write(fd,buf,strlen(buf)) == -1) goto err;
write(fd," | signal handler] (",20); if (write(fd," | signal handler] (",20) == -1) goto err;
ll2string(buf,sizeof(buf),time(NULL)); ll2string(buf,sizeof(buf),time(NULL));
write(fd,buf,strlen(buf)); if (write(fd,buf,strlen(buf)) == -1) goto err;
write(fd,") ",2); if (write(fd,") ",2) == -1) goto err;
write(fd,msg,strlen(msg)); if (write(fd,msg,strlen(msg)) == -1) goto err;
write(fd,"\n",1); if (write(fd,"\n",1) == -1) goto err;
err:
if (server.logfile) close(fd); if (server.logfile) close(fd);
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment