From: Christophe CURIS <christophe.cu...@free.fr>

As pointed by Coverity, if the user does not a an entry in the password
file then the function would assume its home path to be "/" but still
continue and later try to check for user->pw_dir which would dereference
the NULL pointer.

Signed-off-by: Christophe CURIS <christophe.cu...@free.fr>
---
 WINGs/findfile.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/WINGs/findfile.c b/WINGs/findfile.c
index 42845d0..048731e 100644
--- a/WINGs/findfile.c
+++ b/WINGs/findfile.c
@@ -57,6 +57,7 @@ char *wgethomedir()
        if (!user) {
                werror(_("could not get password entry for UID %i"), getuid());
                home = "/";
+               return home;
        }
 
        if (!user->pw_dir)
-- 
1.9.2


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.

Reply via email to