[java-oidfed-common] 03/05: Declare index as final and exploit it in the if-clause.

Codeberg noreply at shibboleth.net
Wed Sep 23 07:34:32 UTC 2026


This is an automated email from the git hooks/post-receive script.

codeberg pushed a commit to branch main
in repository java-oidfed-common.

View the commit online:
https://codeberg.org/Shibboleth/java-oidfed-common/commit/3047b6fce7cf4009a5b9b2a505c8078e36c03c19

commit 3047b6fce7cf4009a5b9b2a505c8078e36c03c19
Author: Henri Mikkonen <henri.mikkonen at iki.fi>
AuthorDate: Wed Sep 23 10:13:04 2026 +0300

    Declare index as final and exploit it in the if-clause.
---
 .../metadata/cache/trustchain/DefaultTrustChainFetchingStrategy.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/metadata/cache/trustchain/DefaultTrustChainFetchingStrategy.java b/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/metadata/cache/trustchain/DefaultTrustChainFetchingStrategy.java
index 38bb8a1..0072b9d 100644
--- a/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/metadata/cache/trustchain/DefaultTrustChainFetchingStrategy.java
+++ b/oidfed-common-impl/src/main/java/net/shibboleth/oidfed/metadata/cache/trustchain/DefaultTrustChainFetchingStrategy.java
@@ -401,8 +401,8 @@ public class DefaultTrustChainFetchingStrategy extends AbstractIdentifiableIniti
         if (preSelectedChain.isEmpty()) {
             return true;
         }
-        int index = chain.size() > 1 ? chain.size() - 1 : chain.size();
-        if (preSelectedChain.size() < chain.size() || !preSelectedChain.get(index).equals(authority)) {
+        final int index = chain.size() > 1 ? chain.size() - 1 : chain.size();
+        if (preSelectedChain.size() <= index || !preSelectedChain.get(index).equals(authority)) {
             log.debug("Ignoring authority {} as it doesn't match with the preselected chain {}", authority,
                     preSelectedChain);
             return false;

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list