From fb21b2f0b71a6bcbb6422ca66d9877582ea35df7 Mon Sep 17 00:00:00 2001 From: Christopher Friedt Date: Fri, 29 May 2020 09:54:13 -0400 Subject: [PATCH] samples: sockets: socketpair: check return value of send Fixes #25736 Coverity-CID: 210583 Signed-off-by: Christopher Friedt --- samples/net/sockets/socketpair/src/socketpair_example.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/samples/net/sockets/socketpair/src/socketpair_example.c b/samples/net/sockets/socketpair/src/socketpair_example.c index d7a8a48af49..54fdb1323e3 100644 --- a/samples/net/sockets/socketpair/src/socketpair_example.c +++ b/samples/net/sockets/socketpair/src/socketpair_example.c @@ -47,7 +47,12 @@ static const char *const names[] = { static void hello(int fd, const char *name) { /* write(2) should be used after #25443 */ - send(fd, name, strlen(name), 0); + int res = send(fd, name, strlen(name), 0); + + if (res != strlen(name)) { + printf("%s(): send: expected: %d actual: %d errno: %d\n", + __func__, (int)strlen(name), res, errno); + } } static void *fun(void *arg) @@ -153,7 +158,7 @@ int main(int argc, char *argv[]) if ((fds[i].revents & POLLHUP) != 0) { printf("fd: %d: hung up\n", fd); close(ctx[idx].spair[0]); - printf("main: closed fd %d\n", + printf("%s: closed fd %d\n", __func__, ctx[idx].spair[0]); pthread_join(ctx[idx].thread, &unused); printf("joined %s\n", ctx[idx].name);