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

windows library: Add config for some Local* functions (#1139) 

Add configuration and tests for LocalAlloc, LocalFree and some other
Local* functions.
LocalReAlloc is currently not configured as an alloc/dealloc function
(like realloc is not configured in std.cfg), i am not sure how to
correctly configure it.
This commit is contained in:
Sebastian 2018-03-29 20:47:22 +02:00 committed by GitHub
parent 96167ffa51
commit 3c5777fbc6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 93 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

77
cfg/windows.cfg
View File

@ -5098,6 +5098,83 @@ HFONT CreateFont(
<not-uninit/>
</arg>
</function>
<!-- HLOCAL WINAPI LocalAlloc(
_In_ UINT uFlags,
_In_ SIZE_T uBytes); -->
<function name="LocalAlloc">
<noreturn>false</noreturn>
<returnValue type="HLOCAL"/>
<arg nr="1">
<not-uninit/>
<not-bool/>
<valid>0:</valid>
</arg>
<arg nr="2">
<not-uninit/>
<not-bool/>
</arg>
<warn severity="style" reason="Obsolete" alternatives="HeapAlloc"/>
</function>
<!-- HLOCAL WINAPI LocalReAlloc(
_In_ HLOCAL hMem,
_In_ SIZE_T uBytes,
_In_ UINT uFlags); -->
<function name="LocalReAlloc">
<noreturn>false</noreturn>
<returnValue type="HLOCAL"/>
<arg nr="1">
<not-bool/>
</arg>
<arg nr="2">
<not-uninit/>
<not-bool/>
</arg>
<arg nr="3">
<not-uninit/>
<not-bool/>
<valid>0:</valid>
</arg>
</function>
<!-- HLOCAL WINAPI LocalFree(_In_ HLOCAL hMem); -->
<function name="LocalFree">
<noreturn>false</noreturn>
<returnValue type="HLOCAL"/>
<arg nr="1">
<not-bool/>
</arg>
</function>
<!-- UINT WINAPI LocalSize(_In_ HLOCAL hMem); -->
<!-- UINT WINAPI LocalFlags(_In_ HLOCAL hMem); -->
<function name="LocalSize,LocalFlags">
<noreturn>false</noreturn>
<returnValue type="UINT"/>
<use-retval/>
<leak-ignore/>
<arg nr="1">
<not-null/>
<not-bool/>
</arg>
</function>
<!-- LPVOID WINAPI LocalLock(_In_ HLOCAL hMem); -->
<function name="LocalLock">
<noreturn>false</noreturn>
<returnValue type="LPVOID"/>
<leak-ignore/>
<arg nr="1">
<not-null/>
<not-bool/>
</arg>
</function>
<!-- BOOL WINAPI LocalUnlock(_In_ HLOCAL hMem); -->
<function name="LocalUnlock">
<noreturn>false</noreturn>
<returnValue type="BOOL"/>
<leak-ignore/>
<arg nr="1">
<not-null/>
<not-bool/>
</arg>
</function>
<!-- Intrinsic __noop https://docs.microsoft.com/en-us/cpp/intrinsics/noop -->
<function name="__noop">
<noreturn>false</noreturn>

16
test/cfg/windows.cpp
View File

@ -159,6 +159,10 @@ void validCode()
SecureZeroMemory(byteBuf, sizeof(byteBuf));
RtlFillMemory(byteBuf, sizeof(byteBuf), 0xff);
// cppcheck-suppress LocalAllocCalled
HLOCAL pLocalAlloc = LocalAlloc(1, 2);
LocalFree(pLocalAlloc);
// Intrinsics
__noop();
__noop(1, "test", NULL);
@ -326,6 +330,18 @@ void memleak_HeapAlloc()
// cppcheck-suppress memleak
}
void memleak_LocalAlloc()
{
LPTSTR pszBuf;
// cppcheck-suppress LocalAllocCalled
pszBuf = (LPTSTR)LocalAlloc(LPTR, MAX_PATH*sizeof(TCHAR));
(void)LocalSize(pszBuf);
(void)LocalFlags(pszBuf);
LocalLock(pszBuf);
LocalUnlock(pszBuf);
// cppcheck-suppress memleak
}
void resourceLeak_CreateSemaphoreA()
{
HANDLE hSemaphore;