summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEndi Sukma Dewata <edewata@redhat.com>2010-09-16 12:38:32 -0400
committerNathan Kinder <nkinder@redhat.com>2010-09-17 12:15:50 -0700
commitba741cade5aef1cbb8ede386b7f2b85d57745d75 (patch)
tree8638af65bc69396f4a7de65bcda259263f94f4c7
parent4078b6113628c0a842a6caf7c54535ca95cacfd7 (diff)
downloadds-ba741cade5aef1cbb8ede386b7f2b85d57745d75.tar.gz
ds-ba741cade5aef1cbb8ede386b7f2b85d57745d75.tar.xz
ds-ba741cade5aef1cbb8ede386b7f2b85d57745d75.zip
Bug 630092 - Coverity #15484: Resource leaks issues
https://bugzilla.redhat.com/show_bug.cgi?id=630092 Description: The bulk_import_queue() has been modified to release ep when an error occurs.
-rw-r--r--ldap/servers/slapd/back-ldbm/import-threads.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/ldap/servers/slapd/back-ldbm/import-threads.c b/ldap/servers/slapd/back-ldbm/import-threads.c
index 171be087..9629a9ef 100644
--- a/ldap/servers/slapd/back-ldbm/import-threads.c
+++ b/ldap/servers/slapd/back-ldbm/import-threads.c
@@ -2736,6 +2736,7 @@ static int bulk_import_queue(ImportJob *job, Slapi_Entry *entry)
ep = import_make_backentry(entry, id);
if ((ep == NULL) || (ep->ep_entry == NULL)) {
import_abort_all(job, 1);
+ backentry_free( &ep ); /* release the backend wrapper, here */
PR_Unlock(job->wire_lock);
return -1;
}