PATCH: sync_client fails to exit via shutdown file

ellie timoney ellie at fastmail.com
Thu May 26 21:11:15 EDT 2016


And this is also now in git, on cyrus-imapd-2.5 and master branches.

Thanks again!

ellie

On Wed, May 25, 2016, at 07:59 AM, John Capo via Cyrus-devel wrote:
> Path is also attached.
> 
> --- cyrus-imapd-2.5.7/imap/sync_client.c.orig   2016-05-22
> 00:56:24.302742893 +0000
> +++ cyrus-imapd-2.5.7/imap/sync_client.c        2016-05-22
> 10:25:34.956598506 +0000
> @@ -2621,6 +2621,10 @@
>         /* Check for shutdown file */
>         if (sync_shutdown_file && !stat(sync_shutdown_file, &sbuf)) {
>             unlink(sync_shutdown_file);
> +           /* Have to exit with r == 0 or do_daemon() will call us
> again.
> +            * The value of r is unknown from calls to
> sync_log_reader_begin() below.
> +           */
> +           r = 0;
>             break;
>         }
> Email had 1 attachment:
> + sync_client-patch
>   1k (application/octet-stream)


More information about the Cyrus-devel mailing list