Move fflush and fsync to the mbox and mmdf commit_msg functions.

The case statement in mx_commit_message() was previously distributed
to the various ops->commit_msg() handlers, but the fflush and fsync were
not.

1 files changed, 14 insertions, 4 deletions

diff --git a/mbox.c b/mbox.c
index 36dd81e7..1aabe9e5 100644
--- a/mbox.c
+++ b/mbox.c
@@ -463,20 +463,30 @@ static int mbox_close_message (CONTEXT *ctx, MESSAGE *msg)
 
 static int mbox_commit_message (CONTEXT *ctx, MESSAGE *msg)
 {
-  int r = fputc ('\n', msg->fp);
+  if (fputc ('\n', msg->fp) == EOF)
+    return -1;
 
-  if (r == EOF)
+  if ((fflush (msg->fp) == EOF) ||
+      (fsync (fileno (msg->fp)) == -1))
+  {
+    mutt_perror _("Can't write message");
     return -1;
+  }
 
   return 0;
 }
 
 static int mmdf_commit_message (CONTEXT *ctx, MESSAGE *msg)
 {
-  int r = fputs (MMDF_SEP, msg->fp);
+  if (fputs (MMDF_SEP, msg->fp) == EOF)
+    return -1;
 
-  if (r == EOF)
+  if ((fflush (msg->fp) == EOF) ||
+      (fsync (fileno (msg->fp)) == -1))
+  {
+    mutt_perror _("Can't write message");
     return -1;
+  }
 
   return 0;
 }