[Tarantool-patches] [PATCH 2/4] test: add corpus to be used with fuzzers

[Sergey Bronnikov]

Hello!

thanks for review!

On 07.12.2020 20:34, Igor Munkin wrote:
> Sergey,
>
> Thanks for the patch! Please consider my comments below.
>
> On 30.11.20, sergeyb at tarantool.org wrote:
>> From: Sergey Bronnikov <sergeyb at tarantool.org>
>>
>> Fuzzing tools uses evolutionary algorithms. Supplying seed corpus consisting
>> of good sample inputs is one of the best ways to improve fuzz target’s
>> coverage. Patch adds a corpuses that can be used with existed fuzzers.
>> The name of each file in the corpus is the sha1 checksum of its contents.
> Minor: The body of the commit message should be wrapped up to 72
> characters (except the links I believe) according to our guidelines[1].
Fixed in a branch.
>
>> Corpus with http headers was added from [1] and [2].
>>
>> 1. https://google.github.io/oss-fuzz/getting-started/new-project-guide/#seed-corpus
>> 2. https://en.wikipedia.org/wiki/List_of_HTTP_header_fields
>> 3. https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers
>>
>> The libFuzzer allow to minimize corpus with help of `-merge` flag:
>> when 1 is passed, any corpus inputs from the 2nd, 3rd etc. corpus directories
>> that trigger new code coverage will be merged into the first corpus directory,
>> when 0 is passed an existed corpus will be minimized.
> Ditto.
Fixed in a branch.
>> All provided corpuses in a patch were minimized.
>>
>> Part of #1809
> I have no idea, how to review this patch further. It looks OK except the
> testing fails the way I wrote in the previous reply. So, I stop here.
>
>> ---
> <snipped>
>
>> -- 
>> 2.25.1
>>
> [1]: https://www.tarantool.io/en/doc/latest/dev_guide/developer_guidelines/
>