Merge pull request #3275 from brauner/2020-03-05/api_extension

utils: only move_fd() when fdopen() has been successful

Merge pull request #3275 from brauner/2020-03-05/api_extension
1b3e4274 · Stéphane Graber · GitHub · 4f31df93 · 92bdc593 · 1b3e4274
Unverified Commit 1b3e4274 authored Mar 05, 2020 by Stéphane Graber Committed by GitHub Mar 05, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

utils.c src/lxc/utils.c +2 -1

No files found.
--- a/src/lxc/utils.c
+++ b/src/lxc/utils.c
@@ -1724,9 +1724,10 @@ static int process_dead(/* takes */ int status_fd)
 		return -1;

 	/* transfer ownership of fd */
-	f = fdopen(move_fd(dupfd), "re");
+	f = fdopen(dupfd, "re");
 	if (!f)
 		return -1;
+	move_fd(dupfd);

 	ret = 0;
 	while (getline(&line, &n, f) != -1) {