fts-sorl: Skip returned duplicate results.

Similar to bugfix d63b209737be in fts-lucene.

This caused assert-crash at least when searching from virtual mailbox
because the score array was larger than the actual number of results.

diff --git a/src/plugins/fts-solr/solr-connection.c b/src/plugins/fts-solr/solr-connection.c
index 15cb1b6d5b8315e69076f3585c3ec656fc92e7c0..85ae6f4be5ec919f08aa074a8f368bb53f562275 100644 (file)
--- a/src/plugins/fts-solr/solr-connection.c
+++ b/src/plugins/fts-solr/solr-connection.c
@@ -260,8 +260,9 @@ static void solr_lookup_add_doc(struct solr_lookup_xml_context *ctx)
        }
        result = solr_result_get(ctx, box_id);
 
-       seq_range_array_add(&result->uids, ctx->uid);
-       if (ctx->score != 0) {
+       if (seq_range_array_add(&result->uids, ctx->uid)) {
+               /* duplicate result */
+       } else if (ctx->score != 0) {
                score = array_append_space(&result->scores);
                score->uid = ctx->uid;
                score->score = ctx->score;