From 1a94ac4a0524a760c1c3ea7a5ca9032e994785c2 Mon Sep 17 00:00:00 2001 From: Ondřej Vašík Date: Wed, 2 Sep 2009 10:34:27 +0100 Subject: 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". --- src/copy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/copy.c b/src/copy.c index d1e508d7f..e604ec55a 100644 --- a/src/copy.c +++ b/src/copy.c @@ -853,7 +853,7 @@ copy_reg (char const *src_name, char const *dst_name, if (x->preserve_xattr && ! copy_attr_by_fd (src_name, source_desc, dst_name, dest_desc, x) && x->require_preserve_xattr) - return false; + return_val = false; if (x->preserve_mode || x->move_mode) { -- cgit v1.2.3-70-g09d2