On Mon, 2013-04-15 at 19:53 +0300, Timo Sirainen wrote: > On Mon, 2013-04-15 at 19:50 +0300, Timo Sirainen wrote: > > On Mon, 2013-04-15 at 16:37 +0200, Pascal Volk wrote: > > > > > some imap processes seems to have some problems. > > > > I still can't reproduce .. but I wonder if the attached patch makes a > > difference. > > Uh, wrong directory. This patch. :)
Once more, with some error checks :)
diff -r b38d5ebacf25 src/imap/cmd-append.c --- a/src/imap/cmd-append.c Mon Apr 15 17:22:06 2013 +0300 +++ b/src/imap/cmd-append.c Mon Apr 15 19:54:49 2013 +0300 @@ -777,6 +777,11 @@ if (ctx->litinput->eof || client->input->closed) { bool all_written = ctx->litinput->v_offset == ctx->literal_size; + if (all_written && ctx->save_ctx != NULL && !ctx->failed) { + if (mailbox_save_continue(ctx->save_ctx) < 0) + ctx->failed = TRUE; + } + /* finished - do one more read, to make sure istream-chain unreferences its stream, which is needed for litinput's unreferencing to seek the client->input to correct