[Top][All Lists]

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

Re: [Bug-wget] fuzz tests

From: Nam Nguyen
Subject: Re: [Bug-wget] fuzz tests
Date: Wed, 20 Feb 2019 19:31:03 -0800
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (berkeley-unix)

Hi Tim,

Tim Rühsen <address@hidden> writes:
> not sure why, but your email just popped up here though it is dated
> 18.2.2019 9:39 AM.

I sent that e-mail before subscribing to the mailing list and only
noticed that it would be moderated afterward. Now that I am subscribed,
responses should be delivered promptly.

> latest git master should have the tests in fuzz/ being fixed. Please
> give it a try and report back if anything doesn't work on OpenBSD.

I cherry-picked your upstream changes to patch the latest tarball of
wget. Unit tests work and fuzz tests work. (I wonder why the unit tests
did not run previously?) The only change I had to make was
`patch-lib_Makefile_am', because otherwise compilation/linking fails due
to missing symbols in libgnu.a (e.g., from
unicase/empty-prefix-context.c). What do you think?

--8<---------------cut here---------------start------------->8---

add unknown symbols to libgnu

Index: lib/Makefile.am
--- lib/Makefile.am.orig
+++ lib/Makefile.am
@@ -3114,17 +3114,13 @@ EXTRA_DIST += unicase/cased.h unicase/caseprop.h unict
 ## begin gnulib module unicase/empty-prefix-context
 libgnu_a_SOURCES += unicase/empty-prefix-context.c
 ## end   gnulib module unicase/empty-prefix-context
 ## begin gnulib module unicase/empty-suffix-context
 libgnu_a_SOURCES += unicase/empty-suffix-context.c
 ## end   gnulib module unicase/empty-suffix-context
@@ -3447,9 +3443,7 @@ EXTRA_DIST += unistr.in.h
 ## begin gnulib module unistr/u8-cpy
 libgnu_a_SOURCES += unistr/u8-cpy.c
 EXTRA_DIST += unistr/u-cpy.h
@@ -3457,9 +3451,7 @@ EXTRA_DIST += unistr/u-cpy.h
 ## begin gnulib module unistr/u8-mbtouc-unsafe
 libgnu_a_SOURCES += unistr/u8-mbtouc-unsafe.c unistr/u8-mbtouc-unsafe-aux.c
 ## end   gnulib module unistr/u8-mbtouc-unsafe
@@ -3473,9 +3465,7 @@ endif
 ## begin gnulib module unistr/u8-uctomb
 libgnu_a_SOURCES += unistr/u8-uctomb.c unistr/u8-uctomb-aux.c
 ## end   gnulib module unistr/u8-uctomb
--8<---------------cut here---------------end--------------->8---

> ASAN / UBSAN builds (or valgrind test runs)
> The test data comes mainly from OSS-Fuzz and covers all kinds of code
> paths (though we are far away from 100%).

This sounds like a really interesting concept to me. I also need to
learn more about autoconf and Makefiles. Thank you for the clear

Attaching test logs:
test.log: output from ports tree `make test'
wget_css_fuzzer.log: log file found in fuzz/

Best Regards,

Attachment: test.log
Description: test.log

Attachment: wget_css_fuzzer.log
Description: wget_css_fuzzer.log

reply via email to

[Prev in Thread] Current Thread [Next in Thread]