From 565621d070ab0cbbabb288bcfb6bc099557f7507 Mon Sep 17 00:00:00 2001 From: Tim Cava Date: Wed, 31 Jul 2013 09:28:46 +0000 Subject: [PATCH] Use strmcpy, instead of strcpy, to initialize FSstr. Fixes a potential buffer overflow found by Coverity. git-svn-id: svn://svn.code.sf.net/p/bitchx/code/trunk@302 13b04d17-f746-0410-82c6-800466cd88b0 --- dll/fserv/fserv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dll/fserv/fserv.c b/dll/fserv/fserv.c index 4fdd766..1bde3ea 100644 --- a/dll/fserv/fserv.c +++ b/dll/fserv/fserv.c @@ -1162,7 +1162,7 @@ char buffer[BIG_BUFFER_SIZE+1]; add_completion_type("fsload", 3, FILE_COMPLETION); add_timer(0, empty_string, get_dllint_var("fserv_time"), 1, impress_me, NULL, NULL, -1, "fserv"); - strcpy(FSstr, cparse(FS, NULL, NULL)); + strmcpy(FSstr, cparse(FS, NULL, NULL), sizeof(FSstr)); put_it("%s %s", FSstr, convert_output_format("$0 v$1 by panasync.", "%s %s", fserv_version, AUTO_VERSION)); sprintf(buffer, "$0+%s by panasync - $2 $3", fserv_version); fset_string_var(FORMAT_VERSION_FSET, buffer);