From 92efd03e87a97dd160446afa794bf94e0b9ad067 Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Sat, 23 Jan 2016 10:37:28 +0100 Subject: Do not use strlen on binary inpit Coverity points out that buf might not be NULL terminated. For the memcpy ret was already used, so use ret for msgb_put as well. Fixes: CID#57922 --- examples/stream-client.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'examples') diff --git a/examples/stream-client.c b/examples/stream-client.c index a1f57b0..112d78b 100644 --- a/examples/stream-client.c +++ b/examples/stream-client.c @@ -80,7 +80,7 @@ static int kbd_cb(struct osmo_fd *fd, unsigned int what) LOGP(DSTREAMTEST, LOGL_ERROR, "cannot allocate message\n"); return 0; } - ptr = msgb_put(msg, strlen(buf)); + ptr = msgb_put(msg, ret); memcpy(ptr, buf, ret); osmo_stream_cli_send(conn, msg); -- cgit v1.2.3