dovecot-2.2-pigeonhole: managesieve: Remove the client from clie...

[pigeonhole at rename-it.nl]

details:   http://hg.rename-it.nl/dovecot-2.2-pigeonhole/rev/e547bdec932f
changeset: 1869:e547bdec932f
user:      Stephan Bosch <stephan at rename-it.nl>
date:      Tue Jun 17 20:58:23 2014 +0200
description:
managesieve: Remove the client from clients-list at the very end of the destroy function.
Parallel to identical Dovecot change for imap and pop3.

diffstat:

 src/managesieve/managesieve-client.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (23 lines):

diff -r 54ccf712b9d0 -r e547bdec932f src/managesieve/managesieve-client.c
--- a/src/managesieve/managesieve-client.c	Tue Jun 17 20:48:50 2014 +0200
+++ b/src/managesieve/managesieve-client.c	Tue Jun 17 20:58:23 2014 +0200
@@ -218,9 +218,6 @@
 		i_info("%s %s", reason, client_stats(client));
 	}
 
-	managesieve_client_count--;
-	DLLIST_REMOVE(&managesieve_clients, client);
-
 	if (client->command_pending) {
 		/* try to deinitialize the command */
 		i_assert(client->cmd.func != NULL);
@@ -263,6 +260,9 @@
 	sieve_deinit(&client->svinst);
 
 	pool_unref(&client->cmd.pool);
+
+	managesieve_client_count--;
+	DLLIST_REMOVE(&managesieve_clients, client);
 	pool_unref(&client->pool);
 
 	master_service_client_connection_destroyed(master_service);