diff options
author | Rich Megginson <rmeggins@redhat.com> | 2006-10-12 21:21:10 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2006-10-12 21:21:10 +0000 |
commit | 131b7fe13af04a0dc13bb63591dc589adb5d3230 (patch) | |
tree | 749b480fa0b18d6a958f9fd0d35b0bfc885930b4 /ldap | |
parent | b4b87efa99f5dff4907c15fbb93c726f24f91ec3 (diff) | |
download | ds-131b7fe13af04a0dc13bb63591dc589adb5d3230.tar.gz ds-131b7fe13af04a0dc13bb63591dc589adb5d3230.tar.xz ds-131b7fe13af04a0dc13bb63591dc589adb5d3230.zip |
Bug(s) fixed: 183903
Bug Description: Memory leak in ldbm_config.c:replace_ldbm_config_value
Reviewed by: nhosoi (Thanks!)
Fix Description: Just needed to call slapi_mods_done(&smods) after the call to slapi_modify_internal_pb(). This is the same as in the other places in the server that perform an internal modify operation.
Platforms tested: RHEL4
Flag Day: no
Doc impact: no
Diffstat (limited to 'ldap')
-rw-r--r-- | ldap/servers/slapd/back-ldbm/ldbm_config.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ldap/servers/slapd/back-ldbm/ldbm_config.c b/ldap/servers/slapd/back-ldbm/ldbm_config.c index 169d8e29..9021dd7f 100644 --- a/ldap/servers/slapd/back-ldbm/ldbm_config.c +++ b/ldap/servers/slapd/back-ldbm/ldbm_config.c @@ -1760,5 +1760,6 @@ void replace_ldbm_config_value(char *conftype, char *val, struct ldbminfo *li) slapi_mods_get_ldapmods_byref(&smods), NULL, NULL, li->li_identity, 0); slapi_modify_internal_pb(&pb); + slapi_mods_done(&smods); pblock_done(&pb); } |