dovecot: FETCH/STORE n:* should also give BAD if n > number of m...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot/rev/f0d8c9056706
changeset: 6155:f0d8c9056706
user:      Timo Sirainen <tss at iki.fi>
date:      Wed Aug 01 14:38:28 2007 +0300
description:
FETCH/STORE n:* should also give BAD if n > number of messages in mailbox.

diffstat:

1 file changed, 2 insertions(+), 1 deletion(-)
src/imap/imap-search.c |    3 ++-

diffs (13 lines):

diff -r e87bfcbc490d -r f0d8c9056706 src/imap/imap-search.c
--- a/src/imap/imap-search.c	Wed Aug 01 14:33:01 2007 +0300
+++ b/src/imap/imap-search.c	Wed Aug 01 14:38:28 2007 +0300
@@ -450,7 +450,8 @@ static int imap_search_get_msgset_arg(st
 	arg = p_new(cmd->pool, struct mail_search_arg, 1);
 	arg->type = SEARCH_SEQSET;
 	arg->value.seqset = imap_messageset_parse(cmd->pool, messageset);
-	if (arg->value.seqset == NULL || cmd->client->messages_count == 0 ||
+	if (arg->value.seqset == NULL ||
+	    arg->value.seqset->seq1 > cmd->client->messages_count ||
 	    (arg->value.seqset->seq2 > cmd->client->messages_count &&
 	     arg->value.seqset->seq2 != (uint32_t)-1)) {
 		*error_r = "Invalid messageset";