Re: [PATCH] copy_fd: close ifd on error

Sam Ravnborg [off-list ref] writes:

In copy_fd when write fails we ought to close input file descriptor.

Signed-off-by: Sam Ravnborg <redacted>
---

Found while looking at the source for inspiration for something else.
I did not see any error related to this.

Thanks.  I think the only place this matters is git-local-fetch
which nobody uses these days where it can leak handful ifd.
All other users immediately exit upon seeing failure return from
the function.

Of course this would matter much when we do libification.