From 0b85a64bf075779987d61c2f26c501dfd19a4d08 Mon Sep 17 00:00:00 2001 From: Andy Green Date: Mon, 15 Feb 2016 21:08:18 +0800 Subject: [PATCH] coverity 158147 test ping sprintf bounds Well, just in the test app arg processing, but yes... Signed-off-by: Andy Green --- test-server/test-ping.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test-server/test-ping.c b/test-server/test-ping.c index 1548f1c4..ff2d8e35 100644 --- a/test-server/test-ping.c +++ b/test-server/test-ping.c @@ -450,7 +450,8 @@ int main(int argc, char **argv) /* create client websockets using dumb increment protocol */ address = argv[optind]; - sprintf(ads_port, "%s:%u", address, port & 65535); + snprintf(ads_port, sizeof(ads_port), "%s:%u", + address, port & 65535); lwsl_notice("Connecting to %s...\n", ads_port); memset(&i, 0, sizeof(i)); i.context = context;