abhay123lp / jdigest

Automatically exported from code.google.com/p/jdigest
0 stars 0 forks source link

JDigest should ignore symbolic links in Linux #5

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently, JDigest follows symbolic links. This can cause issues.

Test case:

inside a temp folder:
$ mkdir test01
$ touch test01/test02.txt
$ ln -s ../test01 test01/test03
$ ls -l test01
total 0
-rw-r--r-- 1 sephiroth sephiroth 0 2011-07-09 20:34 test02.txt
lrwxrwxrwx 1 sephiroth sephiroth 9 2011-07-09 20:34 test03 -> ../test01
$ jdigest --digest test01/ test01.md5

JDigest will fail due to the below error:
java.io.FileNotFoundException: 
test01/test03/test03/test03/test03/test03/test03/test03/test03/test03/test03/tes
t03/test03/test03/test03/test03/test03/test03/test03/test03/test03/test03/test03
/test03/test03/test03/test03/test03/test03/test03/test03/test03/test03/test03/te
st03/test03/test03/test03/test03/test03/test03/test03 (Is a directory)

Basically, it is recursing into the Symbolic link until the path becomes longer 
than what the OS can support, and then you get the exception.

Original issue reported on code.google.com by gdass...@gmail.com on 9 Jul 2011 at 11:37

GoogleCodeExporter commented 9 years ago
Fixed in R35

Original comment by gdass...@gmail.com on 9 Jul 2011 at 2:18