Create temporary mailbox instead of the temporary message, to avoid problems whe…

…n running sieve fileinto.

Create temporary mailbox instead of the temporary message, to avoid problems whe…
…n running sieve fileinto.
Sergey Poznyakoff
Commit d7938606 ... d793860694309d3d382528a0b45a15a218d93a57 authored 2002-12-13 15:50:14 +0000 by Sergey Poznyakoff
Showing 3 changed files with 38 additions and 40 deletions
mail.local/mail.local.h
mail.local/main.c
mail.local/script.c
--- a/mail.local/mail.local.h
View file @d793860
+++ b/mail.local/mail.local.h
View file @d793860
@@ -100,15 +100,14 @@ extern int exit_code;
 extern void setgroupquota __P((char *str));
 extern int check_quota __P((char *name, size_t size, size_t *rest));
-int mda __P((message_t msg, char *username));
+int mda __P((mailbox_t mbx, char *username));
 int switch_user_id __P((struct mu_auth_data *auth, int user));
 void mailer_err __P((char *fmt, ...));
 #ifdef WITH_GUILE
 struct mda_data
 {
-  message_t msg;
+  mailbox_t mbox;
-  char *tempfile;
  char *progfile;
  char *progfile_pattern;
  char **argv;
--- a/mail.local/main.c
View file @d793860
+++ b/mail.local/main.c
View file @d793860
@@ -34,12 +34,12 @@ mu_debug_t mudebug;
 #define MAXFD 64
 #define EX_QUOTA() (ex_quota_tempfail ? EX_TEMPFAIL : EX_UNAVAILABLE)
-void close_fds ();
+void close_fds __P((void));
-int make_tmp __P((const char *from, message_t *msg, char **tempfile));
+int make_tmp __P((const char *from, mailbox_t *mbx));
-void deliver (message_t msg, char *name);
+void deliver __P((mailbox_t msg, char *name));
-void guess_retval (int ec);
+void guess_retval __P((int ec));
-void mailer_err (char *fmt, ...);
+void mailer_err __P((char *fmt, ...));
-void notify_biff (mailbox_t mbox, char *name, size_t size);
+void notify_biff __P((mailbox_t mbox, char *name, size_t size));
 const char *argp_program_version = "mail.local (" PACKAGE_STRING ")";
 static char doc[] =
@@ -265,8 +265,7 @@ _sieve_parse_error (void *user_name, const char *filename, int lineno,
 int
 main (int argc, char *argv[])
 {
-  char *tempfile = NULL;
+  mailbox_t mbox = NULL;
-  message_t message = NULL;
  int arg_index;
  /* Preparative work: close inherited fds, force a reasonable umask
@@ -326,7 +325,7 @@ main (int argc, char *argv[])
    list_append (bookie, smtp_record);
  }
-  if (make_tmp (from, &message, &tempfile))
+  if (make_tmp (from, &mbox))
    exit (exit_code);
  if (multiple_delivery)
@@ -338,22 +337,20 @@ main (int argc, char *argv[])
      struct mda_data mda_data;
      memset (&mda_data, 0, sizeof mda_data);
-      mda_data.msg = message;
+      mda_data.mbox = mbox;
      mda_data.argv = argv;
      mda_data.progfile_pattern = progfile_pattern;
-      mda_data.tempfile = tempfile;
      return prog_mda (&mda_data);
    }
 #endif
-  unlink (tempfile);
  for (; *argv; argv++)
-    mda (message, *argv);
+    mda (mbox, *argv);
  return exit_code;
 }
 int
-sieve_test (struct mu_auth_data *auth, message_t msg)
+sieve_test (struct mu_auth_data *auth, mailbox_t mbx)
 {
  int rc = 1;
  char *progfile;
@@ -388,7 +385,9 @@ sieve_test (struct mu_auth_data *auth, message_t msg)
 	  if (rc == 0)
 	    {
 	      attribute_t attr;
+	      message_t msg = NULL;
+	      mailbox_get_message (mbx, 1, &msg);
 	      message_get_attribute (msg, &attr);
 	      attribute_unset_deleted (attr);
 	      if (switch_user_id (auth, 1) == 0)
@@ -411,9 +410,9 @@ sieve_test (struct mu_auth_data *auth, message_t msg)
 }
 int
-mda (message_t msg, char *username)
+mda (mailbox_t mbx, char *username)
 {
-  deliver (msg, username);
+  deliver (mbx, username);
  if (multiple_delivery)
    exit_code = EX_OK;
@@ -478,7 +477,7 @@ tmp_write (stream_t stream, off_t *poffset, char *buf, size_t len)
 }
 int
-make_tmp (const char *from, message_t *msg, char **tempfile)
+make_tmp (const char *from, mailbox_t *mbox)
 {
  stream_t stream;
  char *buf = NULL;
@@ -487,9 +486,10 @@ make_tmp (const char *from, message_t *msg, char **tempfile)
  size_t line;
  int status;
  static char *newline = "\n";
+  char *tempfile;
-  *tempfile = mu_tempname (NULL);
+  tempfile = mu_tempname (NULL);
-  if ((status = file_stream_create (&stream, *tempfile, MU_STREAM_RDWR)))
+  if ((status = file_stream_create (&stream, tempfile, MU_STREAM_RDWR)))
    {
      mailer_err ("unable to open temporary file: %s", mu_errstring (status));
      exit (exit_code);
@@ -557,6 +557,8 @@ make_tmp (const char *from, message_t *msg, char **tempfile)
  status = tmp_write (stream, &offset, newline, 1);
  free (buf);
+  unlink (tempfile);
+  free (tempfile);
  if (status)
    {
@@ -566,9 +568,17 @@ make_tmp (const char *from, message_t *msg, char **tempfile)
      return status;
    }
-  if ((status = message_create (msg, NULL)) == 0)
+  stream_flush (stream);
-    status = message_set_stream (*msg, stream, NULL);
+  if ((status = mailbox_create (mbox, "/dev/null")) 
+      || (status = mailbox_open (*mbox, MU_STREAM_READ))
+      || (status = mailbox_set_stream (*mbox, stream)))
+    {
+      mailer_err ("temporary file open error: %s", mu_errstring (status));
+      stream_destroy (&stream, stream_get_owner (stream));
+      return status;
+    }
+  status = mailbox_messages_count (*mbox, &n);
  if (status)
    {
      errno = status;
@@ -582,7 +592,7 @@ make_tmp (const char *from, message_t *msg, char **tempfile)
 }
 void
-deliver (message_t msg, char *name)
+deliver (mailbox_t imbx, char *name)
 {
  mailbox_t mbox;
  char *path;
@@ -602,14 +612,14 @@ deliver (message_t msg, char *name)
      return;
    }
-  if (!sieve_test (auth, msg))
+  if (!sieve_test (auth, imbx))
    {
      exit_code = EX_OK;
      mu_auth_data_free (auth);
      return;
    }
-  if ((status = message_get_stream (msg, &istream)) != 0)
+  if ((status = mailbox_get_stream (imbx, &istream)) != 0)
    {
      mailer_err ("can't get input message stream: %s", mu_errstring (status));
      mu_auth_data_free (auth);
--- a/mail.local/script.c
View file @d793860
+++ b/mail.local/script.c
View file @d793860
@@ -39,19 +39,8 @@ prog_mda (struct mda_data *data)
  x_argv[0] = "mail.local";
  x_argv[1] = NULL;
-  message_get_stream (data->msg, &stream);
-  stream_flush (stream);
-  if (mailbox_create (&mbox, data->tempfile)
-      || mailbox_open (mbox, MU_STREAM_RDWR) != 0)
-    {
-      mailer_err ("can't open temporary storage");
-      return EX_UNAVAILABLE;
-    }
-  unlink (data->tempfile);
  param.debug_guile = debug_guile;
-  param.mbox = mbox;
+  param.mbox = data->mbox;
  param.user_name = NULL;
  param.init = mda_init;
  param.catch_body = mda_catch_body;
@@ -117,7 +106,7 @@ mda_catch_body (void *data, mailbox_t mbox)
    return SCM_BOOL_F;
  mda_switch_to_user (NULL);
-  mda (md->msg, md->argv[0]);
+  mda (md->mbox, md->argv[0]);
  return SCM_BOOL_F;
 }