]> Zhao Yanbai Git Server - minix.git/commitdiff
tests: don't chmod 777 everything upon cleanup
authorThomas Veerman <thomas@minix3.org>
Wed, 19 Sep 2012 14:58:32 +0000 (14:58 +0000)
committerThomas Veerman <thomas@minix3.org>
Wed, 19 Sep 2012 16:20:41 +0000 (16:20 +0000)
rm -rf works just fine no matter what mode bits are set (modulo
file ownership and current user id). Test 43 creates a symlink
to / and the chmod operation would change file permissions outside
of the test directory.

test/common.c

index 8e78336f47cf8df9f31bf420dcc05de663da69a0..2da265d974f8812a108516392c7fe906d0533929 100644 (file)
@@ -80,14 +80,6 @@ int test_nr;
 {
   char buf[128];
 
-  /* "rm -rf dir" will not work unless all the subdirectories have suitable
-   * permissions.  Minix chmod is not recursive so it is not easy to change
-   * all the permissions.  I had to fix opendir() to stop the bash shell
-   * from hanging when it opendir()s fifos.
-   */
-  sprintf(buf, "chmod 777 DIR_%02d DIR_%02d/* DIR_%02d/*/* >/dev/null 2>&1",
-         test_nr, test_nr, test_nr);
-  (void) system(buf);          /* usually fails */
   sprintf(buf, "rm -rf DIR_%02d >/dev/null 2>&1", test_nr);
   if (system(buf) != 0) printf("Warning: system(\"%s\") failed\n", buf);
 }