Re: [Xen-devel] [PATCH 2/4] tools/fuzz: add AFL stub program for x86 insn emulator fuzzer

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Tue, Jan 24, 2017 at 05:30:32PM +0000, Ian Jackson wrote:
> Andrew Cooper writes ("Re: [PATCH 2/4] tools/fuzz: add AFL stub program for 
> x86 insn emulator fuzzer"):
> > On 24/01/17 17:22, Ian Jackson wrote:
> > > You don't need to test with stat unless you want to support files
> > > bigger than your static buffer.
> > 
> > It is expected that AFL data will eventually become longer than the
> > fuzzer wishes.  You should ignore extraneous data.
> 
> No, you should exit nonzero.
> 
> That will signal to fuzzers that this is not a "valid" input and so
> not interesting.  It will signal to other kinds of test code that
> something is wrong.
> 

Hmm... I originally thought exit(1) would cause AFL to think the program
has crashed, but I was wrong.

So using exit(1) is more sensible in this case.

Wei.

> Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel