cp: don't leak resources for each xattr preservation failure
authorOndřej Vašík <ovasik@redhat.com>
Wed, 2 Sep 2009 09:34:27 +0000 (10:34 +0100)
committerPádraig Brady <P@draigBrady.com>
Thu, 3 Sep 2009 09:00:32 +0000 (10:00 +0100)
commit1a94ac4a0524a760c1c3ea7a5ca9032e994785c2
tree5967a5342034bc49f5c9db71272944415e2441ba
parent1b2d2635eeabce4a2176427dc1de9c1b9b87bc7d
cp: don't leak resources for each xattr preservation failure

* src/copy.c (copy_reg): Don't return from the function after an
unsuccessful and required preservation of extended attributes.
This resulted in leaking the copy buffer and file descriptors.
* NEWS (Bug fixes): Mention the fix.
The bug was introduced in coreutils-7.1 via commit 0889381c, 2009-01-23,
"cp/mv: add xattr support".
NEWS
src/copy.c