diff options
| author | Simo Sorce <ssorce@redhat.com> | 2012-04-17 15:35:59 -0400 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2012-04-17 15:40:58 -0400 |
| commit | adf16a9b1c52ea8ee1e9989b99ab7da32adddf38 (patch) | |
| tree | 9a9eb39995eac4ec8c9052169e778d838c7edabe /daemons/ipa-kdb/ipa_kdb_principals.c | |
| parent | 72efa64c81fc44dbc05c48730c339120888fecbe (diff) | |
| download | freeipa-adf16a9b1c52ea8ee1e9989b99ab7da32adddf38.tar.gz freeipa-adf16a9b1c52ea8ee1e9989b99ab7da32adddf38.tar.xz freeipa-adf16a9b1c52ea8ee1e9989b99ab7da32adddf38.zip | |
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 :)
Diffstat (limited to 'daemons/ipa-kdb/ipa_kdb_principals.c')
| -rw-r--r-- | daemons/ipa-kdb/ipa_kdb_principals.c | 1 |
1 files changed, 1 insertions, 0 deletions
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; |