[LTP] [PATCH 2/2] syscalls/chmod05: Use new ltp library

Sandeep Patil sspatil@google.com
Tue Nov 6 18:01:21 CET 2018


On Tue, Nov 06, 2018 at 05:33:05PM +0100, Cyril Hrubis wrote:
> Hi!
> > This was added, but I don't think it hurts anything, if at all it resets what
> > the test does in the setup (even though its probably not needed)?. I'll leave
> > it upto you to keep / delete this.
> 
> I've removed the cleanup and did a few minor changes as well and pushed,
> thanks.
> 
> * Deleted /* stat struct */ and other comments commenting obvious
> 
> * Added return; in the if (TST_RET == -1) statement, there is no point
>   in checking the directory permissions if the call has failed
> 
> * Removed spaces before tabs in the tst_test structure

That was done to align the fields, I will be more mindful next time.
Thanks for making the changes.

- ssp

> 
> 
> -- 
> Cyril Hrubis
> chrubis@suse.cz


More information about the ltp mailing list