Re: Sieve redirect works but vacation doesn't

[Gary Mills]

Well, I found a bug in sieve/script.c that seems related to my
problem.  The patch below fixes it.  The bug is that the snprintf()
will never be executed.  When I tested the new lmtpd, I expected
to see either "Sent vacation reply\n" or "Vacation reply suppressed\n",
but neither of those messages appeared in any log.  Where are they
supposed to go?  The patch is for cyrus-imapd-2.0.7.

================================================================
*** script.Oc   Mon Sep  4 23:15:35 2000
--- script.c    Thu Nov 16 20:45:43 2000
***************
*** 929,938 ****
                                 "Sent vacation reply\n");
  
                } else if (ret == SIEVE_DONE) {
!                   if (ret == SIEVE_OK)
!                       snprintf(actions_string+strlen(actions_string),
!                                sizeof(actions_string)-strlen(actions_string),
!                                "Vacation reply suppressed\n");
  
                    ret = SIEVE_OK;
                }
--- 929,937 ----
                                 "Sent vacation reply\n");
  
                } else if (ret == SIEVE_DONE) {
!                   snprintf(actions_string+strlen(actions_string),
!                            sizeof(actions_string)-strlen(actions_string),
!                            "Vacation reply suppressed\n");
  
                    ret = SIEVE_OK;
                }
================================================================



-- 
-Gary Mills-    -Unix Support-    -U of M Academic Computing and Networking-