http://bugzilla.spamassassin.org/show_bug.cgi?id=3794
------- Additional Comments From [EMAIL PROTECTED] 2004-09-20 11:31 -------
> it kills further scanning of *other* messages, or just that
> one message?
In case of amavisd-new, it kills just that one message, which stays in the MTA
queue (after a 4xx temporary failure) and delivery is re-attempted every so
often for a couple of days, leaving a pile of intentionally preserved temporary
files behind.
> it should be in an eval { } block anyway.
Perhaps, although I think a cleaner solution would be two-part:
- don't assemble illegal DNS queries in the first place
(in this particular example one 'node label' in DNS parlance
exceeded 63 characters, and the complete query should
not be too long - rfc1034 mentions 255)
- when analyzing the Net::DNS::Packet in process_dnsbl_result,
allow for its fields to be empty (the $packet->question in this case),
i.e. ignore the result, if does not appear valid.
I think an eval {} could be used an additional safety net,
but the known/expected cases should be handled explicitly.
Mark
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
