From 8c1f857ea6d08f2d0d0084a3fdac87ea45de49dc Mon Sep 17 00:00:00 2001 From: YoungHun Kim Date: Wed, 16 Jan 2019 10:25:37 +0900 Subject: [PATCH] Fix the svace issue Change-Id: Id03c37441b6759414c91767706ec3ccceeb7b002 --- packaging/mused.spec | 2 +- server/src/muse_server.c | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/packaging/mused.spec b/packaging/mused.spec index a83a3b2..1cb00fc 100644 --- a/packaging/mused.spec +++ b/packaging/mused.spec @@ -1,6 +1,6 @@ Name: mused Summary: A multimedia daemon -Version: 0.3.61 +Version: 0.3.62 Release: 0 Group: System/Libraries License: Apache-2.0 and BSD-3-Clause diff --git a/server/src/muse_server.c b/server/src/muse_server.c index d18d8ad..d51529c 100644 --- a/server/src/muse_server.c +++ b/server/src/muse_server.c @@ -41,7 +41,7 @@ static void _ms_setup_syslog(void) static void _ms_fork(int *notify_fd) { pid_t pid; - int result, fds[2]; + int fds[2]; char err_msg[MUSE_MSG_LEN_MAX] = {'\0',}; char msg[MUSE_MSG_LEN_MAX] = {'\0',}; @@ -58,7 +58,11 @@ static void _ms_fork(int *notify_fd) } else if (pid != 0) { close(fds[1]); /* Read in a string from the pipe */ - result = read(fds[0], msg, sizeof(msg)); + if (read(fds[0], msg, sizeof(msg)) <= 0) { + LOGE("Failed to read from a file descriptor [%d]", fds[0]); + close(fds[0]); + return; + } close(fds[0]); /* Parent process closes up output side of pipe */ -- 2.7.4