From 5cc8090de237d5c765b7096278c9af3eb6089de1 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Mon, 29 Dec 1997 21:04:26 +0000 Subject: (remove_cwd_entries): Plug a gross leak -- don't call obstack_init inside the loop. --- src/rm.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'src/rm.c') diff --git a/src/rm.c b/src/rm.c index 476a0ab3e..2da31aeb8 100644 --- a/src/rm.c +++ b/src/rm.c @@ -533,6 +533,8 @@ remove_cwd_entries (void) enum RM_status status = RM_OK; + obstack_init (&entry_name_pool); + if (dirp) { if (CLOSEDIR (dirp)) @@ -567,8 +569,6 @@ remove_cwd_entries (void) enum RM_status tmp_status; struct dirent *dp; - obstack_init (&entry_name_pool); - /* FILE should be skipped if it is `.' or `..', or if it is in the table, HT, of entries we've already processed. */ #define SKIPPABLE(Ht, File) (DOT_OR_DOTDOT(File) \ @@ -593,8 +593,7 @@ remove_cwd_entries (void) dp = readdir (dirp); -/* FIXME: add autoconf test to detect this. */ -#ifndef HAVE_WORKING_READDIR +#if ! HAVE_WORKING_READDIR if (dp == NULL) { /* Since we have probably modified the directory since it -- cgit v1.2.3-54-g00ecf