diff options
| author | Sumit Bose <sbose@redhat.com> | 2013-02-22 13:30:30 +0100 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2013-02-28 12:31:02 +0100 |
| commit | 29ddcf3bcb0596fd60cbc090d3f70a2f20d70c72 (patch) | |
| tree | bdb5103f0fd792208859ac526d6372f742085511 /daemons/ipa-kdb/ipa_kdb_mspac.c | |
| parent | caa218b6b81e8782534551a8936c704242a33160 (diff) | |
| download | freeipa-29ddcf3bcb0596fd60cbc090d3f70a2f20d70c72.tar.gz freeipa-29ddcf3bcb0596fd60cbc090d3f70a2f20d70c72.tar.xz freeipa-29ddcf3bcb0596fd60cbc090d3f70a2f20d70c72.zip | |
ipa-kdb: Dereference after null check in ipa_kdb_mspac.c
A wrong logic was used to check ipactx.
Fixes https://fedorahosted.org/freeipa/ticket/3424
Diffstat (limited to 'daemons/ipa-kdb/ipa_kdb_mspac.c')
| -rw-r--r-- | daemons/ipa-kdb/ipa_kdb_mspac.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/daemons/ipa-kdb/ipa_kdb_mspac.c b/daemons/ipa-kdb/ipa_kdb_mspac.c index aeadc6d75..5071348d9 100644 --- a/daemons/ipa-kdb/ipa_kdb_mspac.c +++ b/daemons/ipa-kdb/ipa_kdb_mspac.c @@ -1231,7 +1231,7 @@ static krb5_error_code filter_logon_info(krb5_context context, * */ if (info->info->info3.sidcount != 0) { ipactx = ipadb_get_context(context); - if (!ipactx && !ipactx->mspac) { + if (!ipactx || !ipactx->mspac) { return KRB5_KDB_DBNOTINITED; } count = info->info->info3.sidcount; |