From: Timo Sirainen <timo.sirainen@open-xchange.com>
Date: Thu, 31 Oct 2019 13:36:02 +0000 (+0200)
Subject: auth: auth-worker-client: Fix setting process title to [idling]
X-Git-Tag: 2.3.15~241
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=82918227d8c1ae2f2ec860b664cb416f2ccc95c8;p=thirdparty%2Fdovecot%2Fcore.git

auth: auth-worker-client: Fix setting process title to [idling]

The existing code was completely wrong. It was set only when LIST command
started running. For other commands it wasn't set at all.
---

diff --git a/src/auth/auth-worker-client.c b/src/auth/auth-worker-client.c
index 53ea61a963..11fad490cd 100644
--- a/src/auth/auth-worker-client.c
+++ b/src/auth/auth-worker-client.c
@@ -106,6 +106,8 @@ auth_worker_request_finished_full(struct auth_worker_command *cmd,
 	auth_worker_client_unref(&cmd->client);
 	event_unref(&cmd->event);
 	i_free(cmd);
+
+	auth_worker_refresh_proctitle(CLIENT_STATE_IDLE);
 }
 
 static void auth_worker_request_finished(struct auth_worker_command *cmd,
@@ -607,8 +609,6 @@ static void list_iter_deinit(struct auth_worker_list_context *ctx)
 	auth_worker_request_finished(cmd, error);
 	auth_request_unref(&ctx->auth_request);
 	i_free(ctx);
-
-	auth_worker_refresh_proctitle(CLIENT_STATE_IDLE);
 }
 
 static void list_iter_callback(const char *user, void *context)
@@ -811,8 +811,6 @@ auth_worker_client_input_args(struct connection *conn, const char *const *args)
 	if (!ret) {
 		auth_worker_request_finished_bug(cmd, error);
 		return -1;
-	} else if (client->conn.io == NULL) {
-		auth_worker_refresh_proctitle(CLIENT_STATE_IDLE);
 	}
 	auth_worker_client_unref(&client);
 	return 1;