]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
auth: auth_worker_call() - Return void instead of the connection
authorTimo Sirainen <timo.sirainen@open-xchange.com>
Tue, 19 Apr 2022 08:42:05 +0000 (11:42 +0300)
committeraki.tuomi <aki.tuomi@open-xchange.com>
Thu, 21 Apr 2022 07:45:56 +0000 (07:45 +0000)
The return value is no longer necessary, and it most likely would just be
used wrong.

src/auth/auth-worker-server.c
src/auth/auth-worker-server.h

index 16617528289d4fb1cecdeab7903ea36d6d1315a3..41b2b2bb486375cec5d9f6871e82e96d8e007b03 100644 (file)
@@ -442,9 +442,8 @@ static void worker_input_resume(struct auth_worker_connection *conn)
        worker_input(conn);
 }
 
-struct auth_worker_connection *
-auth_worker_call(pool_t pool, const char *username, const char *data,
-                auth_worker_callback_t *callback, void *context)
+void auth_worker_call(pool_t pool, const char *username, const char *data,
+                     auth_worker_callback_t *callback, void *context)
 {
        struct auth_worker_connection *conn;
        struct auth_worker_request *request;
@@ -474,7 +473,6 @@ auth_worker_call(pool_t pool, const char *username, const char *data,
                /* reached the limit, queue the request */
                aqueue_append(worker_request_queue, &request);
        }
-       return conn;
 }
 
 void auth_worker_server_resume_input(struct auth_worker_connection *conn)
index 6b1b2414462113f7b6ef154c23d63ee9885d1be7..d2a30634a3876c8ea6e4ab9056a25ab077828b86 100644 (file)
@@ -8,9 +8,8 @@ struct auth_worker_connection;
 typedef bool auth_worker_callback_t(struct auth_worker_connection *conn,
                                    const char *reply, void *context);
 
-struct auth_worker_connection * ATTR_NOWARN_UNUSED_RESULT
-auth_worker_call(pool_t pool, const char *username, const char *data,
-                auth_worker_callback_t *callback, void *context);
+void auth_worker_call(pool_t pool, const char *username, const char *data,
+                     auth_worker_callback_t *callback, void *context);
 void auth_worker_server_resume_input(struct auth_worker_connection *conn);
 
 void auth_worker_server_init(void);