Repository: karaf
Updated Branches:
  refs/heads/karaf-2.x 1bd45b3c7 -> 71bd2c545


[KARAF-2913]JAAS JDBCBackingEngine has the wrong condition to iterate results, 
no users are returned
(cherry picked from commit 5ae68719554dd13d72598ad18cef533518ab9cf4)


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

Branch: refs/heads/karaf-2.x
Commit: 71bd2c545e219a7d83b4b5bead2b2ade2b805989
Parents: 1bd45b3
Author: Freeman Fang <[email protected]>
Authored: Wed Apr 16 09:16:47 2014 +0800
Committer: Freeman Fang <[email protected]>
Committed: Wed Apr 16 09:18:19 2014 +0800

----------------------------------------------------------------------
 .../org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/karaf/blob/71bd2c54/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java
----------------------------------------------------------------------
diff --git 
a/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java
 
b/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java
index e6e3071..0365a7f 100644
--- 
a/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java
+++ 
b/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/jdbc/JDBCBackingEngine.java
@@ -193,7 +193,7 @@ public class JDBCBackingEngine implements BackingEngine {
                 //Remove from users
                 listUserStatement = 
connection.prepareStatement(selectUsersQuery);
                 usersResultSet = listUserStatement.executeQuery();
-                while (!usersResultSet.next()) {
+                while (usersResultSet.next()) {
                     String username = usersResultSet.getString("USERNAME");
                     users.add(new UserPrincipal(username));
                 }
@@ -243,7 +243,7 @@ public class JDBCBackingEngine implements BackingEngine {
 
                 rolesResultSet = listRolesStatement.executeQuery();
 
-                while (!rolesResultSet.next()) {
+                while (rolesResultSet.next()) {
                     String role = rolesResultSet.getString(1);
                     roles.add(new RolePrincipal(role));
                 }

Reply via email to