coverity: #1425921

free allocated memory Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

coverity: #1425921
ec113ac4 · Christian Brauner · e89919e3 · ec113ac4
Unverified Commit ec113ac4 authored Dec 07, 2017 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 3 deletions

attach.c src/lxc/attach.c +10 -3

No files found.
--- a/src/lxc/attach.c
+++ b/src/lxc/attach.c
@@ -591,6 +591,7 @@ static char *lxc_attach_getpwshell(uid_t uid)
 		if (waitpid(pid, &status, 0) < 0) {
 			if (errno == EINTR)
 				goto again;
+			free(result);
 			return NULL;
 		}

@@ -599,14 +600,20 @@ static char *lxc_attach_getpwshell(uid_t uid)
 		 * don't.
 		 */

-		if (!WIFEXITED(status))
+		if (!WIFEXITED(status)) {
+			free(result);
 			return NULL;
+		}

-		if (WEXITSTATUS(status) != 0)
+		if (WEXITSTATUS(status) != 0) {
+			free(result);
 			return NULL;
+		}

-		if (!found)
+		if (!found) {
+			free(result);
 			return NULL;
+		}

 		return result;
 	} else {