Do not sort lists that have only one element
authorkkolinko <kkolinko@13f79535-47bb-0310-9956-ffa450edef68>
Tue, 4 Oct 2011 05:07:52 +0000 (05:07 +0000)
committerkkolinko <kkolinko@13f79535-47bb-0310-9956-ffa450edef68>
Tue, 4 Oct 2011 05:07:52 +0000 (05:07 +0000)
git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1178684 13f79535-47bb-0310-9956-ffa450edef68

java/org/apache/catalina/ha/session/SerializablePrincipal.java
java/org/apache/catalina/realm/GenericPrincipal.java

index 5955791..5d1101f 100644 (file)
@@ -108,7 +108,7 @@ public class SerializablePrincipal  implements java.io.Serializable {
         if (roles != null) {
             this.roles = new String[roles.size()];
             this.roles = roles.toArray(this.roles);
-            if (this.roles.length > 0)
+            if (this.roles.length > 1)
                 Arrays.sort(this.roles);
         }
         if (userPrincipal instanceof Serializable) {
index 6b45faf..7a3a086 100644 (file)
@@ -127,7 +127,7 @@ public class GenericPrincipal implements Principal {
         if (roles != null) {
             this.roles = new String[roles.size()];
             this.roles = roles.toArray(this.roles);
-            if (this.roles.length > 0)
+            if (this.roles.length > 1)
                 Arrays.sort(this.roles);
         }
         this.loginContext = loginContext;