[#712] Fix StringIndexOutOfBoundsException on blank bind rule in ACI (#715)

Valery Kharseko

2 days ago 6399d638df9570b9a4537beffeb214dca253fcae

[#712] Fix StringIndexOutOfBoundsException on blank bind rule in ACI (#715)

 opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/BindRule.java

@@ -146,11 +146,17 @@
     * @throws AciException If the string is an invalid bind rule.
     */
    public static BindRule decode (String input) throws AciException {
        if (input == null || input.length() == 0)
        if (input == null)
        {
          return null;
        }
        String bindruleStr = input.trim();
        if (bindruleStr.isEmpty())
        {
          // A blank bind rule (e.g. "(          )") must be rejected as a
          // syntax error rather than throwing StringIndexOutOfBoundsException.
          return null;
        }
        char firstChar = bindruleStr.charAt(0);
        char[] bindruleArray = bindruleStr.toCharArray();


 opendj-server-legacy/src/test/java/org/opends/server/authorization/dseecompat/AciBodyTest.java

@@ -12,6 +12,7 @@
 * information: "Portions Copyright [year] [name of copyright owner]".
 *
 * Copyright 2013-2016 ForgeRock AS.
 * Portions Copyright 2026 3A Systems, LLC
 */
package org.opends.server.authorization.dseecompat;

@@ -61,4 +62,15 @@
    assertThat(aciBody.toString()).isEqualTo(aci);
    assertThat(aciBody.getPermBindRulePairs()).hasSize(1);
  }

  /**
   * A blank bind rule (only whitespace between the parentheses) must be
   * rejected with an {@link AciException} rather than crashing with an
   * unchecked {@code StringIndexOutOfBoundsException}. See issue #712.
   */
  @Test(expectedExceptions = AciException.class)
  public void decodeBlankBindRuleThrowsAciException() throws Exception
  {
    AciBody.decode("(version 3.0; acl \"C\"; allow (search)(          ) (userdn=\"ldap:///self\"); )");
  }
}