From e63f88146d353091488322a1fb6a2b9b10887bef Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Sat, 26 Nov 2016 15:49:21 +0100 Subject: stream-client: check read() return value before using it as length input to memcpy Change-Id: Id962821c71b3a1c4c01c1131eb809b8ec8eaa062 Fixes: Coverity CID 57859 --- examples/stream-client.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/examples/stream-client.c b/examples/stream-client.c index c54db02..b0aea5e 100644 --- a/examples/stream-client.c +++ b/examples/stream-client.c @@ -72,8 +72,9 @@ static int kbd_cb(struct osmo_fd *fd, unsigned int what) int ret; ret = read(STDIN_FILENO, buf, sizeof(buf)); - LOGP(DSTREAMTEST, LOGL_NOTICE, "read %d byte from keyboard\n", ret); + if (ret < 0) + return ret; msg = msgb_alloc(1024, "STREAMCLIENT/test"); if (msg == NULL) { -- cgit v1.2.3