From: Denys Vlasenko <vda.linux@googlemail.com>
Date: Sat, 4 Aug 2018 19:06:02 +0000 (+0200)
Subject: tar: fix a thinko in prev commit - we need to copy to _unused_ fd
X-Git-Tag: 1_30_0~475
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=2005d3ff3661220f11e8ff1911b24051b3669566;p=oweals%2Fbusybox.git

tar: fix a thinko in prev commit - we need to copy to _unused_ fd

function                                             old     new   delta
vfork_compressor                                     257     246     -11

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---

diff --git a/archival/tar.c b/archival/tar.c
index 120c77f3b..1c71f7f66 100644
--- a/archival/tar.c
+++ b/archival/tar.c
@@ -632,8 +632,7 @@ static void NOINLINE vfork_compressor(int tar_fd, const char *gzip)
 			 * Swapping move_fd's order wouldn't work:
 			 * data.rd is 1 and _it_ would be destroyed.
 			 */
-			xmove_fd(tfd, 3);
-			tfd = 3;
+			tfd = dup(tfd);
 		}
 		xmove_fd(data.rd, 0);
 		xmove_fd(tfd, 1);