From: Christophe CURIS <christophe.cu...@free.fr> As reported by Coverity, in the (rare) case where the wstrlcat would fail the array for the result would not be freed before returning.
Signed-off-by: Christophe CURIS <christophe.cu...@free.fr> --- WINGs/wbrowser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/WINGs/wbrowser.c b/WINGs/wbrowser.c index d390407..8d1858a 100644 --- a/WINGs/wbrowser.c +++ b/WINGs/wbrowser.c @@ -783,6 +783,7 @@ WMArray *WMGetBrowserPaths(WMBrowser * bPtr) for (i = 0; i <= column; i++) { if (wstrlcat(path, bPtr->pathSeparator, slen) >= slen) { wfree(path); + WMFreeArray(paths); return NULL; } if (i == column) { -- 1.9.2 -- To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.