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 v2: - add nonzero return value Signed-off-by: Libo Chen <clbchenlibo.c...@huawei.com> Acked-by: Ivana Hutarova Varekova <varek...@redhat.com> --- src/tools/cgcreate.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/tools/cgcreate.c b/src/tools/cgcreate.c index b0003b1..f1920ea 100644 --- a/src/tools/cgcreate.c +++ b/src/tools/cgcreate.c @@ -139,14 +139,20 @@ int main(int argc, char *argv[]) case 'd': dirm_change = 1; ret = parse_mode(optarg, &dir_mode, argv[0]); + if (ret) + goto err; break; case 'f': filem_change = 1; ret = parse_mode(optarg, &file_mode, argv[0]); + if (ret) + goto err; break; case 's': filem_change = 1; ret = parse_mode(optarg, &tasks_mode, argv[0]); + if (ret) + 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=58041391&iu=/4140/ostg.clktrk _______________________________________________ Libcg-devel mailing list Libcg-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libcg-devel