>>> On 25.09.17 at 16:26, <george.dun...@citrix.com> wrote: > --- a/tools/fuzz/x86_instruction_emulator/fuzz-emul.c > +++ b/tools/fuzz/x86_instruction_emulator/fuzz-emul.c > @@ -52,6 +52,22 @@ struct fuzz_state > struct x86_emulate_ops ops; > }; > > +static inline bool input_available(struct fuzz_state *s, size_t size)
s can be pointer to const Also how about shortening the function name to what the title says? > +{ > + return s->data_index + size < s->data_num; Shouldn't this be <= ? > +} > + > +static inline bool input_read(struct fuzz_state *s, void *dst, size_t size) > +{ > + if ( !input_available(s, size) ) > + return 0; false > + > + memcpy(dst, &s->corpus->data[s->data_index], size); > + s->data_index += size; > + > + return 1; true Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel