lib-login: Don't reset process title after successful login

The login_server_conn_unref() function was unconditionally calling
login_server_proctitle_refresh(), which would reset the process title
to "[idling]". This happened even after a successful login, overwriting
the process title set by the service (e.g., imap, pop3).

diff --git a/src/lib-login/login-server.c b/src/lib-login/login-server.c
index 6dcebfcc9a55f61f0c46c637cd219d288f437019..b2c3e448f567bce87bb41179bb1a870d680e8156 100644 (file)
--- a/src/lib-login/login-server.c
+++ b/src/lib-login/login-server.c
@@ -586,10 +586,10 @@ static void login_server_conn_unref(struct login_server_connection **_conn)
        o_stream_unref(&conn->output);
 
        DLLIST_REMOVE(&conn->server->conns, conn);
-       login_server_proctitle_refresh(conn->server);
-
-       if (!conn->login_success)
+       if (!conn->login_success) {
+               login_server_proctitle_refresh(conn->server);
                master_service_client_connection_destroyed(conn->server->service);
+       }
        event_unref(&conn->event);
        i_free(conn);
 }