coverity: Check return value of setenv

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
Acked-by: Serge E. Hallyn <serge.hallyn@ubuntu.com>

diff --git a/src/lxc/attach.c b/src/lxc/attach.c
index 87826522d5ee0a27d74242a71a54560435e10683..28cf9163504667a6b06ab9ea8eb5f744e40040e9 100644 (file)
--- a/src/lxc/attach.c
+++ b/src/lxc/attach.c
@@ -317,8 +317,10 @@ static int lxc_attach_set_environment(enum lxc_attach_env_policy_t policy, char*
                if (extra_keep_store) {
                        size_t i;
                        for (i = 0; extra_keep[i]; i++) {
-                               if (extra_keep_store[i])
-                                       setenv(extra_keep[i], extra_keep_store[i], 1);
+                               if (extra_keep_store[i]) {
+                                       if (setenv(extra_keep[i], extra_keep_store[i], 1) < 0)
+                                               SYSERROR("Unable to set environment variable");
+                               }
                                free(extra_keep_store[i]);
                        }
                        free(extra_keep_store);