From adf16a9b1c52ea8ee1e9989b99ab7da32adddf38 Mon Sep 17 00:00:00 2001
From: Simo Sorce <ssorce@redhat.com>
Date: Tue, 17 Apr 2012 15:35:59 -0400
Subject: Fix theoretical leak discovered by coverity

This was introduced when we started checking the return from
ipadb_get_context() to silence another coverity report.
That condition can never be true in this function but whatever ... let's
silence Coverity once again :)
---
 daemons/ipa-kdb/ipa_kdb_principals.c | 1 +
 1 file changed, 1 insertion(+)

(limited to 'daemons/ipa-kdb')

diff --git a/daemons/ipa-kdb/ipa_kdb_principals.c b/daemons/ipa-kdb/ipa_kdb_principals.c
index 143261982..139741291 100644
--- a/daemons/ipa-kdb/ipa_kdb_principals.c
+++ b/daemons/ipa-kdb/ipa_kdb_principals.c
@@ -374,6 +374,7 @@ static krb5_error_code ipadb_parse_ldap_entry(krb5_context kcontext,
      * krb5_db_entry in kdb.h */
     ipactx = ipadb_get_context(kcontext);
     if (!ipactx) {
+        free(entry);
         return KRB5_KDB_DBNOTINITED;
     }
     lcontext = ipactx->lcontext;
-- 
cgit