Recent efforts to reduce string-copy ops while also avoiding using excessive memory
tripped a check on freeing the still-live variable. It is unclear why the variable
was set anyway, even though commented. The use was introduced between Exim 3.36 and 4.0
because otherwise the log can be confusing. */
if ( !sender_host_name
because otherwise the log can be confusing. */
if ( !sender_host_name
- && (deliver_domain = sender_helo_name, /* set $domain */
- match_isinlist(sender_helo_name, CUSS &helo_lookup_domains, 0,
- &domainlist_anchor, NULL, MCL_DOMAIN, TRUE, NULL)) == OK)
+ && match_isinlist(sender_helo_name, CUSS &helo_lookup_domains, 0,
+ &domainlist_anchor, NULL, MCL_DOMAIN, TRUE, NULL) == OK)
(void)host_name_lookup();
/* Rebuild the fullhost info to include the HELO name (and the real name
(void)host_name_lookup();
/* Rebuild the fullhost info to include the HELO name (and the real name