From e9df14dcef5bde6e77760243783c07b4533ee7dc Mon Sep 17 00:00:00 2001 From: Geoff Levand Date: Thu, 3 Oct 2013 20:19:34 +0000 Subject: kexec: Fix return value build warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add a local variable 'result' to the putnode() routine of ds2dt and use it to hold return values of calls to read(). Fixes build warnings like these: kexec/fs2dt.c: warning: ignoring return value of ‘read’ Signed-off-by: Geoff Levand for Huawei, Linaro Signed-off-by: Simon Horman --- kexec/fs2dt.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/kexec/fs2dt.c b/kexec/fs2dt.c index 1f5b0cf..98d6cb4 100644 --- a/kexec/fs2dt.c +++ b/kexec/fs2dt.c @@ -555,6 +555,7 @@ static void putnode(void) /* Add cmdline to the second kernel. Check to see if the new * cmdline has a root=. If not, use the old root= cmdline. */ if (!strcmp(basename,"chosen/")) { + size_t result; size_t cmd_len = 0; char *param = NULL; char filename[MAXPATH]; @@ -636,7 +637,7 @@ static void putnode(void) close(fd); goto no_debug; } - read(fd, buff, statbuf.st_size); + result = read(fd, buff, statbuf.st_size); close(fd); strncpy(filename, "/proc/device-tree/", MAXPATH); strncat(filename, buff, MAXPATH); @@ -659,7 +660,7 @@ static void putnode(void) close(fd); goto no_debug; } - read(fd, buff, statbuf.st_size); + result = read(fd, buff, statbuf.st_size); if (!strcmp(buff, "hvterm1") || !strcmp(buff, "hvterm-protocol")) my_debug = 1; close(fd); -- cgit