https://issues.apache.org/bugzilla/show_bug.cgi?id=51704

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #1 from Mark Thomas <ma...@apache.org> 2011-08-26 17:12:51 UTC ---
There were ~25 uses of mkdirs() throughout the codebase. All have been reviewed
and fixed as appropriate.

If we are being this pedantic then

if (!dir.mkdirs() && !dir.isDirectory()) {

is a better test since it ensures the file that exists is a directory rather
than a regular file.

This has been fixed in trunk and 7.0.x and will be included in 7.0.21 onwards.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to