From: Eric Andersen <andersen@codepoet.org>
Date: Wed, 7 Apr 2004 17:59:04 +0000 (-0000)
Subject: Vladimir N. Oleynik writes:
X-Git-Tag: 1_00_pre10~21
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=66a56aa028b430a3d20ac47c20dccf2aad3b21a0;p=platform%2Fupstream%2Fbusybox.git

Vladimir N. Oleynik writes:

Hi.

Last changes (rev 1.12) to recursive_actions() by Christian Grigis
have problem.
Test for demonstrate:

$ mkdir aaa bbb ccc
$ su
# chown root bbb
# chmod 700 bbb
# exit
$ busybox chmod 777 -R .
./bbb: Permision denied

But "./ccc" mode not changed. Previous variant works fine,
errors skiped and continued recursion.


--w
vodz
---

diff --git a/libbb/recursive_action.c b/libbb/recursive_action.c
index 7237196..d276298 100644
--- a/libbb/recursive_action.c
+++ b/libbb/recursive_action.c
@@ -100,7 +100,7 @@ int recursive_action(const char *fileName,
 			return FALSE;
 		}
 		status = TRUE;
-		while (status && (next = readdir(dir)) != NULL) {
+		while ((next = readdir(dir)) != NULL) {
 			char *nextFile;
 
 			nextFile = concat_subpath_file(fileName, next->d_name);