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 72371963fe8216644ed3790d1dd9d7ca4b72f611..d27629829f5f0a1638a2a6d40848bc2692c5cdaf 100644 (file)
--- 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);