You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by Justin Mason <jm...@jmason.org> on 2009/09/02 19:03:33 UTC

bad sandbox rules on trunk

type "make" in an SVN checkout and this happens:

/usr/bin/perl build/mkrules --exit_on_no_src --src rulesrc --out rules --manifest MANIFEST --manifestskip MANIFEST.SKIP
rulesrc/10_force_active.cf: 0 active rules, 0 other
rulesrc/sandbox/axb/20_axb_misc.cf: 1 active rules, 0 other
rulesrc/sandbox/axb/20_axb_pdf.cf: 0 active rules, 2 other
rulesrc/sandbox/dos/20_uri.cf: 1 active rules, 2 other
rulesrc/sandbox/dos/70_bugs.cf: 0 active rules, 0 other
rulesrc/sandbox/dos/70_other.cf: 6 active rules, 117 other
rulesrc/sandbox/duncf/20_debt.cf: 2 active rules, 0 other
rulesrc/sandbox/duncf/20_header.cf: 1 active rules, 2 other
rulesrc/sandbox/emailed/00_FVGT_File001.cf: 181 active rules, 424 other
rulesrc/sandbox/emailed/99_alex_dev.cf: 4 active rules, 6 other
rulesrc/sandbox/emailed/99_alex_test.cf: 1 active rules, 0 other
rulesrc/sandbox/fanf/10_headers.cf: 1 active rules, 2 other
rulesrc/sandbox/fanf/20_uri_tests.cf: 0 active rules, 1 other
rulesrc/sandbox/fanf/30_text.cf: 3 active rules, 1 other
rulesrc/sandbox/felicity/70_dnswl.cf: 0 active rules, 4 other
rulesrc/sandbox/felicity/70_iadb.cf: 0 active rules, 27 other
rulesrc/sandbox/felicity/70_other.cf: 43 active rules, 60 other
rulesrc/sandbox/felicity/70_phishing.cf: 9 active rules, 47 other
rulesrc/sandbox/fredt/99_zFVGT_FakeReply.cf: 1 active rules, 70 other
rulesrc/sandbox/hege/20_hk.cf: 0 active rules, 19 other
rulesrc/sandbox/hege/20_hk_dev.cf: 0 active rules, 57 other
rulesrc/sandbox/hstern/20_body_tests.cf: 5 active rules, 3 other
rulesrc/sandbox/hstern/20_head_tests.cf: 2 active rules, 3 other
rulesrc/sandbox/hstern/20_meta_tests.cf: 1 active rules, 0 other
rulesrc/sandbox/hstern/20_uri_tests.cf: 3 active rules, 13 other
rulesrc/sandbox/hstern/70_syndicate.cf: 1 active rules, 15 other
rulesrc/sandbox/jhardin/20_MIME_no_text.cf: 0 active rules, 6 other
rulesrc/sandbox/jhardin/20_fillform.cf: 1 active rules, 9 other
rulesrc/sandbox/jhardin/20_lotsa_money.cf: 1 active rules, 17 other
rulesrc/sandbox/jhardin/20_misc_testing.cf: 0 active rules, 14 other
rulesrc/sandbox/jhardin/20_tbird_image_spam.cf: 0 active rules, 15 other
rulesrc/sandbox/jhardin/20_uri_obfu_ws.cf: 1 active rules, 1 other
rulesrc/sandbox/jm/20_basic.cf: 37 active rules, 80 other
rulesrc/sandbox/jm/20_bug_5920.cf: 0 active rules, 3 other
rulesrc/sandbox/jm/20_bug_5984.cf: 0 active rules, 2 other
rulesrc/sandbox/jm/20_bug_6000.cf: 0 active rules, 2 other
rulesrc/sandbox/jm/20_bug_6152.cf: 0 active rules, 1 other
rulesrc/sandbox/jm/20_dob.cf: 1 active rules, 0 other
rulesrc/sandbox/jm/20_games.cf: 0 active rules, 3 other
rulesrc/sandbox/jm/20_khop_sc_bug_6114.cf: 0 active rules, 10 other
rulesrc/sandbox/jm/20_sought.cf: 0 active rules, 355 other
rulesrc/sandbox/jm/20_sought2.cf: 0 active rules, 3 other
rulesrc/sandbox/jm/20_sought_fraud.cf: 0 active rules, 668 other
rulesrc/sandbox/jm/20_storm.cf: 0 active rules, 2 other
rulesrc/sandbox/jm/20_xmailer.cf: 43 active rules, 86 other
rulesrc/sandbox/jm/22_bug_5667.cf: 0 active rules, 0 other
rulesrc/sandbox/jm/70_tt_drugs.cf: 2 active rules, 10 other
rulesrc/sandbox/jm/80_sane.cf: 0 active rules, 150 other
rulesrc/sandbox/kb/20_bug_6080.cf: 0 active rules, 1 other
rulesrc/sandbox/kb/20_bug_6156.cf: 0 active rules, 1 other
rulesrc/sandbox/kb/20_header.cf: 0 active rules, 5 other
rulesrc/sandbox/kb/70_misc.cf: 0 active rules, 23 other
rulesrc/sandbox/kmcgrail/20_test.cf: 0 active rules, 43 other
rulesrc/sandbox/kmcgrail/70_mx.cf: 0 active rules, 4 other
rulesrc/sandbox/kmcgrail/70_phishing.cf: 0 active rules, 1 other
rulesrc/sandbox/kmcgrail/80_deadrules.cf: 6 active rules, 0 other
rulesrc/sandbox/maddoc/99_doc_test.cf: 1 active rules, 47 other
rulesrc/sandbox/mkettler/20_drugs.cf: 1 active rules, 1 other
rulesrc/sandbox/mkettler/25_uribl.cf: 0 active rules, 5 other
rulesrc/sandbox/mmartinec/20_misc.cf: 0 active rules, 14 other
rulesrc/sandbox/mmartinec/20_rpvalid.cf: 0 active rules, 1 other
rulesrc/sandbox/mmartinec/60_whitelist.cf: 1 active rules, 0 other
rulesrc/sandbox/sidney/70_other.cf: 0 active rules, 0 other
lint: config: failed to parse line, skipping, in "rules/70_sandbox.cf": mimeheader __JPEG_ATTACH           Content-Type =~ /image\/jpeg/i at build/mkrules line 253.
rules: failed to run __ONE_IMG test, skipping:
        (Can't locate object method "image_count" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 44.
)
rules: failed to run __IMG_LE_300K test, skipping:
        (Can't locate object method "pixel_coverage" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 64.
)

ERROR: LINT FAILED, suppressing output: rules/70_sandbox.cf

lint: config: invalid regexp for rule redirector_pattern: /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i: missing or invalid delimiters
lint: config: SpamAssassin failed to parse line, "/^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i" is not valid for "redirector_pattern", skipping: redirector_pattern      /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i at build/mkrules line 253.

ERROR: LINT FAILED, suppressing output: rules/72_active.cf

rules/70_sandbox.cf: no rules promoted
rules/72_active.cf: no rules promoted



This is why rules have been missing from the rule-qa output.
Kind of bad timing given that we were about to cut the mass-check tarball.... ;)

--j.

Re: bad sandbox rules on trunk

Posted by Justin Mason <jm...@jmason.org>.
btw, Hudson is supposed to trigger a BUILD FAILED mail when this
happens, but naturally _that_ seems to be broken too. ;)

--j.

On Wed, Sep 2, 2009 at 20:04, John Hardin<jh...@impsec.org> wrote:
> On Wed, 2 Sep 2009, Justin Mason wrote:
>
>> lint: config: invalid regexp for rule redirector_pattern:
>> /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern
>> m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i: missing or invalid
>> delimiters
>> lint: config: SpamAssassin failed to parse line,
>> "/^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern
>> m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i" is not valid for
>> "redirector_pattern", skipping: redirector_pattern
>>  /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern
>> m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i at build/mkrules line
>> 253.
>>
>> ERROR: LINT FAILED, suppressing output: rules/72_active.cf
>
> Fixed. Missing a newline at EOF.
>
> --
>  John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
>  jhardin@impsec.org    FALaholic #11174     pgpk -a jhardin@impsec.org
>  key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
> -----------------------------------------------------------------------
>  You can't reason a person out of a position if he didn't use
>  reason to get there in the first place.   -- Kristopher, at Marko's
> -----------------------------------------------------------------------
>  15 days until the 222nd anniversary of the signing of the U.S. Constitution
>
>



-- 
--j.

Re: bad sandbox rules on trunk

Posted by John Hardin <jh...@impsec.org>.
On Wed, 2 Sep 2009, Justin Mason wrote:

> lint: config: invalid regexp for rule redirector_pattern: /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i: missing or invalid delimiters
> lint: config: SpamAssassin failed to parse line, "/^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i" is not valid for "redirector_pattern", skipping: redirector_pattern      /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i at build/mkrules line 253.
>
> ERROR: LINT FAILED, suppressing output: rules/72_active.cf

Fixed. Missing a newline at EOF.

-- 
  John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
  jhardin@impsec.org    FALaholic #11174     pgpk -a jhardin@impsec.org
  key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
   You can't reason a person out of a position if he didn't use
   reason to get there in the first place.   -- Kristopher, at Marko's
-----------------------------------------------------------------------
  15 days until the 222nd anniversary of the signing of the U.S. Constitution

Re: bad sandbox rules on trunk

Posted by "Daryl C. W. O'Shea" <sp...@dostech.ca>.
Ah, thanks!  That's why I couldn't get score generation working again
last night.

On 02/09/2009 1:03 PM, Justin Mason wrote:
> type "make" in an SVN checkout and this happens:
> 
> /usr/bin/perl build/mkrules --exit_on_no_src --src rulesrc --out rules --manifest MANIFEST --manifestskip MANIFEST.SKIP
> rulesrc/10_force_active.cf: 0 active rules, 0 other
> rulesrc/sandbox/axb/20_axb_misc.cf: 1 active rules, 0 other
> rulesrc/sandbox/axb/20_axb_pdf.cf: 0 active rules, 2 other
> rulesrc/sandbox/dos/20_uri.cf: 1 active rules, 2 other
> rulesrc/sandbox/dos/70_bugs.cf: 0 active rules, 0 other
> rulesrc/sandbox/dos/70_other.cf: 6 active rules, 117 other
> rulesrc/sandbox/duncf/20_debt.cf: 2 active rules, 0 other
> rulesrc/sandbox/duncf/20_header.cf: 1 active rules, 2 other
> rulesrc/sandbox/emailed/00_FVGT_File001.cf: 181 active rules, 424 other
> rulesrc/sandbox/emailed/99_alex_dev.cf: 4 active rules, 6 other
> rulesrc/sandbox/emailed/99_alex_test.cf: 1 active rules, 0 other
> rulesrc/sandbox/fanf/10_headers.cf: 1 active rules, 2 other
> rulesrc/sandbox/fanf/20_uri_tests.cf: 0 active rules, 1 other
> rulesrc/sandbox/fanf/30_text.cf: 3 active rules, 1 other
> rulesrc/sandbox/felicity/70_dnswl.cf: 0 active rules, 4 other
> rulesrc/sandbox/felicity/70_iadb.cf: 0 active rules, 27 other
> rulesrc/sandbox/felicity/70_other.cf: 43 active rules, 60 other
> rulesrc/sandbox/felicity/70_phishing.cf: 9 active rules, 47 other
> rulesrc/sandbox/fredt/99_zFVGT_FakeReply.cf: 1 active rules, 70 other
> rulesrc/sandbox/hege/20_hk.cf: 0 active rules, 19 other
> rulesrc/sandbox/hege/20_hk_dev.cf: 0 active rules, 57 other
> rulesrc/sandbox/hstern/20_body_tests.cf: 5 active rules, 3 other
> rulesrc/sandbox/hstern/20_head_tests.cf: 2 active rules, 3 other
> rulesrc/sandbox/hstern/20_meta_tests.cf: 1 active rules, 0 other
> rulesrc/sandbox/hstern/20_uri_tests.cf: 3 active rules, 13 other
> rulesrc/sandbox/hstern/70_syndicate.cf: 1 active rules, 15 other
> rulesrc/sandbox/jhardin/20_MIME_no_text.cf: 0 active rules, 6 other
> rulesrc/sandbox/jhardin/20_fillform.cf: 1 active rules, 9 other
> rulesrc/sandbox/jhardin/20_lotsa_money.cf: 1 active rules, 17 other
> rulesrc/sandbox/jhardin/20_misc_testing.cf: 0 active rules, 14 other
> rulesrc/sandbox/jhardin/20_tbird_image_spam.cf: 0 active rules, 15 other
> rulesrc/sandbox/jhardin/20_uri_obfu_ws.cf: 1 active rules, 1 other
> rulesrc/sandbox/jm/20_basic.cf: 37 active rules, 80 other
> rulesrc/sandbox/jm/20_bug_5920.cf: 0 active rules, 3 other
> rulesrc/sandbox/jm/20_bug_5984.cf: 0 active rules, 2 other
> rulesrc/sandbox/jm/20_bug_6000.cf: 0 active rules, 2 other
> rulesrc/sandbox/jm/20_bug_6152.cf: 0 active rules, 1 other
> rulesrc/sandbox/jm/20_dob.cf: 1 active rules, 0 other
> rulesrc/sandbox/jm/20_games.cf: 0 active rules, 3 other
> rulesrc/sandbox/jm/20_khop_sc_bug_6114.cf: 0 active rules, 10 other
> rulesrc/sandbox/jm/20_sought.cf: 0 active rules, 355 other
> rulesrc/sandbox/jm/20_sought2.cf: 0 active rules, 3 other
> rulesrc/sandbox/jm/20_sought_fraud.cf: 0 active rules, 668 other
> rulesrc/sandbox/jm/20_storm.cf: 0 active rules, 2 other
> rulesrc/sandbox/jm/20_xmailer.cf: 43 active rules, 86 other
> rulesrc/sandbox/jm/22_bug_5667.cf: 0 active rules, 0 other
> rulesrc/sandbox/jm/70_tt_drugs.cf: 2 active rules, 10 other
> rulesrc/sandbox/jm/80_sane.cf: 0 active rules, 150 other
> rulesrc/sandbox/kb/20_bug_6080.cf: 0 active rules, 1 other
> rulesrc/sandbox/kb/20_bug_6156.cf: 0 active rules, 1 other
> rulesrc/sandbox/kb/20_header.cf: 0 active rules, 5 other
> rulesrc/sandbox/kb/70_misc.cf: 0 active rules, 23 other
> rulesrc/sandbox/kmcgrail/20_test.cf: 0 active rules, 43 other
> rulesrc/sandbox/kmcgrail/70_mx.cf: 0 active rules, 4 other
> rulesrc/sandbox/kmcgrail/70_phishing.cf: 0 active rules, 1 other
> rulesrc/sandbox/kmcgrail/80_deadrules.cf: 6 active rules, 0 other
> rulesrc/sandbox/maddoc/99_doc_test.cf: 1 active rules, 47 other
> rulesrc/sandbox/mkettler/20_drugs.cf: 1 active rules, 1 other
> rulesrc/sandbox/mkettler/25_uribl.cf: 0 active rules, 5 other
> rulesrc/sandbox/mmartinec/20_misc.cf: 0 active rules, 14 other
> rulesrc/sandbox/mmartinec/20_rpvalid.cf: 0 active rules, 1 other
> rulesrc/sandbox/mmartinec/60_whitelist.cf: 1 active rules, 0 other
> rulesrc/sandbox/sidney/70_other.cf: 0 active rules, 0 other
> lint: config: failed to parse line, skipping, in "rules/70_sandbox.cf": mimeheader __JPEG_ATTACH           Content-Type =~ /image\/jpeg/i at build/mkrules line 253.
> rules: failed to run __ONE_IMG test, skipping:
>         (Can't locate object method "image_count" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 44.
> )
> rules: failed to run __IMG_LE_300K test, skipping:
>         (Can't locate object method "pixel_coverage" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 64.
> )
> 
> ERROR: LINT FAILED, suppressing output: rules/70_sandbox.cf
> 
> lint: config: invalid regexp for rule redirector_pattern: /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i: missing or invalid delimiters
> lint: config: SpamAssassin failed to parse line, "/^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i" is not valid for "redirector_pattern", skipping: redirector_pattern      /^http\:\/\/www\.att\.net\/s\/context\.dll\?redirecturl\=(.*)$/iredirector_pattern m'/(?:index.php)?\?.*(?<=[?&])URL=(.*?)(?:$|[&#])'i at build/mkrules line 253.
> 
> ERROR: LINT FAILED, suppressing output: rules/72_active.cf
> 
> rules/70_sandbox.cf: no rules promoted
> rules/72_active.cf: no rules promoted
> 
> 
> 
> This is why rules have been missing from the rule-qa output.
> Kind of bad timing given that we were about to cut the mass-check tarball.... ;)
> 
> --j.



Re: bad sandbox rules on trunk

Posted by John Hardin <jh...@impsec.org>.
On Wed, 2 Sep 2009, Justin Mason wrote:

> lint: config: failed to parse line, skipping, in "rules/70_sandbox.cf": mimeheader __JPEG_ATTACH           Content-Type =~ /image\/jpeg/i at build/mkrules line 253.
> rules: failed to run __ONE_IMG test, skipping:
>        (Can't locate object method "image_count" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 44.
> )
> rules: failed to run __IMG_LE_300K test, skipping:
>        (Can't locate object method "pixel_coverage" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 64.
> )
>
> ERROR: LINT FAILED, suppressing output: rules/70_sandbox.cf

So why aren't the ImageInfo and MIMEHeader plugins available to lint at 
this point?

-- 
  John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
  jhardin@impsec.org    FALaholic #11174     pgpk -a jhardin@impsec.org
  key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
   ...every time I sit down in front of a Windows machine I feel as
   if the computer is just a place for the manufacturers to put their
   advertising.                                 -- fwadling on Y! SCOX
-----------------------------------------------------------------------
  15 days until the 222nd anniversary of the signing of the U.S. Constitution

Re: bad sandbox rules on trunk

Posted by John Hardin <jh...@impsec.org>.
On Wed, 2 Sep 2009, Justin Mason wrote:

> type "make" in an SVN checkout and this happens:
>
> rulesrc/sandbox/jhardin/20_tbird_image_spam.cf: 0 active rules, 15 other

> lint: config: failed to parse line, skipping, in "rules/70_sandbox.cf": mimeheader __JPEG_ATTACH           Content-Type =~ /image\/jpeg/i at build/mkrules line 253.
> rules: failed to run __ONE_IMG test, skipping:
>        (Can't locate object method "image_count" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 44.
> )
> rules: failed to run __IMG_LE_300K test, skipping:
>        (Can't locate object method "pixel_coverage" via package "Mail::SpamAssassin::PerMsgStatus" at (eval 363) line 64.
> )
>
> ERROR: LINT FAILED, suppressing output: rules/70_sandbox.cf

Owtch! That's me, in the tbird image spam rule. image_count and 
pixel-coverage _were_ working. What happened to them?

> This is why rules have been missing from the rule-qa output.
> Kind of bad timing given that we were about to cut the mass-check tarball.... ;)

I'll pull that out of the tbird rule right now.

-- 
  John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
  jhardin@impsec.org    FALaholic #11174     pgpk -a jhardin@impsec.org
  key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
   Ignorance is no excuse for a law.
-----------------------------------------------------------------------
  15 days until the 222nd anniversary of the signing of the U.S. Constitution