[PATCH weston] weston-launch: On error putenv returns a non-zero value
Dawid Gajownik
gajownik at gmail.com
Sun Aug 9 08:50:40 PDT 2015
According to POSIX standard "upon successful completion, putenv() shall
return 0; otherwise, it shall return a non-zero value." Unlike in
setenv() we should not be checking only for negative values.
Signed-off-by: Dawid Gajownik <gajownik at gmail.com>
---
src/weston-launch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/weston-launch.c b/src/weston-launch.c
index 90a070f..b8dfb17 100644
--- a/src/weston-launch.c
+++ b/src/weston-launch.c
@@ -603,7 +603,7 @@ setup_session(struct weston_launch *wl)
env = pam_getenvlist(wl->ph);
if (env) {
for (i = 0; env[i]; ++i) {
- if (putenv(env[i]) < 0)
+ if (putenv(env[i]) != 0)
error(0, 0, "putenv %s failed", env[i]);
}
free(env);
--
2.4.3
More information about the wayland-devel
mailing list