If SORT/THREAD has invalid search parameters, return BAD instead of NO.

--HG--
branch : HEAD

diff --git a/src/imap/cmd-sort.c b/src/imap/cmd-sort.c
index 820af5bd40d719020f37807c25bd26f42c8d861e..0ca6a7e51d522472c3f49233f67d54252c15a206 100644 (file)
--- a/src/imap/cmd-sort.c
+++ b/src/imap/cmd-sort.c
@@ -130,7 +130,7 @@ bool cmd_sort(struct client_command_context *cmd)
        sargs = imap_search_args_build(pool, client->mailbox, args, &error);
        if (sargs == NULL) {
                /* error in search arguments */
-               client_send_tagline(cmd, t_strconcat("NO ", error, NULL));
+               client_send_tagline(cmd, t_strconcat("BAD ", error, NULL));
        } else if (imap_sort(cmd, charset, sargs, sorting) == 0) {
                pool_unref(&pool);
                return cmd_sync(cmd, MAILBOX_SYNC_FLAG_FAST |

diff --git a/src/imap/cmd-thread.c b/src/imap/cmd-thread.c
index 32395e6e9887687d1d368c709952716f8e6b2a73..9e6bca1f0e66a07b450c4642c3db0a144f695e2c 100644 (file)
--- a/src/imap/cmd-thread.c
+++ b/src/imap/cmd-thread.c
@@ -63,7 +63,7 @@ bool cmd_thread(struct client_command_context *cmd)
        sargs = imap_search_args_build(pool, client->mailbox, args, &error);
        if (sargs == NULL) {
                /* error in search arguments */
-               client_send_tagline(cmd, t_strconcat("NO ", error, NULL));
+               client_send_tagline(cmd, t_strconcat("BAD ", error, NULL));
        } else if (imap_thread(cmd, charset, sargs, threading) == 0) {
                pool_unref(&pool);
                return cmd_sync(cmd, MAILBOX_SYNC_FLAG_FAST |