Re: [bug-mailutils] Bug in sieve engine

[Kostas Zorbadelos]

On Tue, Nov 01, 2005 at 11:06:51AM +0200, Sergey Poznyakoff wrote:
> Kostas Zorbadelos <address@hidden> wrote:
> 
> > My program
> > opens a mailbox and for each message it creates a new sieve machine,
> 
> Wow, that's terribly ineffective! Why don't you use a single instance?
>

I know, but the real case that showed the bug, is an MDA that speaks
LMTP. In this case, if the LMTP connection is re-used and a delivery
is attempted to the same user, the error appears...
(I have a sieve_test() function in my MDA similar to the mailutils'
MDA)  
 
> > char *sieve_filename;  
> > int sieve_line_num;  
> > ino_t sieve_source_inode;
> > 
> > in sieve.l that are not initialized.
> 
> They are in BSS and therefore are initialized to zero values.
> 

BSS(?) 
Therefore you believe that if I initialize the variables in sieve.l,
the issue won't be solved?
My trace with gdb shows that these variables are not initialized and
keep the garbage from the previous pass... 

> > I include my program, the makefile, the sieve file and the test
> > mailbox. All tests are against snapshot 20050929 (with a patch of mine
> > that has to do with the reject action, already sent to the list).
> > Let me know if you need anything else to trace the problem.
> 
> Thank you, I will investigate it. By the way, your patch (in a slightly
> modified way), have already been applied to the CVS HEAD.
> 

Thanks.

> Regards,
> Sergey
> 

-- 
  Kostas Zorbadelos
  Systems Designer/Developer, Otenet SA 
  address@hidden contact: kzorba (at) otenet.gr
  
  Out there in the darkness, out there in the night
  out there in the starlight, one soul burns brighter
  than a thousand suns.