From 272837f1c07729392cdbc88b99a221390d01e70d Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Thu, 7 Feb 2019 12:11:42 +0100 Subject: [PATCH] Remove ZERO_STRUCT() call ipa_sam uses Samba's macro ZERO_STRUCT() to safely zero out a block in memory. On F30 ZERO_STRUCT() is currently broken, because it uses the undefined C11 function memset_s(). During investigation of the bug, it turned out that ZERO_STRUCT(td->security_identifier) is not needed. The whole td struct is allocated with talloc_zero(), so td->security_identifier is already zeroed. See: https://bugzilla.redhat.com/show_bug.cgi?id=1672231 Signed-off-by: Christian Heimes Reviewed-By: Alexander Bokovoy --- daemons/ipa-sam/ipa_sam.c | 1 - 1 file changed, 1 deletion(-) diff --git a/daemons/ipa-sam/ipa_sam.c b/daemons/ipa-sam/ipa_sam.c index 675a511f0..b1e0294e3 100644 --- a/daemons/ipa-sam/ipa_sam.c +++ b/daemons/ipa-sam/ipa_sam.c @@ -2179,7 +2179,6 @@ static bool fill_pdb_trusted_domain(TALLOC_CTX *mem_ctx, if (dummy == NULL) { DEBUG(9, ("Attribute %s not present.\n", LDAP_ATTRIBUTE_TRUST_SID)); - ZERO_STRUCT(td->security_identifier); } else { err = sss_idmap_sid_to_smb_sid(ipasam_state->idmap_ctx, dummy, &sid);