From 12dba4e70f831d6d27a7f769cc9dab19c19a155d Mon Sep 17 00:00:00 2001 From: Murai Takashi Date: Tue, 25 Dec 2018 06:36:49 +0900 Subject: [PATCH] poorman: Fix PVS 922 Remove unneed appending a null character, since inet_ntop() adds null to 'addr'. Change-Id: I21f5be7f737badd388f36a6dcf56a3b446b5d91c Reviewed-on: https://review.haiku-os.org/795 Reviewed-by: Barrett17 --- src/apps/poorman/PoorManWindow.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/apps/poorman/PoorManWindow.cpp b/src/apps/poorman/PoorManWindow.cpp index 1d15415b17..caa639a029 100644 --- a/src/apps/poorman/PoorManWindow.cpp +++ b/src/apps/poorman/PoorManWindow.cpp @@ -251,7 +251,6 @@ PoorManWindow::MessageReceived(BMessage* message) struct in_addr sin_addr; sin_addr.s_addr = address; if (inet_ntop(AF_INET, &sin_addr, addr, sizeof(addr)) != NULL) { - addr[strlen(addr)] = '\0'; line << '(' << addr << ") "; } }