This is an automated email from the ASF dual-hosted git repository. twolf pushed a commit to branch dev_3.0 in repository https://gitbox.apache.org/repos/asf/mina-sshd.git
commit 04fe313f2b31b32e4c68b65d74702f6c301d9bb5 Author: Thomas Wolf <[email protected]> AuthorDate: Sun Sep 21 22:00:05 2025 +0200 SecurityProviderRegistrar: binary search needs a sorted array Somehow I suspect nobody is actually using these configuration possibilities, otherwise someone surely would have tripped over this. --- .../sshd/common/util/security/SecurityProviderRegistrar.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sshd-common/src/main/java/org/apache/sshd/common/util/security/SecurityProviderRegistrar.java b/sshd-common/src/main/java/org/apache/sshd/common/util/security/SecurityProviderRegistrar.java index 68c9756a4..4df4f8022 100644 --- a/sshd-common/src/main/java/org/apache/sshd/common/util/security/SecurityProviderRegistrar.java +++ b/sshd-common/src/main/java/org/apache/sshd/common/util/security/SecurityProviderRegistrar.java @@ -274,8 +274,12 @@ public interface SecurityProviderRegistrar extends SecurityProviderChoice, Optio return false; } - if ((values.length == 1) && isAllOptionsValue(values[0])) { - return true; + if (values.length == 1) { + if (isAllOptionsValue(values[0])) { + return true; + } + } else { + Arrays.sort(values, String.CASE_INSENSITIVE_ORDER); } String effectiveName = getEffectiveSecurityEntityName(entityType, name);
