Yair Zaslavsky has uploaded a new change for review.

Change subject: aaa: put failure as result only if there is no result already 
set
......................................................................

aaa: put failure as result only if there is no result already set

Change-Id: I32c71b854fde5c0734a6c1739eaaa3745a250273
Topic: AAA
Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com>
---
M 
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapAuthenticateUserCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/48/27248/1

diff --git 
a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapAuthenticateUserCommand.java
 
b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapAuthenticateUserCommand.java
index b1c8205..f41ab43 100644
--- 
a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapAuthenticateUserCommand.java
+++ 
b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapAuthenticateUserCommand.java
@@ -81,7 +81,7 @@
         }
 
         if (!getSucceeded()) {
-            getParameters().getOutputMap().put(Authn.InvokeKeys.RESULT, 
Authn.AuthResult.GENERAL_ERROR);
+            
getParameters().getOutputMap().putIfAbsent(Authn.InvokeKeys.RESULT, 
Authn.AuthResult.GENERAL_ERROR);
         }
     }
 


-- 
To view, visit http://gerrit.ovirt.org/27248
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I32c71b854fde5c0734a6c1739eaaa3745a250273
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to