http://bugzilla.spamassassin.org/show_bug.cgi?id=3997





------- Additional Comments From [EMAIL PROTECTED]  2005-01-22 19:49 -------
Well, here's what I propose. Vance, you might want to see what Daniel's opinion
of this is before you try it, as I am not in a position right now to test any
code I type here.

In Daniel's patch, change the line

  next if ($rr->type eq 'A' && $rr->rdatastr !~ /^127\./);

to

  if ($rr->type eq 'A' && $rr->rdatastr !~ /^127\./) {
    dbg ("URIDNSBL: 1 bogus rr " . $rr->string);
    next;
  }

and similary change

  next if ($rr->type eq 'A' && $rr->rdatastr !~ /^127\./ &&
           !($uridnsbl_subs_bits & 0xff000000));

to

  if ($rr->type eq 'A' && $rr->rdatastr !~ /^127\./ &&
      !($uridnsbl_subs_bits & 0xff000000)) {
    dbg ("URIDNSBL: 2 bogus rr " . $rr->string);
    next;
  }

With the patch you should not get any more FPs, but the debug lines might get
logged a lot more often than you had been getting FPs. I suspect that you are
getting a lot of incorrect DNS results in the tests, but most don't happen to
generate an FP.

Of course you have to run with the debug option to see these log messages.




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

Reply via email to