[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