Fix two small leaks.

* src/remove.c (AD_stack_clear): New function.
(rm_1): Use it.
(AD_pop_and_chdir): Free *prev_dir just before longjmp.

2 files changed, 17 insertions, 0 deletions

diff --git a/ChangeLog b/ChangeLog
index c263aeeaa..183a8d155 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2006-07-03  Jim Meyering  <jim@meyering.net>
 
+	Fix two small leaks.
+	* src/remove.c (AD_stack_clear): New function.
+	(rm_1): Use it.
+	(AD_pop_and_chdir): Free *prev_dir just before longjmp.
+
 	* tests/Makefile.am, tests/*/Makefile.am: (TESTS_ENVIRONMENT):
 	Add $VG_PATH_PREFIX as a prefix to $PATH
 
diff --git a/src/remove.c b/src/remove.c
index a219955bc..bc0b92b7b 100644
--- a/src/remove.c
+++ b/src/remove.c
@@ -345,6 +345,15 @@ AD_stack_pop (Dirstack_state *ds)
   obstack_blank (&ds->Active_dir, -(int) sizeof (struct AD_ent));
 }
 
+static void
+AD_stack_clear (Dirstack_state *ds)
+{
+  while (0 < AD_stack_height (ds))
+    {
+      AD_stack_pop (ds);
+    }
+}
+
 static Dirstack_state *
 ds_init (void)
 {
@@ -469,6 +478,7 @@ AD_pop_and_chdir (DIR **dirp, Dirstack_state *ds, char **prev_dir)
 	  close (fd);
 
 	next_cmdline_arg:;
+	  free (*prev_dir);
 	  longjmp (ds->current_arg_jumpbuf, 1);
 	}
     }
@@ -1375,6 +1385,8 @@ rm_1 (Dirstack_state *ds, char const *filename,
 	status = RM_ERROR;
       else
 	status = remove_dir (fd_cwd, ds, filename, x, cwd_errno);
+
+      AD_stack_clear (ds);
     }
 
   ds_clear (ds);