Skip to content

Commit 936508c

Browse files
committed
Close issue #602 by changing "/bin" to "/dev".
1 parent c6da91d commit 936508c

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/test/java/org/owasp/esapi/reference/ValidatorTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,10 @@ public void testIsValidDirectoryPath() throws IOException {
372372
// Unix specific paths should pass
373373
assertTrue(instance.isValidDirectoryPath("test6", "/", parent, false, errors)); // Root directory
374374
assertTrue(errors.size()==5);
375-
assertTrue(instance.isValidDirectoryPath("test7", "/bin", parent, false, errors)); // Always exist directory
375+
// Note, we used to say that about '/bin', but Ubuntu 20.x
376+
// changed '/bin' to a sym link to 'usr/bin'. We can't use '/etc'
377+
// because under MacOS, that is a sym link to 'private/etc'.
378+
assertTrue(instance.isValidDirectoryPath("test7", "/dev", parent, false, errors)); // Always exist directory
376379
assertTrue(errors.size()==5);
377380

378381
// Unix specific paths that should not exist or work

0 commit comments

Comments
 (0)