diff options
| author | Sumit Bose <sbose@redhat.com> | 2013-02-27 12:12:45 +0100 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2013-02-28 12:27:51 +0100 |
| commit | caa218b6b81e8782534551a8936c704242a33160 (patch) | |
| tree | a7812e4f39f4135e82a503f4210906b394549630 /daemons/ipa-sam | |
| parent | db1a07b7f275b181f87496687593d3d2b7d013e8 (diff) | |
| download | freeipa-caa218b6b81e8782534551a8936c704242a33160.tar.gz freeipa-caa218b6b81e8782534551a8936c704242a33160.tar.xz freeipa-caa218b6b81e8782534551a8936c704242a33160.zip | |
ipa-sam: Array compared against 0 in ipasam_set_trusted_domain()
ipa_mspac_well_known_sids is a globally defined array so the check was
always true.
Fixes https://fedorahosted.org/freeipa/ticket/3423
Diffstat (limited to 'daemons/ipa-sam')
| -rw-r--r-- | daemons/ipa-sam/ipa_sam.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/daemons/ipa-sam/ipa_sam.c b/daemons/ipa-sam/ipa_sam.c index 0d4a27cf6..b9fc00c8d 100644 --- a/daemons/ipa-sam/ipa_sam.c +++ b/daemons/ipa-sam/ipa_sam.c @@ -2293,7 +2293,7 @@ static NTSTATUS ipasam_set_trusted_domain(struct pdb_methods *methods, &td->trust_forest_trust_info); } - for (i = 0; ipa_mspac_well_known_sids && ipa_mspac_well_known_sids[i]; i++) { + for (i = 0; ipa_mspac_well_known_sids[i]; i++) { smbldap_make_mod(priv2ld(ldap_state), entry, &mods, LDAP_ATTRIBUTE_SID_BLACKLIST_INCOMING, ipa_mspac_well_known_sids[i]); |