mbox failure in 0.14.5

[David Relson]

On Mon, 18 Aug 2003 18:58:46 +0200
Boris 'pi' Piwinger <3.14 at logic.univie.ac.at> wrote:

> Matthias Andree <matthias.andree at gmx.de> wrote:
> 
> >Don't expect this to be fixed before bogofilter has a clean layer
> >separation between splitting mboxes, decoding things and the lexer.
> 
> OK, so this shows the importance of this layer
> implementation.
> 
> pi

pi,

The problem was caused by a parsing change (in lexer_v3.l).  As far as I could tell, the test of global variable 'msg_header' was unnecessary.  So I removed it.  We now have evidence that the test _was_ necessary.

The patch below has been committed to CVS and will be in the next build.

David

diff -u -r1.49 lexer_v3.l
--- lexer_v3.l	14 Aug 2003 12:33:55 -0000	1.49
+++ lexer_v3.l	18 Aug 2003 20:59:50 -0000
@@ -165,7 +165,7 @@
 
 %%
 
-^From\ 						{ if (is_from((const unsigned char *)yytext, yyleng)) { 
+^From\ 						{ if (msg_header && is_from((const unsigned char *)yytext, yyleng)) { 
 						      BEGIN(INITIAL); 
 						      return FROM;
 						  } else {