Untested... If there is an exception thrown during the processing, the file won't get deleted because it isn't in a try/finally statement... -jon cvs diff StandardManager.java Index: StandardManager.java =================================================================== RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/se ssion/StandardManager.java,v retrieving revision 1.10 diff -r1.10 StandardManager.java 353a354,355 > try { > 445,447c447,451 < // Delete the persistent storage file < file.delete(); < --- > } finally { > // Delete the persistent storage file > if (file != null && file.exists() ) > file.delete(); > }
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- Re: cvs commit:jakarta-tomcat-4.0/catalina/src/share/o... Jon Stevens
- Re: cvs commit:jakarta-tomcat-4.0/catalina/src/sha... Remy Maucherat
- Re: cvscommit:jakarta-tomcat-4.0/catalina/src/... Jon Stevens
- Re: cvscommit:jakarta-tomcat-4.0/catalina/... Remy Maucherat
- Re:cvscommit:jakarta-tomcat-4.0/catal... Jon Stevens
- Re: Re:cvscommit:jakarta-tomcat-4... Remy Maucherat
- [t4] again...classloader stuff... Jon Stevens
- Re: [t4] again...classloader stuf... Remy Maucherat
- Re: [t4] again...classloader stuf... Jon Stevens
- Re: [t4 patch] delete the damn fi... Jon Stevens
- Re: [t4 patch] delete the damn fi... Remy Maucherat
- [t4] classloader doesn't work Jon Stevens
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... craigmcc
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... pier
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... remm
- cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apach... patrickl