Lionel Debroux : msvcrt/tests: Fix memory leak (found by Smatch).
Alexandre Julliard
julliard at winehq.org
Fri Oct 26 08:34:48 CDT 2007
Module: wine
Branch: master
Commit: efebe8b29bc2c984cf07e38c7048689120217a1a
URL: http://source.winehq.org/git/wine.git/?a=commit;h=efebe8b29bc2c984cf07e38c7048689120217a1a
Author: Lionel Debroux <lionel_debroux at yahoo.fr>
Date: Fri Oct 26 11:15:26 2007 +0200
msvcrt/tests: Fix memory leak (found by Smatch).
---
dlls/msvcrt/tests/string.c | 12 +++++-------
1 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c
index 6dabc61..66b018a 100644
--- a/dlls/msvcrt/tests/string.c
+++ b/dlls/msvcrt/tests/string.c
@@ -388,7 +388,7 @@ static void test_strdup(void)
START_TEST(string)
{
- void *mem;
+ char mem[100];
static const char xilstring[]="c:/xilinx";
int nLen;
@@ -401,13 +401,11 @@ START_TEST(string)
/* MSVCRT memcpy behaves like memmove for overlapping moves,
MFC42 CString::Insert seems to rely on that behaviour */
- mem = malloc(100);
- ok(mem != NULL, "memory not allocated for size 0\n");
- strcpy((char*)mem,xilstring);
+ strcpy(mem,xilstring);
nLen=strlen(xilstring);
- pmemcpy((char*)mem+5, mem,nLen+1);
- ok(pmemcmp((char*)mem+5,xilstring, nLen) == 0,
- "Got result %s\n",(char*)mem+5);
+ pmemcpy(mem+5, mem,nLen+1);
+ ok(pmemcmp(mem+5,xilstring, nLen) == 0,
+ "Got result %s\n",mem+5);
/* Test _swab function */
test_swab();
More information about the wine-cvs
mailing list