[PATCH] ntdll/tests: Fix a string leak (Valgrind).
Signed-off-by: Sven Baars <sven.wine(a)gmail.com> --- dlls/ntdll/tests/om.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c index 84f97d4028..931628b69d 100644 --- a/dlls/ntdll/tests/om.c +++ b/dlls/ntdll/tests/om.c @@ -1335,6 +1335,8 @@ static void _test_object_type(unsigned line, HANDLE handle, const char *expected ok_(__FILE__,line)( len >= sizeof(OBJECT_TYPE_INFORMATION) + str->Length + sizeof(WCHAR), "unexpected len %u\n", len ); ok_(__FILE__,line)( str->Length == expect.Length && !memcmp( str->Buffer, expect.Buffer, expect.Length ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + + pRtlFreeUnicodeString( &expect ); } static void test_query_object(void) -- 2.17.1
Hi, While running your changed tests, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=53154 Your paranoid android. === debian9 (32 bit report) === ntdll: om.c:2118: Test failed: got 99 om.c:2133: Test failed: got 99 om.c:2133: Test failed: got 99 === debian9 (32 bit French report) === ntdll: om.c:2118: Test failed: got 99 om.c:2133: Test failed: got 99 === debian9 (32 bit Japanese:Japan report) === ntdll: om.c:2133: Test failed: got 99 === debian9 (32 bit Chinese:China report) === ntdll: om.c:2133: Test failed: got 99 === debian9 (32 bit WoW report) === ntdll: om.c:2133: Test failed: got 99 om.c:2133: Test failed: got 99 === debian9 (64 bit WoW report) === ntdll: om.c:2118: Test failed: got 99 om.c:2133: Test failed: got 99
participants (2)
-
Marvin -
Sven Baars