Re: [PATCH 1/3] fuzz: port fuzz-credential-from-url-gently from OSS-Fuzz

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



just some nits:

On Thu, Oct 10, 2024 at 02:11:53PM -0700, Josh Steadmon wrote:
+++ b/ci/run-build-and-minimal-fuzzers.sh
+fuzzers="
+commit-graph \
+config \
+credential-from-url-gently \
+date \
+pack-headers \
+pack-idx \
+"
the trailing space-backslashes can be left out, which would make the
code less noisy. then the variable would contain linebreaks instead of
spaces, which the for loop would be just fine with (as $IFS contains
LF).

+for fuzzer in $fuzzers ; do

the space before the semicolon seems excessive.

+++ b/oss-fuzz/fuzz-credential-from-url-gently.c
+int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size);
+
are these seemingly redundant prototypes meant to suppress compiler
warnings?

+int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size)
+{





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux