Repository: karaf Updated Branches: refs/heads/karaf-3.0.x 39fe55632 -> bd0a2949f
[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/bd0a2949 Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/bd0a2949 Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/bd0a2949 Branch: refs/heads/karaf-3.0.x Commit: bd0a2949f576649dd57028c96da549248df19d3b Parents: 39fe556 Author: Freeman Fang <[email protected]> Authored: Wed Apr 16 09:16:47 2014 +0800 Committer: Freeman Fang <[email protected]> Committed: Wed Apr 16 09:17:42 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/bd0a2949/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 4bb15cf..9723de8 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 @@ -192,7 +192,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)); } @@ -242,7 +242,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)); }
