You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tw...@apache.org on 2025/09/21 21:09:53 UTC
(mina-sshd) 01/02: SecurityProviderRegistrar: binary search needs a sorted array
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 <tw...@apache.org>
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);