X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=security%2Fruntime%2Forg.argeo.security.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fjcr%2FOsJcrAuthenticationProvider.java;h=9abac5972a7f584fc1aaca81326d49877e4c41af;hb=8b8ee149b20e2578a55e17413fa5f7399ff7ba14;hp=2eadf5669724e5f7cb6000e2819f766f929e72c5;hpb=c813b98b8339977ff0fb5f6209b54e5852d3ff6e;p=lgpl%2Fargeo-commons.git

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/jcr/OsJcrAuthenticationProvider.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/jcr/OsJcrAuthenticationProvider.java
index 2eadf5669..9abac5972 100644
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/jcr/OsJcrAuthenticationProvider.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/jcr/OsJcrAuthenticationProvider.java
@@ -35,7 +35,7 @@ public class OsJcrAuthenticationProvider extends OsAuthenticationProvider {
 				Session session = null;
 				try {
 					session = repository.login(workspace);
-					// WARNING: at this stage we assume that teh java properties
+					// WARNING: at this stage we assume that the java properties
 					// will have the same value
 					String userName = System.getProperty("user.name");
 					Node userHome = JcrUtils.getUserHome(session, userName);
@@ -44,6 +44,7 @@ public class OsJcrAuthenticationProvider extends OsAuthenticationProvider {
 								homeBasePath, userName);
 					// authen.setDetails(getUserDetails(userHome, authen));
 				} catch (RepositoryException e) {
+					JcrUtils.discardQuietly(session);
 					throw new ArgeoException(
 							"Unexpected exception when synchronizing OS and JCR security ",
 							e);