You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by da...@apache.org on 2015/09/07 08:06:12 UTC

[1/2] git commit: updated refs/heads/master to 8442bd8

Repository: cloudstack
Updated Branches:
  refs/heads/master b6e212f2c -> 8442bd87d


Fixing findbugs error due to PR #755 merge

issue: Boxing/unboxing to parse a primitive


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b69a69de
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b69a69de
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b69a69de

Branch: refs/heads/master
Commit: b69a69def7c9af29b0d0abfb530fa0a1a6e13b91
Parents: b6e212f
Author: Rajani Karuturi <ra...@citrix.com>
Authored: Mon Sep 7 10:33:29 2015 +0530
Committer: Rajani Karuturi <ra...@citrix.com>
Committed: Mon Sep 7 10:40:55 2015 +0530

----------------------------------------------------------------------
 .../ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b69a69de/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java
----------------------------------------------------------------------
diff --git a/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java b/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java
index 5570084..0df638a 100644
--- a/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java
+++ b/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java
@@ -85,7 +85,7 @@ public class ADLdapUserManagerImpl extends OpenLdapUserManagerImpl implements Ld
         boolean isDisabledUser = false;
         String userAccountControl = LdapUtils.getAttributeValue(result.getAttributes(), _ldapConfiguration.getUserAccountControlAttribute());
         if (userAccountControl != null) {
-            int control = Integer.valueOf(userAccountControl);
+            int control = Integer.parseInt(userAccountControl);
             // second bit represents disabled user flag in AD
             if ((control & 2) > 0) {
                 isDisabledUser = true;


[2/2] git commit: updated refs/heads/master to 8442bd8

Posted by da...@apache.org.
Merge pull request #779 from karuturi/findbugs-pr-755

Fixing findbugs error due to PR #755 mergeissue: Boxing/unboxing to parse a primitive

* pr/779:
  Fixing findbugs error due to PR #755 merge

Signed-off-by: Daan Hoogland <da...@onecht.net>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/8442bd87
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/8442bd87
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/8442bd87

Branch: refs/heads/master
Commit: 8442bd87d7a6ce3a503faccca86582a0ae483c2f
Parents: b6e212f b69a69d
Author: Daan Hoogland <da...@onecht.net>
Authored: Mon Sep 7 08:05:51 2015 +0200
Committer: Daan Hoogland <da...@onecht.net>
Committed: Mon Sep 7 08:05:52 2015 +0200

----------------------------------------------------------------------
 .../ldap/src/org/apache/cloudstack/ldap/ADLdapUserManagerImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------