tools/cgconfig: use exit code 129 on invalid usage

The exit code 1 refers to general error and using -1 wraps to
255, both codes do not signify bad arguments. Let's use exit code 129
(EXIT_BADARGS) for hinting to the users about invalid usage.

Signed-off-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

diff --git a/src/tools/cgconfig.c b/src/tools/cgconfig.c
index ef57e18401c0f4e5837b2818480e24ff66dc7dde..b541d4b12123eb8447f45d962280da29f03e2ae3 100644 (file)
--- a/src/tools/cgconfig.c
+++ b/src/tools/cgconfig.c
@@ -80,7 +80,7 @@ int main(int argc, char *argv[])
        cgroup_set_default_logger(-1);
        if (argc < 2) {
                usage(1, argv[0]);
-               return -1;
+               exit(EXIT_BADARGS);
        }
 
        ret = cgroup_init();
@@ -141,14 +141,14 @@ int main(int argc, char *argv[])
                        break;
                default:
                        usage(1, argv[0]);
-                       error = -1;
+                       error = EXIT_BADARGS;
                        goto err;
                }
        }
 
        if (argv[optind]) {
                usage(1, argv[0]);
-               error = -1;
+               error = EXIT_BADARGS;
                goto err;
        }