walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

windows.cfg: SendMessage must not be <leak-ignore/>

This commit is contained in:
PKEuS 2016-07-26 11:57:26 +02:00
parent 9082c097c8
commit 5c92c231b8
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cfg/windows.cfg
View File

@ -2161,7 +2161,6 @@ HFONT CreateFont(
</function> </function>
<function name="SendMessage,SendMessageA,SendMessageW"> <function name="SendMessage,SendMessageA,SendMessageW">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<leak-ignore/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>

9
test/testmemleak.cpp
View File

@ -5978,6 +5978,7 @@ private:
TEST_CASE(openfileNoLeak); TEST_CASE(openfileNoLeak);
TEST_CASE(returnValueNotUsed_tfopen_s); TEST_CASE(returnValueNotUsed_tfopen_s);
TEST_CASE(sendMessage);
} }
void openfileNoLeak() { void openfileNoLeak() {
@ -6011,6 +6012,14 @@ private:
"}"); "}");
TODO_ASSERT_EQUALS("[test.c:3]: (error) Resource leak: fp\n", "", errout.str()); TODO_ASSERT_EQUALS("[test.c:3]: (error) Resource leak: fp\n", "", errout.str());
} }
void sendMessage() {
check("void SetFont() {\n"
" HFONT hf = CreateFontIndirect(&lf);\n"
" SendMessage(hwnd, WM_SETFONT, (WPARAM)hf, TRUE);\n" // We do not know what the handler for the message will do with 'hf', so it might be closed later
"}");
ASSERT_EQUALS("", errout.str());
}
}; };
REGISTER_TEST(TestMemleakWindows) REGISTER_TEST(TestMemleakWindows)