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

Added missing <use-retval/>-tag to strdup()-like functions. In case the return value is not taken in account, a logical error is revealed.

This commit is contained in:
orbitcowboy 2019-02-03 20:38:33 +01:00
parent b472d74f1f
commit acebfac176
3 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cfg/gtk.cfg
View File

@ -18088,6 +18088,7 @@
<function name="g_strdup"> <function name="g_strdup">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<returnValue type="gchar *"/> <returnValue type="gchar *"/>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>

3
cfg/posix.cfg
View File

@ -701,6 +701,7 @@ The obsolescent function 'usleep' is called. POSIX.1-2001 declares usleep() func
<function name="strdup"> <function name="strdup">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<returnValue type="char *"/> <returnValue type="char *"/>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>
@ -711,6 +712,7 @@ The obsolescent function 'usleep' is called. POSIX.1-2001 declares usleep() func
<function name="strndup"> <function name="strndup">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<returnValue type="char *"/> <returnValue type="char *"/>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>
@ -725,6 +727,7 @@ The obsolescent function 'usleep' is called. POSIX.1-2001 declares usleep() func
<function name="wcsdup"> <function name="wcsdup">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<returnValue type="wchar_t *"/> <returnValue type="wchar_t *"/>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>

2
cfg/windows.cfg
View File

@ -2154,6 +2154,7 @@ HFONT CreateFont(
unsigned char *_mbsdup(const unsigned char *strSource);--> unsigned char *_mbsdup(const unsigned char *strSource);-->
<function name="_strdup,_wcsdup,_mbsdup,_tcsdup"> <function name="_strdup,_wcsdup,_mbsdup,_tcsdup">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>
@ -2163,6 +2164,7 @@ HFONT CreateFont(
wchar_t *_wcsdup_dbg(const wchar_t *strSource, int blockType, const char *filename, int linenumber);--> wchar_t *_wcsdup_dbg(const wchar_t *strSource, int blockType, const char *filename, int linenumber);-->
<function name="_strdup_dbg,_wcsdup_dbg,_tcsdup_dbg"> <function name="_strdup_dbg,_wcsdup_dbg,_tcsdup_dbg">
<noreturn>false</noreturn> <noreturn>false</noreturn>
<use-retval/>
<arg nr="1"> <arg nr="1">
<not-null/> <not-null/>
<not-uninit/> <not-uninit/>