Bug: When parsing the provided attribute value, a reference was used and modified, the original attribute was corrupted
Fix: Use a copy for parsing
Reviewed by: ?
LGTM. Ack Have not found other place where slapi_entry_attr_get_ref is used for a list of values.
Should we also modify it here? https://pagure.io/389-ds-base/blob/master/f/ldap/servers/plugins/replication/repl5_agmt.c#_533
rebased onto 9e88e197f20a08c7ce21d209d1f6f19e6c229709
yes, good catch
LGTM too!
Pull-Request has been merged by lkrispen
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This pull request has been cloned to Github as issue and is available here: - https://github.com/389ds/389-ds-base/issues/3653
If you want to continue to work on the PR, please navigate to the github issue, download the patch from the attachments and file a new pull request.
Thank you for understanding. We apologize for all inconvenience.
Pull-Request has been closed by spichugi
Bug: When parsing the provided attribute value, a reference was used
and modified, the original attribute was corrupted
Fix: Use a copy for parsing
Reviewed by: ?