From b84880d980d85dc68a1d43fec3ea4f18fd3feeda Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Mon, 7 Jan 2008 17:10:07 +0100 Subject: [PATCH] cp (but not copy.c): plug a small leak. * src/cp.c (do_copy) [--parents]: Free the attribute list. --- ChangeLog | 3 +++ src/cp.c | 10 ++++++++++ 2 files changed, 13 insertions(+) diff --git a/ChangeLog b/ChangeLog index c03c0d1593..0018fdcf49 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2008-01-07 Jim Meyering + 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); } } -- 2.47.3