gryf/wmaker
1
0
Fork 0
mirror of https://github.com/gryf/wmaker.git synced 2025-12-18 20:10:29 +01:00
Code Issues Projects Releases Wiki Activity

Fix memory leak reported by Coverity

Browse Source 
As reported in CID #331577, we re-use the variable 'tmp' without freeing
the previously allocated pointer.

Signed-off-by: Christophe CURIS <christophe.curis@free.fr>
This commit is contained in:
Christophe CURIS
2021-05-14 19:36:33 +02:00
committed by Carlos R. Mafra
parent fd56152202
commit 97a9300aa4
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/rootmenu.c
View File

@@ -673,9 +673,13 @@ static void constructMenu(WMenu * menu, WMenuEntry * entry)
tmp = wexpandpath(path[i]);
if (strstr(tmp, "#usergnusteppath#") == tmp)
if (strstr(tmp, "#usergnusteppath#") == tmp) {
char *old_tmp = tmp;
tmp = wstrconcat(wusergnusteppath(),
tmp + 17);
wfree(old_tmp);
}
wfree(path[i]);
lpath = getLocalizedMenuFile(tmp);