Return BAD instead of NO errors with invalid message sets

--HG--
branch : HEAD

diff --git a/src/lib-storage/index/index-search.c b/src/lib-storage/index/index-search.c
index 553281a44de65764cf77a98fc006a9047ffa30c7..66f9bd5ebc8efa8dbac4f8917de60beb1a560283 100644 (file)
--- a/src/lib-storage/index/index-search.c
+++ b/src/lib-storage/index/index-search.c
@@ -750,17 +750,17 @@ static int search_get_sequid(struct index_mailbox *ibox,
                } else if (args->type == SEARCH_SET) {
                        if (!seq_update(args->value.str, first_seq, last_seq,
                                        ibox->synced_messages_count)) {
-                               mail_storage_set_error(ibox->box.storage,
-                                                      "Invalid messageset: %s",
-                                                      args->value.str);
+                               mail_storage_set_syntax_error(ibox->box.storage,
+                                       "Invalid messageset: %s",
+                                       args->value.str);
                                return FALSE;
                        }
                } else if (args->type == SEARCH_UID) {
                        if (!seq_update(args->value.str, first_uid, last_uid,
                                        ibox->index->header->next_uid-1)) {
-                               mail_storage_set_error(ibox->box.storage,
-                                                      "Invalid messageset: %s",
-                                                      args->value.str);
+                               mail_storage_set_syntax_error(ibox->box.storage,
+                                       "Invalid messageset: %s",
+                                       args->value.str);
                                return FALSE;
                        }
                } else if (args->type == SEARCH_ALL) {
@@ -835,8 +835,8 @@ static int client_seq_to_uid(struct index_mailbox *ibox,
        unsigned int expunges_before;
 
        if (seq > ibox->synced_messages_count) {
-               mail_storage_set_error(ibox->box.storage,
-                                      "Sequence out of range: %u", seq);
+               mail_storage_set_syntax_error(ibox->box.storage,
+                                             "Sequence out of range: %u", seq);
                return FALSE;
        }