From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp34.i.mail.ru (smtp34.i.mail.ru [94.100.177.94]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id ADFCD4765E0 for ; Fri, 25 Dec 2020 17:52:58 +0300 (MSK) Date: Fri, 25 Dec 2020 14:52:54 +0000 From: Kirill Yukhin Message-ID: <20201225145254.cvld4vc2yb4qzsyg@tarantool.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Subject: Re: [Tarantool-patches] [PATCH 0/4] Add fuzzing testing List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: sergeyb@tarantool.org Cc: tarantool-patches@dev.tarantool.org Hello, On 30 ноя 23:24, Sergey Bronnikov via Tarantool-patches wrote: > From: Sergey Bronnikov > > Patch series adds required support in project infrastructure for fuzzing tests, > add tests for uri, csv and http_parser modules, adds integration with OSS Fuzz > (infrastructure for continuous running fuzzing tests). > > NOTE: v2 version had LGTM from Sergey Petrenko, see [1] > > Changelog: > > v2: > - introduce CMake flag to enable building fuzzers > - add fuzzers based on LibFuzzer to csv, http_parser and uri modules > > v3: > - report error on an attempt to set ENABLE_FUZZER=ON with GCC compiler > - fixed return code in tests > - place tests to a common dir test/fuzz/ like we did with unit tests > - enable building fuzzers in Tarantool CI job > - add target to build all availabe fuzzers > - add integration with OSS Fuzz [2] > - add corpus for uri, csv and http_parser tests > > 1. https://lists.tarantool.org/pipermail/tarantool-patches/2020-April/016471.html > 2. https://github.com/google/oss-fuzz/pull/4723 > > CI: https://gitlab.com/tarantool/tarantool/-/pipelines/223112908 > Issue: https://github.com/tarantool/tarantool/issues/1809 I've checked your patchset into 2.5, 2.6 and master. -- Regards, Kirill Yukhin