replicator: fixed infinity loop on sync replication

Before this fix, replicator adds the same lookup into callbacks over and
over, until it reaches out of memory, and crashes.

This regression was introduced 447e086
with initial implementation of replicator.

src/replication/replicator/replicator-queue.c

@@ -321,15 +321,14 @@ replicator_queue_handle_sync_lookups(struct replicator_queue *queue,
 
 	t_array_init(&callbacks, 8);
 	lookups = array_get_modifiable(&queue->sync_lookups, &count);
-	for (i = 0; i < count; ) {
+	for (i = 0; i < count; i++) {
 		if (lookups[i].user != user)
-			i++;
-		else if (lookups[i].wait_for_next_push) {
+			continue;
+		if (lookups[i].wait_for_next_push) {
 			/* another sync request came while user was being
 			   replicated */
 			i_assert(user->priority == REPLICATION_PRIORITY_SYNC);
 			lookups[i].wait_for_next_push = FALSE;
-			i++;
 		} else {
 			array_push_back(&callbacks, &lookups[i]);
 			array_delete(&queue->sync_lookups, i, 1);