[lxc-devel] [PATCH] tests/startone: wrong set_cgroup_item check

Serge Hallyn serge.hallyn at ubuntu.com
Tue Jul 2 14:27:27 UTC 2013


Quoting Bogdan Purcareata (bogdan.purcareata at freescale.com):
> Minor typo.
> 
> Signed-off-by: Bogdan Purcareata <bogdan.purcareata at freescale.com>
> ---
>  src/tests/startone.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/tests/startone.c b/src/tests/startone.c
> index d781e75..2c1f39b 100644
> --- a/src/tests/startone.c
> +++ b/src/tests/startone.c
> @@ -201,7 +201,7 @@ int main(int argc, char *argv[])
>  
>  	sprintf(buf, "FROZEN");
>  	b = c->set_cgroup_item(c, "freezer.state", buf);
> -	if (!b) {
> +	if (b) {

Maybe I'm not thinking right, but why do you think setting freezer.state
should fail?

>  		fprintf(stderr, "%d: not able to set freezer.state.\n", __LINE__);
>  		goto out;
>  	}
> -- 
> 1.7.10.4
> 
> 
> 
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by Windows:
> 
> Build for Windows Store.
> 
> http://p.sf.net/sfu/windows-dev2dev
> _______________________________________________
> Lxc-devel mailing list
> Lxc-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/lxc-devel




More information about the lxc-devel mailing list