Author: ritchiem
Date: Wed Jun 20 02:12:43 2007
New Revision: 549011

URL: http://svn.apache.org/viewvc?view=rev&rev=549011
Log:
Proper update to PrincipalDatabase's to ensure they work correctly with # 
comments.

Modified:
    
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordFilePrincipalDatabase.java
    
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordVhostFilePrincipalDatabase.java

Modified: 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordFilePrincipalDatabase.java
URL: 
http://svn.apache.org/viewvc/incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordFilePrincipalDatabase.java?view=diff&rev=549011&r1=549010&r2=549011
==============================================================================
--- 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordFilePrincipalDatabase.java
 (original)
+++ 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordFilePrincipalDatabase.java
 Wed Jun 20 02:12:43 2007
@@ -211,17 +211,20 @@
             reader = new BufferedReader(new FileReader(_passwordFile));
             String line;
 
-            while ((line = reader.readLine()) != null && !line.startsWith("#"))
+            while ((line = reader.readLine()) != null)
             {
-                String[] result = _regexp.split(line);
-                if (result == null || result.length < 2)
+                if (!line.startsWith("#"))
                 {
-                    continue;
-                }
+                    String[] result = _regexp.split(line);
+                    if (result == null || result.length < 2)
+                    {
+                        continue;
+                    }
 
-                if (name.equals(result[0]))
-                {
-                    return result[1].toCharArray();
+                    if (name.equals(result[0]))
+                    {
+                        return result[1].toCharArray();
+                    }
                 }
             }
             return null;

Modified: 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordVhostFilePrincipalDatabase.java
URL: 
http://svn.apache.org/viewvc/incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordVhostFilePrincipalDatabase.java?view=diff&rev=549011&r1=549010&r2=549011
==============================================================================
--- 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordVhostFilePrincipalDatabase.java
 (original)
+++ 
incubator/qpid/branches/M2/java/broker/src/main/java/org/apache/qpid/server/security/auth/database/PlainPasswordVhostFilePrincipalDatabase.java
 Wed Jun 20 02:12:43 2007
@@ -61,17 +61,20 @@
                 reader = new BufferedReader(new FileReader(_passwordFile));
                 String line;
 
-                while ((line = reader.readLine()) != null && 
!line.startsWith("#"))
+                while ((line = reader.readLine()) != null)
                 {
-                    String[] result = _regexp.split(line);
-                    if (result == null || result.length < 3)
+                    if (!line.startsWith("#"))
                     {
-                        continue;
-                    }
+                        String[] result = _regexp.split(line);
+                        if (result == null || result.length < 3)
+                        {
+                            continue;
+                        }
 
-                    if (user.equals(result[0]))
-                    {
-                        return result[2].split(",");
+                        if (user.equals(result[0]))
+                        {
+                            return result[2].split(",");
+                        }
                     }
                 }
                 return null;


Reply via email to