From: Christophe CURIS <christophe.cu...@free.fr>
As pointed by Coverity, the string passed to 'findPositionInMenu' is not
freed, and as the function does not modify it it is not necessary to
allocate a new storage for the argument.
Signed-off-by: Christophe CURIS <christophe.cu...@free.fr>
---
util/wmmenugen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/util/wmmenugen.c b/util/wmmenugen.c
index 47771f5..8dbe426 100644
--- a/util/wmmenugen.c
+++ b/util/wmmenugen.c
@@ -175,7 +175,7 @@ static void addWMMenuEntryCallback(WMMenuEntry *aEntry)
}
if (aEntry->SubMenu)
- at = findPositionInMenu(wstrdup(aEntry->SubMenu));
+ at = findPositionInMenu(aEntry->SubMenu);
if (!at)
at = menu;
--
1.9.2
--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.
