[PATCH 05/15] WINGs: fix memory leak in WMGetBrowserPaths (Coverity #50111)

[Christophe]

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.