From: Jim Meyering <jim@meyering.net>
Date: Sun, 25 Oct 1998 13:58:50 +0000 (+0000)
Subject: (remove_cwd_entries): Don't apply CLOSEDIR to a NULL
X-Git-Tag: FILEUTILS-4_0-b6~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9e630b86cb456b1551feed0f7e9ae75931a2af29;p=platform%2Fupstream%2Fcoreutils.git

(remove_cwd_entries): Don't apply CLOSEDIR to a NULL
pointer. (provoke with `mkdir -m 0100 x; rm -rf x')
Upon CLOSEDIR failure, set `status' to RM_ERROR, not RM_OK.
(remove_dir): Return `status', rather than always RM_OK.
---

diff --git a/src/remove.c b/src/remove.c
index 2922821..e35c1e3 100644
--- a/src/remove.c
+++ b/src/remove.c
@@ -575,12 +575,15 @@ remove_cwd_entries (const struct rm_options *x)
     }
   while (dirp == NULL);
 
-  if (CLOSEDIR (dirp))
+  if (dirp)
     {
-      error (0, errno, "%s", full_filename ("."));
-      status = RM_OK;
+      if (CLOSEDIR (dirp))
+	{
+	  error (0, errno, "%s", full_filename ("."));
+	  status = RM_ERROR;
+	}
+      dirp = NULL;
     }
-  dirp = NULL;
 
   if (ht)
     {
@@ -766,7 +769,7 @@ remove_dir (struct File_spec *fs, int need_save_cwd, const struct rm_options *x)
       return RM_ERROR;
     }
 
-  return RM_OK;
+  return status;
 }
 
 /* Remove the file or directory specified by FS after checking appropriate