CURRENT branch

[Russ Allbery]

Julien ÉLIE <julien at trigofacile.com> writes:

> What if:
>
>    if (entry->offset + entry->length > search->data->datalen) {
>        warn("Invalid or inaccessible entry for article %lu in %s.IDX:"
>             " offset %lu length %lu datalength %lu",
>             search->current + search->data->base, search->data->path,
>             (unsigned long) entry->offset, (unsigned long) entry->length,
>             (unsigned long) search->data->datalen);
> +       data->remapoutoforder = true;
>        return false;
>    }
>
> in tdx_search()?  (We just have to add "data" to its arguments.)  This
> way, next time tdx_search_open() is called by nnrpd, it will remap the
> data file.  Whence the client will see the article next time it starts
> a search.

Oh, hey, that's a good idea.  It probably won't help with most clients,
but it might help and I don't see any drawbacks.

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>

    Please send questions to the list rather than mailing me directly.
     <http://www.eyrie.org/~eagle/faqs/questions.html> explains why.