[LTP] [PATCH 1/2] lib: Implement SAFE_RUN() macro (new API only)
    Petr Vorel 
    pvorel@suse.cz
       
    Mon Mar 23 10:20:14 CET 2020
    
    
  
Hi Xu,
> > +		tst_brk(TBROK, "%s:%d: %s failed (%d)", file, lineno, rc);
> typo, rc -> rval and  miss  argv[0].
> > +	}
> > +}
> > +#define SAFE_RUNCMD(argv, stdout_path, stderr_path) \
> > +	safe_run_cmd(__FILE__, __LINE__, NULL, (argv), (stdout_path), (stderr_path))
> I guess we don't need the third argument(NULL).
Thanks for catching these! I've sent v2 with your Reviewed-by: tag.
Kind regards,
Petr
    
    
More information about the ltp
mailing list