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);
         }