You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shiro.apache.org by lh...@apache.org on 2012/07/25 20:15:11 UTC
svn commit: r1365683 -
/shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java
Author: lhazlewood
Date: Wed Jul 25 18:15:11 2012
New Revision: 1365683
URL: http://svn.apache.org/viewvc?rev=1365683&view=rev
Log:
SHIRO-377: Applied patch.
Modified:
shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java
Modified: shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java
URL: http://svn.apache.org/viewvc/shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java?rev=1365683&r1=1365682&r2=1365683&view=diff
==============================================================================
--- shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java (original)
+++ shiro/branches/1.2.x/core/src/main/java/org/apache/shiro/realm/text/PropertiesRealm.java Wed Jul 25 18:15:11 2012
@@ -34,7 +34,6 @@ import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.TimeUnit;
-
/**
* A {@link TextConfigurationRealm} that defers all logic to the parent class, but just enables
* {@link java.util.Properties Properties} based configuration in addition to the parent class's String configuration.
@@ -168,6 +167,9 @@ public class PropertiesRealm extends Tex
}
}
+ /**
+ * Destroy reload scheduler if one exists.
+ */
public void destroy() {
try {
if (scheduler != null) {
@@ -177,6 +179,8 @@ public class PropertiesRealm extends Tex
if (log.isInfoEnabled()) {
log.info("Unable to cleanly shutdown Scheduler. Ignoring (shutting down)...", e);
}
+ } finally {
+ scheduler = null;
}
}