Bug #8258
open
ldap: set invalid_data event in all cases
Added by Giuseppe Longo 4 days ago.
Updated about 19 hours ago.
Description
While reviewing the LDAP code, I noticed that in parse_request:
1. tx.tx_data.updated_ts is not set to true when a new transaction is created.
2. LdapEvent::InvalidData is not set when a request is not parsed correctly.
- Status changed from New to In Progress
- Target version changed from TBD to 9.0.0-beta1
- Affected Versions git main added
- Affected Versions deleted (
9.0.0-beta1)
Do we need a backport to 8 as well? Is there a test?
- Subject changed from ldap: set tx_data.updated_ts and invalid_data event to ldap: set invalid_data event
1. tx.tx_data.updated_ts is not set to true when a new transaction is created.
For me this statement is wrong, tx.tx_data.updated_ts is set to true when a new transaction is created.
2. LdapEvent::InvalidData is not set when a request is not parsed correctly.
There is a missed case here indeed
- Subject changed from ldap: set invalid_data event to ldap: set invalid_data event in all cases
Also available in: Atom
PDF