]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
lib-index: fix off-by-one in index flag update test
authorJosef 'Jeff' Sipek <jeff.sipek@dovecot.fi>
Mon, 18 Sep 2017 13:48:41 +0000 (16:48 +0300)
committerAki Tuomi <aki.tuomi@dovecot.fi>
Tue, 19 Sep 2017 09:37:41 +0000 (12:37 +0300)
When selecting the second seq, the max value we should generate is
hdr.messages_count - not one less than that.

src/lib-index/test-mail-index-transaction-update.c

index af9067b719c960be76ade9fc21be2d077e471686..0b90cc198efefb02d2fe1a0c57b8bdf24ddedbf0 100644 (file)
@@ -347,7 +347,7 @@ static void test_mail_index_flag_update_random(void)
                change = i_rand() % (MAIL_FLAGS_NONRECENT+1);
                seq1 = (i_rand() % hdr.messages_count) + 1;
                seq2 = seq1 == hdr.messages_count ? seq1 :
-                       (i_rand() % (hdr.messages_count - seq1)) + seq1;
+                       (i_rand() % (hdr.messages_count - seq1 + 1)) + seq1;
 
                switch (i_rand() % 3) {
                case 0: