Skip to content

Fix LDAP injection level 1 filter escaping - #221

Open
r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/ldap-injection-level-1
Open

r1kka6 wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
r1kka6:fix/ldap-injection-level-1

Conversation

@r1kka6

@r1kka6 r1kka6 commented Aug 10, 2026

Copy link
Copy Markdown

Summary

  • escape the level 1 username before embedding it in the LDAP filter
  • keep the change limited to the level 1 query construction in LDAPInjectionVulnerability.java

Verification

  • before the fix, GET /VulnerableApp/LDAPInjectionVulnerability/LEVEL_1?username=* returned all LDAP users via (uid=*)
  • after the fix, the same username=* request and username=*)(uid=* both return No users found
  • a normal username=alice request still returns only alice

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.

1 participant