imap: If command has no imap_parser, don't crash when freeing the command.

All the current commands have a parser though, so this doesn't actually fix
anything.

diff --git a/src/imap/imap-client.c b/src/imap/imap-client.c
index 4f1257185393ecfc892399b6c133420d7ae76cef..e8d6ba0a4c07460a293c0b4ee7fca967903a92f8 100644 (file)
--- a/src/imap/imap-client.c
+++ b/src/imap/imap-client.c
@@ -699,11 +699,13 @@ void client_command_free(struct client_command_context **_cmd)
        if (client->mailbox_change_lock == cmd)
                client->mailbox_change_lock = NULL;
 
-       if (client->free_parser == NULL) {
-               imap_parser_reset(cmd->parser);
-               client->free_parser = cmd->parser;
-       } else if (cmd->parser != NULL) {
-               imap_parser_unref(&cmd->parser);
+       if (cmd->parser != NULL) {
+               if (client->free_parser == NULL) {
+                       imap_parser_reset(cmd->parser);
+                       client->free_parser = cmd->parser;
+               } else {
+                       imap_parser_unref(&cmd->parser);
+               }
        }
 
        client->command_queue_size--;