Bug in opendmarc_get_policy_to_enforce (also: can't create a sourceforge ticket...)

[Roland Turner]

(in 0.1.0 and 0.1.2)

OPENDMARC_STATUS_T
opendmarc_get_policy_to_enforce(DMARC_POLICY_T *pctx)
{
u_char rev_from_domain[MAXDNSHOSTNAME];
u_char rev_spf_domain[MAXDNSHOSTNAME];
u_char rev_dkim_domain[MAXDNSHOSTNAME];

if (pctx == NULL)
return DMARC_PARSE_ERROR_NULL_CTX;

if (pctx->from_domain == NULL)
return DMARC_POLICY_ABSENT;

if (pctx->from_domain == NULL)
return DMARC_FROM_DOMAIN_ABSENT;
(void) opendmarc_reverse_domain(pctx->from_domain, rev_from_domain, sizeof rev_from_domain);

The first "if (pctx->from_domain == NULL)" is presumably supposed to be "if (pctx->p == NULL)"

[Murray S. Kucherawy]

Not exactly, because "p" is an int. But it does seem incorrect.

Bryan?