]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
auth: auth_client_connection_create() - Change boolean parameters to enum
authorTimo Sirainen <timo.sirainen@open-xchange.com>
Mon, 3 Feb 2025 08:00:00 +0000 (10:00 +0200)
committerAki Tuomi <aki.tuomi@open-xchange.com>
Wed, 12 Feb 2025 10:34:16 +0000 (12:34 +0200)
src/auth/auth-client-connection.c
src/auth/auth-client-connection.h
src/auth/main.c
src/auth/test-auth-client.c

index b328634830341388a1c88878f6cc3f94fa2b9c0c..b890ba35373a9d04ba31c83c2f3de418439fa8de 100644 (file)
@@ -321,7 +321,7 @@ static const struct connection_settings auth_client_connection_set = {
 };
 
 void auth_client_connection_create(struct auth *auth, int fd, const char *name,
-                                  bool login_requests, bool token_auth)
+                                  enum auth_client_connection_flags flags)
 {
        static unsigned int connect_uid_counter = 0;
        struct auth_client_connection *conn;
@@ -337,8 +337,10 @@ void auth_client_connection_create(struct auth *auth, int fd, const char *name,
        conn->auth = auth;
        conn->refcount = 1;
        conn->connect_uid = ++connect_uid_counter;
-       conn->login_requests = login_requests;
-       conn->token_auth = token_auth;
+       conn->login_requests =
+               (flags & AUTH_CLIENT_CONNECTION_FLAG_LOGIN_REQUESTS) != 0;
+       conn->token_auth =
+               (flags & AUTH_CLIENT_CONNECTION_FLAG_TOKEN_AUTH) != 0;
        conn->conn.event_parent = auth_event;
        random_fill(conn->cookie, sizeof(conn->cookie));
 
index f0d5c5c6c5713227f5de5e070ae5f18f6327bf5a..463eb36fefc8df7bcd8bdcb7439d61cf00b09ce1 100644 (file)
@@ -5,6 +5,11 @@
 
 #define AUTH_CLIENT_MINOR_VERSION_CHANNEL_BINDING 3
 
+enum auth_client_connection_flags {
+       AUTH_CLIENT_CONNECTION_FLAG_LOGIN_REQUESTS = BIT(0),
+       AUTH_CLIENT_CONNECTION_FLAG_TOKEN_AUTH = BIT(1),
+};
+
 struct auth_client_connection {
        struct connection conn;
        struct auth *auth;
@@ -20,9 +25,8 @@ struct auth_client_connection {
        bool version_received:1;
        bool token_auth:1;
 };
-
 void auth_client_connection_create(struct auth *auth, int fd, const char *name,
-                                  bool login_requests, bool token_auth);
+                                  enum auth_client_connection_flags flags);
 
 struct auth_client_connection *
 auth_client_connection_lookup(unsigned int pid);
index 7db2fa543fdadc14da49c9242273fc057833f997..5338ff6b49d2032be37ad9bf4ef520a031a25792 100644 (file)
@@ -326,19 +326,19 @@ static void client_connected(struct master_service_connection *conn)
                break;
        case AUTH_SOCKET_LOGIN:
                auth_client_connection_create(auth, conn->fd, conn->name,
-                                             TRUE, FALSE);
+                                             AUTH_CLIENT_CONNECTION_FLAG_LOGIN_REQUESTS);
                break;
        case AUTH_SOCKET_AUTH:
-               auth_client_connection_create(auth, conn->fd, conn->name,
-                                             FALSE, FALSE);
+               auth_client_connection_create(auth, conn->fd, conn->name, 0);
                break;
        case AUTH_SOCKET_TOKEN_LOGIN:
                auth_client_connection_create(auth, conn->fd, conn->name,
-                                             TRUE, TRUE);
+                       AUTH_CLIENT_CONNECTION_FLAG_LOGIN_REQUESTS |
+                       AUTH_CLIENT_CONNECTION_FLAG_TOKEN_AUTH);
                break;
        case AUTH_SOCKET_TOKEN:
                auth_client_connection_create(auth, conn->fd, conn->name,
-                                             FALSE, TRUE);
+                       AUTH_CLIENT_CONNECTION_FLAG_TOKEN_AUTH);
                break;
        default:
                i_unreached();
index 4d3fd8784eaa31074e01a1229455a0f8665a2df5..00b0f69058ed5badebe40d77d957c7c8632b680c 100644 (file)
@@ -14,7 +14,7 @@ static void auth_client_connected(int *server_fd)
 {
        struct auth *auth = auth_default_protocol();
        int fd = net_accept(*server_fd, NULL, NULL);
-       auth_client_connection_create(auth, fd, TEST_AUTH_CLIENT_SOCKET, FALSE, FALSE);
+       auth_client_connection_create(auth, fd, TEST_AUTH_CLIENT_SOCKET, 0);
 }
 
 static void