coverity: #1425921

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

coverity: #1425921
d83e85cf · Christian Brauner · 241f7d14 · d83e85cf
Unverified Commit d83e85cf 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
@@ -557,6 +557,7 @@ static char *lxc_attach_getpwshell(uid_t uid)
 		if (waitpid(pid, &status, 0) < 0) {
 			if (errno == EINTR)
 				goto again;
+			free(result);
 			return NULL;
 		}

@@ -565,14 +566,20 @@ static char *lxc_attach_getpwshell(uid_t uid)
 		 * we assume we 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 {