keycloak-aplcache

KEYCLOAK-2235 Fix possible transaction leak at startup with …

2/1/2016 6:47:46 PM

add-user

mposolda

Commit: 6b2fb9a

Tree: 8f28d68

Parents: c5b0bb6

Changes

services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java 1(+1 -0)

Details

services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java 1(+1 -0)

diff --git a/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java b/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
index ff99c0b..f59263c 100644
--- a/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
+++ b/services/src/main/java/org/keycloak/services/resources/KeycloakApplication.java
@@ -326,6 +326,7 @@ public class KeycloakApplication extends Application {
                             session.getTransaction().commit();
                             logger.addUserSuccess(userRep.getUsername(), realmRep.getRealm());
                         } catch (ModelDuplicateException e) {
+                            session.getTransaction().rollback();
                             logger.addUserFailedUserExists(userRep.getUsername(), realmRep.getRealm());
                         } catch (Throwable t) {
                             session.getTransaction().rollback();