On Wed, Jun 19, 2013 at 10:09:09PM -0400, Vince Weaver wrote:
 > 
 > While working on the perf_event_open() sysfs init problem, I was wondering
 > if it might be easier if we added the possibility of an init() routine
 > to each syscall structure.  That way trinity can support doing setup
 > before fuzzing begins.
 > 
 > Below is a quick patch implementing this, it seems to work but I still 
 > don't have the best grasp of trinity internals.
 > 
 > Signed-off-by: Vince Weaver <[email protected]>
 > 
 > diff --git a/include/syscall.h b/include/syscall.h
 > index ccbe436..a58a506 100644
 > --- a/include/syscall.h
 > +++ b/include/syscall.h
 > @@ -32,6 +32,7 @@ struct arglist {
 >  struct syscall {
 >      void (*sanitise)(int childno);
 >      void (*post)(int);
 > +    int (*init)(void);
 >  
 >      unsigned int number;
 >      const char name[80];

Nothing wrong with this patch, but it's highlighted a bug that's been in trinity
for a while.  Changes to the syscall struct should cause everything in 
syscalls/*
to be rebuilt. But somehow the dependency magic in the Makefile doesn't pick it 
up.

Anyone with better make-fu than me want to take a stab at that ?

        Dave

--
To unsubscribe from this list: send the line "unsubscribe trinity" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to