GHSA-5835-4gvc-32pc: Maddy Mail Server has an LDAP Filter Injection via Unsanitized Username
SummaryThe `auth.ldap` module constructs LDAP search filters and DN strings by directly interpolating user-supplied usernames via `strings.ReplaceAll()` without any LDAP filter escaping. An attacker who can reach the SMTP submission (AUTH PLAIN) or IMAP LOGIN interface can inject arbitrary LDAP filter expressions through the username field, enabling identity spoofing, LDAP directory enumeration, and attribute value extraction. The `go-ldap/ldap/v3` library—already imported in the same file—provides `ldap.EscapeFilter()` specifically for this purpose, but it is never called.Patched versionUpgrade to maddy 0.9.3.Details**Affected file:** `internal/auth/ldap/ldap.go`Three locations substitute the raw, attacker-controlled `username` into LDAP filter or DN strings with no escaping:**1. `Lookup()` — line 228 (filter injection)****2. `AuthPlain()` — line 255 (DN template injection)****3. `AuthPlain()` — line 260 (filter injection)**The `go-ldap/ldap/v3` library (v3.4.10, imported at line 17) provides `ldap.EscapeFilter()` which escapes `(`, `)`, `*`, `\`, and NUL per RFC 4515. It is never called on user input.**No input validation or filter escaping occurs at any point from the protocol handler to the LDAP query.**PoC**Prerequisites:**A maddy instance configured with `auth.ldap` using a `filter` directiveAn LDAP directory (e.g., OpenLDAP) with at least one userNetwork access to maddy's SMTP submission port (587) or IMAP port (993/143)**Step 1: Vulnerable maddy configuration**Assume the LDAP directory contains users `alice` (password: `alice_pass`) and `bob` (password: `bob_pass`).**Step 2: Verify normal authentication works****Step 3: Boolean-based blind LDAP injection (attribute extraction)**An attacker who holds valid credentials for any one account can extract that account's LDAP attributes character by character, using the authentication result (235 vs 535) as a boolean oracle.For extracting attributes of **other users** (whose password the attacker does not know), a timing side-channel is used instead. The `AuthPlain()` function has two distinct failure paths:**0 entries matched** (line 270): returns `ErrUnknownCredentials` immediately — **fast****1 entry matched, bind fails** (line 275): performs `conn.Bind()` over the network, then returns — **slow** (adds LDAP bind round-trip latency)Both return SMTP `535`, but the timing difference is measurable:Impact**Who is affected:** Any maddy deployment that uses the `auth.ldap` module with either the `filter` or `dn_template` directive. Both SMTP submission (AUTH PLAIN) and IMAP (LOGIN) authentication are affected.**What an attacker can do:****Identity spoofing:** An attacker who knows any valid user's password can authenticate using an injected username that resolves to that user's DN via LDAP filter manipulation. The authenticated session identity (`connState.AuthUser` in SMTP, `username` passed to IMAP storage lookup) is the raw injected string, not the actual LDAP user. This can bypass username-based authorization policies downstream.**LDAP directory enumeration:** By injecting wildcard filters (`*`) and observing error responses (e.g., "too many entries" vs. "unknown credentials"), an attacker can determine the number of users, probe for the existence of specific accounts, and discover directory structure.**Attribute value extraction via boolean-based blind injection:** An attacker who holds valid credentials for any one LDAP account can inject additional filter conditions (e.g., `bob)(description=X*`) that turn the authentication response into a boolean oracle, and the same technique works via a timing side-channel.**DN template path traversal:** When `dn_template` is used instead of `filter` (line 255), injected characters can manipulate the DN structure, potentially targeting entries in different organizational units or directory subtrees.Credit[Yuheng Zhang](mailto:zhangyuh25@mails.tsinghua.edu.cn), [Zihan Zhang](mailto:zzh1032@sjtu.edu.cn), [Jianjun Chen](mailto:jianjun@tsinghua.edu.cn) and [Teatime Lab LTD.](mailto:research@teatimelab.com)
CVSS v3.1: 8.2