diff options
author | Endi Sukma Dewata <edewata@redhat.com> | 2010-09-14 20:36:37 -0400 |
---|---|---|
committer | Nathan Kinder <nkinder@redhat.com> | 2010-09-17 12:14:05 -0700 |
commit | 019789548c33ec6155a0cbebfe81c58bac4d1ed8 (patch) | |
tree | b0f78f92e878fdf2c1a7b9c7e44231c56a2639ce /ldap/servers | |
parent | bef946f15f1e7f5694e8e044df3444d78b558d4f (diff) | |
download | ds-019789548c33ec6155a0cbebfe81c58bac4d1ed8.tar.gz ds-019789548c33ec6155a0cbebfe81c58bac4d1ed8.tar.xz ds-019789548c33ec6155a0cbebfe81c58bac4d1ed8.zip |
Bug 630092 - Coverity #15478: Resource leaks issues
https://bugzilla.redhat.com/show_bug.cgi?id=630092
Description:
The mep_pre_op() has been modified to release config_copy before it goes
out of scope by moving mep_free_config_entry() out of the switch statement.
Diffstat (limited to 'ldap/servers')
-rw-r--r-- | ldap/servers/plugins/mep/mep.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ldap/servers/plugins/mep/mep.c b/ldap/servers/plugins/mep/mep.c index c0ce0139..72af4288 100644 --- a/ldap/servers/plugins/mep/mep.c +++ b/ldap/servers/plugins/mep/mep.c @@ -1631,11 +1631,11 @@ mep_pre_op(Slapi_PBlock * pb, int modop) /* Dispose of the test entry */ slapi_entry_free(test_entry); - - /* Free the config copy */ - mep_free_config_entry(&config_copy); break; } + + /* Free the config copy */ + mep_free_config_entry(&config_copy); } mep_config_unlock(); |