This is an automated email from the git hooks/post-receive script.
firstyear pushed a change to branch master in repository 389-ds-base.
from 00d04b0 Ticket 48118 - Add CI test case new 3aa02b6 Ticket 49508 - memory leak in cn=replica plugin setup
The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference.
Summary of changes: ldap/servers/plugins/replication/repl5_replica.c | 2 ++ 1 file changed, 2 insertions(+)
This is an automated email from the git hooks/post-receive script.
firstyear pushed a commit to branch master in repository 389-ds-base.
commit 3aa02b6981802dc6bacdd951bccc24ce55868c07 Author: William Brown firstyear@redhat.com Date: Tue Dec 19 16:10:44 2017 +0100
Ticket 49508 - memory leak in cn=replica plugin setup
Bug Description: during db2ldif a memory leak in cn=replica mt node setup was found.
Fix Description: Free the memory!
https://pagure.io/389-ds-base/issue/49508
Author: wibrown
Review by: mreynolds (Thank you!) --- ldap/servers/plugins/replication/repl5_replica.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/ldap/servers/plugins/replication/repl5_replica.c b/ldap/servers/plugins/replication/repl5_replica.c index db3a8a0..6ffaa19 100644 --- a/ldap/servers/plugins/replication/repl5_replica.c +++ b/ldap/servers/plugins/replication/repl5_replica.c @@ -1964,9 +1964,11 @@ _replica_init_from_config(Replica *r, Slapi_Entry *e, char *errortext) int64_t rflags; if((val = slapi_entry_attr_get_charptr(e, attr_flags))) { if (repl_config_valid_num(attr_flags, val, 0, 1, &rc, errortext, &rflags) != 0) { + slapi_ch_free_string(&val); return -1; } r->repl_flags = (uint32_t)rflags; + slapi_ch_free_string(&val); } else { r->repl_flags = 0; }
389-commits@lists.fedoraproject.org