From 75b6ede30c110706d514a85d61c9d239984a0032 Mon Sep 17 00:00:00 2001 From: Tom Rini Date: Wed, 20 Dec 2017 10:01:48 -0500 Subject: utest-cam-imr-drm: Fix build failure with -Werror utest/utest-common.c::intern_trace() was not checking the return value of write(2). Add check and propagate error return to the caller. Change-Id: I2801b4468c129fd0eb865ea39499d3081336f161 Signed-off-by: Tom Rini --- ...test-common.c-Check-return-value-of-write.patch | 44 ++++++++++++++++++++++ .../recipes-bsp/utest-apps/utest-cam-imr-drm.bb | 1 + 2 files changed, 45 insertions(+) create mode 100644 meta-rcar-gen3-adas/recipes-bsp/utest-apps/files/0001-utest-utest-common.c-Check-return-value-of-write.patch diff --git a/meta-rcar-gen3-adas/recipes-bsp/utest-apps/files/0001-utest-utest-common.c-Check-return-value-of-write.patch b/meta-rcar-gen3-adas/recipes-bsp/utest-apps/files/0001-utest-utest-common.c-Check-return-value-of-write.patch new file mode 100644 index 0000000..80cf032 --- /dev/null +++ b/meta-rcar-gen3-adas/recipes-bsp/utest-apps/files/0001-utest-utest-common.c-Check-return-value-of-write.patch @@ -0,0 +1,44 @@ +From 1fd2e6ce6a443d20e6a103261860816fa5600591 Mon Sep 17 00:00:00 2001 +From: Tom Rini +Date: Wed, 20 Dec 2017 09:58:47 -0500 +Subject: [PATCH 1/1] utest/utest-common.c: Check return value of write() + +Signed-off-by: Tom Rini +--- + utest/utest-common.c | 9 ++++++--- + 1 file changed, 6 insertions(+), 3 deletions(-) + +diff --git a/utest/utest-common.c b/utest/utest-common.c +index 38bd4714bccc..922b4bdc40a9 100644 +--- a/utest/utest-common.c ++++ b/utest/utest-common.c +@@ -66,7 +66,7 @@ int intern_trace(const char *format, ...) + struct timespec ts; + static char buffer[4096]; + char *p = buffer; +- int n = sizeof(buffer), k; ++ int n = sizeof(buffer), k, ret; + + /* ...retrieve value of monotonic clock */ + clock_gettime(CLOCK_MONOTONIC, &ts); +@@ -88,12 +88,15 @@ int intern_trace(const char *format, ...) + /* ...output string terminator */ + (n > 0 ? *p++ = '\n' : 0); + +- write(intern_trace_fd, buffer, p - buffer); ++ ret = write(intern_trace_fd, buffer, p - buffer); + + /* ...release tracing lock */ + pthread_mutex_unlock(&intern_trace_mutex); + +- return 0; ++ if (ret != -1) ++ return 0; ++ else ++ return -1; + } + + /* ...tracing facility initialization */ +-- +2.7.4 + diff --git a/meta-rcar-gen3-adas/recipes-bsp/utest-apps/utest-cam-imr-drm.bb b/meta-rcar-gen3-adas/recipes-bsp/utest-apps/utest-cam-imr-drm.bb index 48747ea..4c6be5b 100644 --- a/meta-rcar-gen3-adas/recipes-bsp/utest-apps/utest-cam-imr-drm.bb +++ b/meta-rcar-gen3-adas/recipes-bsp/utest-apps/utest-cam-imr-drm.bb @@ -7,6 +7,7 @@ S = "${WORKDIR}/utest-cam-imr-drm" SRC_URI = " \ file://utest-cam-imr-drm.tar.gz \ + file://0001-utest-utest-common.c-Check-return-value-of-write.patch \ " DEPENDS = " \ -- cgit 1.2.3-korg