director: Fix assert-crash when doveadm disconnects immediately after sending command

Any command that requires a ring sync should cause the doveadm connection to
wait until the ring is synced. However, if the disconnection happened early
enough the connection is deinitialized before the ring sync is finished.

Fixes:
Panic: file doveadm-connection.c: line 1097 (doveadm_connection_deinit): assertion failed: (conn->to_ring_sync_abort == NULL)

diff --git a/src/director/doveadm-connection.c b/src/director/doveadm-connection.c
index 3a52109fb8d4ed4988fed0faeef30310bdede1cf..be2c03f2f05830e8b8d970e96330e39fce9ad286 100644 (file)
--- a/src/director/doveadm-connection.c
+++ b/src/director/doveadm-connection.c
@@ -1068,8 +1068,11 @@ static void doveadm_connection_input(struct doveadm_connection *conn)
                        ret = doveadm_connection_cmd(conn, line);
                } T_END;
        }
-       if (conn->input->eof || conn->input->stream_errno != 0 ||
-           ret == DOVEADM_DIRECTOR_CMD_RET_FAIL)
+       /* Delay deinit if io was removed, even if the client
+          already disconnected. */
+       if (conn->io != NULL &&
+           (conn->input->eof || conn->input->stream_errno != 0 ||
+            ret == DOVEADM_DIRECTOR_CMD_RET_FAIL))
                doveadm_connection_deinit(&conn);
 }