lib-imap-client: Remove io earlier to avoid hammering

If connection has failed and reconnection is not yet
possible, io has to be removed here, otherwise it will
keep hammering the callback.

diff --git a/src/lib-imap-client/imapc-connection.c b/src/lib-imap-client/imapc-connection.c
index e2629b782197dc187272c9a64e129abab10838e3..5692c849ff7a8499022781b74d52105de81da9ce 100644 (file)
--- a/src/lib-imap-client/imapc-connection.c
+++ b/src/lib-imap-client/imapc-connection.c
@@ -1601,6 +1601,8 @@ static void imapc_connection_connected(struct imapc_connection *conn)
 {
        const struct ip_addr *ip = &conn->ips[conn->prev_connect_idx];
        int err;
+       if (conn->io != NULL)
+               io_remove(&conn->io);
 
        err = net_geterror(conn->fd);
        if (err != 0) {
@@ -1610,7 +1612,6 @@ static void imapc_connection_connected(struct imapc_connection *conn)
                        strerror(err)), IMAPC_CONNECT_RETRY_WAIT_MSECS);
                return;
        }
-       io_remove(&conn->io);
        conn->io = io_add(conn->fd, IO_READ, imapc_connection_input, conn);
 
        if (conn->client->set.ssl_mode == IMAPC_CLIENT_SSL_MODE_IMMEDIATE) {