From: Kamalesh Babulal Date: Wed, 16 Mar 2022 15:36:34 +0000 (+0530) Subject: cgset.c: fix checkpatch.pl warnings X-Git-Tag: v3.0~167 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=80c17c32d156ff1a037cb3844f4db2c3fb0f545f;p=thirdparty%2Flibcgroup.git cgset.c: fix checkpatch.pl warnings Fix all of the warnings/errors reported by Linux Kernel's checkpatch.pl, except SPDX_LICENSE_TAG, USE_NEGATIVE_ERRNO and NEW_TYPEDEFS types. It also introduces reverse xmas tree local variable declarations and header file reordering. In summary, this patch fixes the following checkpatch.pl recommendations: total: 1 errors, 16 warnings, 284 lines checked Signed-off-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- diff --git a/src/tools/cgset.c b/src/tools/cgset.c index 2860913e..76963809 100644 --- a/src/tools/cgset.c +++ b/src/tools/cgset.c @@ -1,14 +1,14 @@ // SPDX-License-Identifier: LGPL-2.1-only +#include "tools-common.h" + #include #include -#include #include #include #include #include - -#include "tools-common.h" +#include #define FL_RULES 1 #define FL_COPY 2 @@ -18,11 +18,10 @@ enum { }; #ifndef UNIT_TEST -static struct option const long_options[] = -{ - {"rule", required_argument, NULL, 'r'}, - {"help", no_argument, NULL, 'h'}, - {"copy-from", required_argument, NULL, COPY_FROM_OPTION}, +static const struct option long_options[] = { + {"rule", required_argument, NULL, 'r'}, + {"help", no_argument, NULL, 'h'}, + {"copy-from", required_argument, NULL, COPY_FROM_OPTION}, {NULL, 0, NULL, 0} }; @@ -30,8 +29,8 @@ int flags; /* used input method */ static struct cgroup *copy_name_value_from_cgroup(char src_cg_path[FILENAME_MAX]) { - int ret = 0; struct cgroup *src_cgroup; + int ret = 0; /* create source cgroup */ src_cgroup = cgroup_new_cgroup(src_cg_path); @@ -44,7 +43,7 @@ static struct cgroup *copy_name_value_from_cgroup(char src_cg_path[FILENAME_MAX] /* copy the name-version values to the cgroup structure */ ret = cgroup_get_cgroup(src_cgroup); if (ret != 0) { - fprintf(stderr, "cgroup %s error: %s \n", + fprintf(stderr, "cgroup %s error: %s\n", src_cg_path, cgroup_strerror(ret)); goto scgroup_err; } @@ -53,26 +52,28 @@ static struct cgroup *copy_name_value_from_cgroup(char src_cg_path[FILENAME_MAX] scgroup_err: cgroup_free(&src_cgroup); + return NULL; } - static void usage(int status, const char *program_name) { if (status != 0) { - fprintf(stderr, "Wrong input parameters," - " try %s --help' for more information.\n", + fprintf(stderr, "Wrong input parameters,"); + fprintf(stderr, " try %s --help' for more information.\n", program_name); return; } - printf("Usage: %s [-r ] ...\n" - " or: %s --copy-from "\ - " ...\n", program_name, program_name); + + printf("Usage: %s [-r ] ...\n", + program_name); + printf(" or: %s --copy-from ", program_name); + printf(" ...\n"); printf("Set the parameters of given cgroup(s)\n"); - printf(" -r, --variable Define parameter "\ - "to set\n"); - printf(" --copy-from Control group whose "\ - "parameters will be copied\n"); + printf(" -r, --variable Define parameter "); + printf("to set\n"); + printf(" --copy-from Control group whose "); + printf("parameters will be copied\n"); } #endif /* !UNIT_TEST */ @@ -80,8 +81,8 @@ STATIC int parse_r_flag(const char * const program_name, const char * const name_value_str, struct control_value * const name_value) { - int ret = 0; char *copy, *buf; + int ret = 0; copy = strdup(name_value_str); if (copy == NULL) { @@ -93,8 +94,7 @@ STATIC int parse_r_flag(const char * const program_name, /* parse optarg value */ buf = strtok(copy, "="); if (buf == NULL) { - fprintf(stderr, "%s: " - "wrong parameter of option -r: %s\n", + fprintf(stderr, "%s: wrong parameter of option -r: %s\n", program_name, optarg); ret = -1; goto err; @@ -104,14 +104,14 @@ STATIC int parse_r_flag(const char * const program_name, name_value->name[FILENAME_MAX-1] = '\0'; buf = strchr(name_value_str, '='); - /* we don't need to check the return value of strchr because we + /* + * we don't need to check the return value of strchr because we * know there's already an '=' character in the string. */ buf++; if (strlen(buf) == 0) { - fprintf(stderr, "%s: " - "wrong parameter of option -r: %s\n", + fprintf(stderr, "%s: wrong parameter of option -r: %s\n", program_name, optarg); ret = -1; goto err; @@ -130,9 +130,6 @@ err: #ifndef UNIT_TEST int main(int argc, char *argv[]) { - int ret = 0; - int c; - struct control_value *name_value = NULL; int nv_number = 0; int nv_max = 0; @@ -141,10 +138,13 @@ int main(int argc, char *argv[]) struct cgroup *src_cgroup; struct cgroup *cgroup; - /* no parametr on input */ + int ret = 0; + int c; + + /* no parameter on input */ if (argc < 2) { - fprintf(stderr, "Usage is %s -r " - "\n", argv[0]); + fprintf(stderr, "Usage is %s -r ", argv[0]); + fprintf(stderr, "\n"); return -1; } @@ -156,7 +156,6 @@ int main(int argc, char *argv[]) usage(0, argv[0]); ret = 0; goto err; - break; case 'r': if ((flags & FL_COPY) != 0) { @@ -166,17 +165,18 @@ int main(int argc, char *argv[]) } flags |= FL_RULES; - /* add name-value pair to buffer - (= name_value variable) */ + /* + * add name-value pair to buffer + * (= name_value variable) + */ if (nv_number >= nv_max) { nv_max += CG_NV_MAX; name_value = (struct control_value *) realloc(name_value, nv_max * sizeof(struct control_value)); if (!name_value) { - fprintf(stderr, "%s: " - "not enough memory\n", - argv[0]); + fprintf(stderr, "%s: ", argv[0]); + fprintf(stderr, "not enough memory\n"); ret = -1; goto err; } @@ -203,7 +203,6 @@ int main(int argc, char *argv[]) usage(1, argv[0]); ret = -1; goto err; - break; } } @@ -256,7 +255,7 @@ int main(int argc, char *argv[]) /* copy the values from the source cgroup to new one */ ret = cgroup_copy_cgroup(cgroup, src_cgroup); if (ret != 0) { - fprintf(stderr, "%s: cgroup %s error: %s \n", + fprintf(stderr, "%s: cgroup %s error: %s\n", argv[0], src_cg_path, cgroup_strerror(ret)); goto cgroup_free_err; } @@ -264,7 +263,7 @@ int main(int argc, char *argv[]) /* modify cgroup based on values of the new one */ ret = cgroup_modify_cgroup(cgroup); if (ret) { - fprintf(stderr, "%s: cgroup modify error: %s \n", + fprintf(stderr, "%s: cgroup modify error: %s\n", argv[0], cgroup_strerror(ret)); goto cgroup_free_err; } @@ -277,9 +276,9 @@ cgroup_free_err: if (cgroup) cgroup_free(&cgroup); cgroup_free(&src_cgroup); - err: free(name_value); + return ret; } #endif /* !UNIT_TEST */