I can't see a mistake.
PTROK_Run returns 1 but has added the score before. The message score
check is done after each chech has returned OK.
This is the same for every check.
if (! PTROK($fh)) {
$reply=$SenderInvalidError;
my $slok=$this->{allLovePTRSpam}==1;
$reply =~ s/REASON/$this->{messagereason}/go;
thisIsSpam($fh,"$this->{messagereason}",$spamPTRLog,$reply,$ptrTestMode,$slok,0);
if ($this->{error}) {$this->{skipnotspam} = 0;return;}
}
if (&MsgScoreTooHigh($fh,$done)) {$this->{skipnotspam} =
0;return;}
Thomas
Von: Michael Thomas <m...@mathbox.com>
An: ASSP development mailing list <assp-test@lists.sourceforge.net>,
Datum: 31.08.2013 03:58
Betreff: [Assp-test] $DoInvalidPTR==3 check in PTROK_Run
Thomas,
In sub PTROK_Run
return 1 if $DoInvalidPTR==3 || $PTRs{sum} < ${'ptiValencePB'}[0];
$DoInvalidPTR==3 is scoring, so should not return 1, unless you want to
check MessageScore value...
I have several examples of messages that stop processing, but do not
block, because PTROK_Run returned 1, so on return bailed out of checking
other tests. Also, because PTROK_Run is performed earlier than other
tests, did not have sufficient score to block, but would have if
checking had continued.
--
Michael Thomas
Mathbox
978-687-3300
Toll Free: 1-877-MATHBOX (1-877-628-4269)
------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft
technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
Assp-test mailing list
Assp-test@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/assp-test
DISCLAIMER:
*******************************************************
This email and any files transmitted with it may be confidential, legally
privileged and protected in law and are intended solely for the use of the
individual to whom it is addressed.
This email was multiple times scanned for viruses. There should be no
known virus in this email!
*******************************************************
------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
Assp-test mailing list
Assp-test@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/assp-test
