You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by am...@apache.org on 2004/11/24 18:35:56 UTC
svn commit: r106438 - /geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java
Author: ammulder
Date: Wed Nov 24 09:35:55 2004
New Revision: 106438
URL: http://svn.apache.org/viewcvs?view=rev&rev=106438
Log:
Only try to release the lock if it needs to be released
(probably never, but what the heck)
Modified:
geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java
Modified: geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java
Url: http://svn.apache.org/viewcvs/geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java?view=diff&rev=106438&p1=geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java&r1=106437&p2=geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java&r2=106438
==============================================================================
--- geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java (original)
+++ geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/realm/providers/FileAuditLoginModule.java Wed Nov 24 09:35:55 2004
@@ -91,7 +91,9 @@
writer.println(DATE_FORMAT.format(date)+" - "+action+" - "+username);
writer.flush();
writer.close();
- lock.release();
+ if(lock.isValid()) {
+ lock.release();
+ }
} catch (IOException e) {
throw new RuntimeException("Unable to write to authentication log file", e);
}