Skip to content

#2511 LDAPUtils.parseDN looks for attributes by name+"="; empty values don't throw - #2512

Open
winne42 wants to merge 1 commit into
bcgit:mainfrom
winne42:#2511-ldap-parsedn-attribute-match
Open

winne42 wants to merge 1 commit into
bcgit:mainfrom
winne42:#2511-ldap-parsedn-attribute-match

Conversation

@winne42

@winne42 winne42 commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Closes #2511.

AI Disclosure

Parts of this contribution were produced with the assistance of a generative AI tool (Claude Code), under the direction and review of the submitter, in line with the contributing guidelines. The submitter has reviewed and understands the code. To the best of his knowledge, the contribution does not reproduce third-party material.

begin = temp.indexOf('=');
temp = temp.substring(begin + 1);
if (temp.charAt(0) == ' ')
if (temp.length() > 0 && temp.charAt(0) == ' ')

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the 1.4 variant already checked for "=", but also threw for empty attribute

hubot pushed a commit that referenced this pull request Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LDAP CertStore searches on the wrong attribute value; empty DN values throw StringIndexOutOfBoundsException

1 participant