----- Original Message -----
> From: "Libo Chen" <clbchenlibo.c...@huawei.com>
> To: "Ivana Hutarova Varekova" <varek...@redhat.com>
> Cc: "Libcg Development list" sourceforge.net>, "Li Zefan" 
> <lize...@huawei.com>, "Huang Qiang"
> huangqi...@huawei.com>, "Wengmeiling" <wengmeiling.w...@huawei.com>
> Sent: Friday, August 30, 2013 5:05:13 AM
> Subject: [PATCH v2 1/5] cgcreate: add err check
> 
> 
> cgroup still can be created even if giving invalid -d,-f,-s parameter
> e.g.
>         cgcreate -g cpu:/test -d 888
> "888" is invalid, but test will be cgreated.
> 
> So we should exit if -d,-f or -s is invalid
> 
> Signed-off-by: Libo Chen <clbchenlibo.c...@huawei.com>
> ---
>  src/tools/cgcreate.c |    9 ++++++---
>  1 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/src/tools/cgcreate.c b/src/tools/cgcreate.c
> index b0003b1..6b80b79 100644
> --- a/src/tools/cgcreate.c
> +++ b/src/tools/cgcreate.c
> @@ -138,15 +138,18 @@ int main(int argc, char *argv[])
>                          break;
>                  case 'd':
>                          dirm_change = 1;
> -                        ret = parse_mode(optarg, &dir_mode, argv[0]);
> +                        if (parse_mode(optarg, &dir_mode, argv[0]))
> +                                goto err;
 in this way return value is 0 even if this problem occurs and no cgroup is 
created. I prefer to set ret variable to nonzero value here (and in the next 
hunks).
 
>                          break;
>                  case 'f':
>                          filem_change = 1;
> -                        ret = parse_mode(optarg, &file_mode, argv[0]);
> +                        if (parse_mode(optarg, &file_mode, argv[0]))
> +                                goto err;
>                          break;
>                  case 's':
>                          filem_change = 1;
> -                        ret = parse_mode(optarg, &tasks_mode, argv[0]);
> +                        if (parse_mode(optarg, &tasks_mode, argv[0]))
> +                                goto err;
>                          break;
>                  default:
>                          usage(1, argv[0]);
> --
> 1.7.1
> 
> 

------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
Libcg-devel mailing list
Libcg-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to