From: Jim Meyering <meyering@redhat.com>
Date: Mon, 7 Jan 2008 16:10:07 +0000 (+0100)
Subject: cp (but not copy.c): plug a small leak.
X-Git-Tag: v6.9.92~6
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b84880d980d85dc68a1d43fec3ea4f18fd3feeda;p=thirdparty%2Fcoreutils.git

cp (but not copy.c): plug a small leak.

* src/cp.c (do_copy) [--parents]: Free the attribute list.
---

diff --git a/ChangeLog b/ChangeLog
index c03c0d1593..0018fdcf49 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2008-01-07  Jim Meyering  <meyering@redhat.com>
 
+	cp (but not copy.c): plug a small leak.
+	* src/cp.c (do_copy) [--parents]: Free the attribute list.
+
 	Make a racy test failure less likely to happen.
 	* tests/misc/tty-eof: Uncomment a debug "warn".
 	This change happens to make this test far less likely to fail.
diff --git a/src/cp.c b/src/cp.c
index be3701fc8c..01d98cc647 100644
--- a/src/cp.c
+++ b/src/cp.c
@@ -683,6 +683,16 @@ do_copy (int n_files, char **file, const char *target_directory,
 				  attr_list, x);
 	    }
 
+	  if (parents_option)
+	    {
+	      while (attr_list)
+		{
+		  struct dir_attr *p = attr_list;
+		  attr_list = attr_list->next;
+		  free (p);
+		}
+	    }
+
 	  free (dst_name);
 	}
     }