Harald Welte has submitted this change and it was merged. Change subject: SocketsTest: Fix printing of non-nul-terminated string ......................................................................
SocketsTest: Fix printing of non-nul-terminated string Change-Id: I33d0ddf851d84b81ab5252e3755422170cee54ee Fixes: Coverity CID#149363 --- M CommonLibs/SocketsTest.cpp 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/CommonLibs/SocketsTest.cpp b/CommonLibs/SocketsTest.cpp index c2849e0..3198a5e 100644 --- a/CommonLibs/SocketsTest.cpp +++ b/CommonLibs/SocketsTest.cpp @@ -61,7 +61,8 @@ readSocket.nonblocking(); int rc = 0; while (rc<gNumToSend) { - char buf[MAX_UDP_LENGTH]; + char buf[MAX_UDP_LENGTH+1]; + buf[MAX_UDP_LENGTH] = '\0'; int count = readSocket.read(buf, MAX_UDP_LENGTH); if (count>0) { COUT("read: " << buf); -- To view, visit https://gerrit.osmocom.org/4696 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33d0ddf851d84b81ab5252e3755422170cee54ee Gerrit-PatchSet: 2 Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Owner: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder