[bug-mailutils] Sieve vacation action bug(s) in 20050906 snapshot

[Kostas Zorbadelos]

Well,
I am back with 2 issues in vacation action. I include the (familiar)
program to reproduce them, a test mbox, as well as gdb backtraces. The
program is called as 

$testsieve mbox:test.mbox filters.sieve

- The first has to do with the fact that temporary files created by
  the vacation action in /tmp (files of the form /tmp/mu*) are not
  unlinked. These files contain the message text sent in the body
  of the produced mail message from MAILER-DAEMON (the argument to the
  action). The relevant gdb backtrace is shown below.    

---------- The temporary file that is not deleted -----------------------------
#0  mu_body_get_stream (body=0x8055f88, pstream=0xbfffdc08) at body.c:159
#1  0xb7dd5423 in build_mime (pmime=0xbfffdc68, msg=0x8054010,
    text=0x804ff60 "I am on vacation. Yahoo!! \305\353\353\347\355\351\352\334 
\352\341\351 \363\364\357 body...") at vacation.c:\
66
#2  0xb7dd5c38 in vacation_reply (mach=0x804b0f8, tags=0x8050098, msg=0x8054010,
    text=0x804ff60 "I am on vacation. Yahoo!! \305\353\353\347\355\351\352\334 
\352\341\351 \363\364\357 body...", to=0x8054738 "\
address@hidden",
    from=0x80546c0 "address@hidden") at vacation.c:435
#3  0xb7dd609f in sieve_action_vacation (mach=0x804b0f8, args=0x80504e8, 
tags=0x8050098) at vacation.c:539
#4  0xb7f7b0fa in instr_run (mach=0x804b0f8) at runtime.c:85
#5  0xb7f7b18f in instr_action (mach=0x804b0f8) at runtime.c:98
#6  0xb7f7b7c1 in sieve_run (mach=0x804b0f8) at runtime.c:283
#7  0xb7f7ba84 in mu_sieve_message (mach=0x804b0f8, msg=0x8054010) at 
runtime.c:362
#8  0x0804902e in main (argc=3, argv=0xbfffde44) at testsieve.c:104
-------------------------------------------------------------------------------

As I can see the mu_body_get_stream creates the temporary file with a
call to mu_tempname() but this is never unlinked. I cannot tell with
certainty where it would be best to unlink the file, so I leave it up
to you to decide -:)

In earlier snapshots, I also experienced crashes in the vacation
action. I will stress test this snapshot to see if it exibits the same
behaviour.  

- The second issue has to do with sending Greek text in the subject of
  the vacation action. The subject becomes corrupted and is shown as 
  =?iso-8859-1?Q?<the greek text here>?= in mail clients (mutt and
  gmail tested). Below is the backtrace

-------------------------- Greek subject -----------------------------------
#0  mu_rfc2047_encode (charset=0xb7dd9277 "iso-8859-1", encoding=0xb7dd92da 
"quoted-printable",
    text=0x804fef8 "\304\351\341\352\357\360\335\362 \341\353\334 
\305\353\353\347\355\351\352\334...", result=0xbfffdc0c) at rfc\
2047.c:233
#1  0xb7dd5b86 in vacation_subject (mach=0x804b0f8, tags=0x8050098, 
msg=0x8054010, newhdr=0x80551d0) at vacation.c:409
#2  0xb7dd5d0f in vacation_reply (mach=0x804b0f8, tags=0x8050098, msg=0x8054010,
    text=0x804ff60 "I am on vacation. Yahoo!! \305\353\353\347\355\351\352\334 
\352\341\351 \363\364\357 body...", to=0x8054738 "\
address@hidden",
    from=0x80546c0 "address@hidden") at vacation.c:450
#3  0xb7dd609f in sieve_action_vacation (mach=0x804b0f8, args=0x80504e8, 
tags=0x8050098) at vacation.c:539
#4  0xb7f7b0fa in instr_run (mach=0x804b0f8) at runtime.c:85
#5  0xb7f7b18f in instr_action (mach=0x804b0f8) at runtime.c:98
#6  0xb7f7b7c1 in sieve_run (mach=0x804b0f8) at runtime.c:283
#7  0xb7f7ba84 in mu_sieve_message (mach=0x804b0f8, msg=0x8054010) at 
runtime.c:362
#8  0x0804902e in main (argc=3, argv=0xbfffde44) at testsieve.c:104
----------------------------------------------------------------------------

I can see that mu_rfc2047_encode is called with iso-8859-1 argument
and this creates the string in the subject. Any suggestions as to what
can fix the situation?
     
Thanks,
Kostas

 
-- 
  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.

testsieve.c
Description: Text Data

test.mbox
Description: Text document

filters.sieve
Description: Text document