[CRIU] [PATCH] zdtm.sh: don't change owner of a test directory

[Andrew Vagin]

On Fri, Oct 03, 2014 at 11:01:24AM -0400, Christopher Covington wrote:
> On 10/03/2014 09:44 AM, Andrey Vagin wrote:
> > After changing an owner the current user will not be able to remove or
> > change the directory. It isn't convenient.
> > 
> > Reported-by: Tycho Andersen <tycho.andersen at canonical.com>
> > Signed-off-by: Andrey Vagin <avagin at openvz.org>
> > ---
> >  test/zdtm.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/test/zdtm.sh b/test/zdtm.sh
> > index b5133c3..9a2c188 100755
> > --- a/test/zdtm.sh
> > +++ b/test/zdtm.sh
> > @@ -407,7 +407,7 @@ start_test()
> >  	if ! echo $TEST_SUID_LIST | grep -q $tname; then
> >  		export ZDTM_UID=18943
> >  		export ZDTM_GID=58467
> > -		chown $ZDTM_UID:$ZDTM_GID $tdir
> > +		chmod a+w $tdir
> 
> I think the variables can be deleted. I don't seem them used elsewhere.

They are used by the zdtm library.

test/zdtm/lib/test.c:   val = getenv("ZDTM_GID");

> 
> Christopher
> 
> -- 
> Employee of Qualcomm Innovation Center, Inc.
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
> hosted by the Linux Foundation.