Commit 1b5572f7 authored by Philipp Schafft's avatar Philipp Schafft 🦁

Fix: Fixed a memory leak caused by assigning the client's role twice in some cases

parent 9ffd16cb
...@@ -365,7 +365,7 @@ static void __handle_auth_client (auth_t *auth, auth_client *auth_user) { ...@@ -365,7 +365,7 @@ static void __handle_auth_client (auth_t *auth, auth_client *auth_user) {
if (auth_user->client->acl) if (auth_user->client->acl)
acl_release(auth_user->client->acl); acl_release(auth_user->client->acl);
acl_addref(auth_user->client->acl = auth->acl); acl_addref(auth_user->client->acl = auth->acl);
if (auth->role) /* TODO: Handle errors here */ if (auth->role && !auth_user->client->role) /* TODO: Handle errors here */
auth_user->client->role = strdup(auth->role); auth_user->client->role = strdup(auth->role);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment