https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7869

--- Comment #3 from John Hardin <[email protected]> ---
(In reply to RW from comment #2)

> what may cause some confusion is that check_fromname_spoof() doesn't work
> without a To header. This seems like an unnecessary condition.

The plugin also checks for From name == To name, but it has this fairly early
in the core _check_fromnamespoof():

  my @toaddrs = $pms->all_to_addrs();
  return 0 unless @toaddrs;

Perhaps that is a bit too aggressive of a shortcut? It seems the to-address
cleanups and checks could be put into an if (@toaddrs) {} block instead so that
they don't skip the from-address checks in the absence of a to address.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to