From 3aa3c6e17b2874b8a8d4afed503f92b5117946d0 Mon Sep 17 00:00:00 2001 From: Ken'ichi Ohmichi Date: Wed, 3 Jun 2009 15:02:59 +0900 Subject: [PATCH] Cleanup: Add cg_skip_unused_charactors_in_rule(). Hi, CHANGELOG of v2: ================ * No change. The loop in cgroup_parse_rules() is a little long now, and it is not easy to read the loop. Then, This patch shortens the loop for the readability. Thanks Ken'ichi Ohmichi Signed-off-by: Ken'ichi Ohmichi Reviewed-By: Jan Safranek Signed-off-by: Dhaval Giani --- src/api.c | 45 +++++++++++++++++++++++++++++---------------- 1 file changed, 29 insertions(+), 16 deletions(-) diff --git a/src/api.c b/src/api.c index d016538e..3af9de04 100644 --- a/src/api.c +++ b/src/api.c @@ -243,6 +243,32 @@ static void cgroup_free_rule_list(struct cgroup_rule_list *rl) rl->tail = NULL; } +static char *cg_skip_unused_charactors_in_rule(char *rule) +{ + char *itr; + + /* We ignore anything after a # sign as comments. */ + itr = strchr(rule, '#'); + if (itr) + *itr = '\0'; + + /* We also need to remove the newline character. */ + itr = strchr(rule, '\n'); + if (itr) + *itr = '\0'; + + /* Now, skip any leading tabs and spaces. */ + itr = rule; + while (itr && isblank(*itr)) + itr++; + + /* If there's nothing left, we can ignore this line. */ + if (!strlen(itr)) + return NULL; + + return itr; +} + /** * Parse the configuration file that maps UID/GIDs to cgroups. If ever the * configuration file is modified, applications should call this function to @@ -334,24 +360,11 @@ static int cgroup_parse_rules(bool cache, uid_t muid, gid_t mgid) /* Now, parse the configuration file one line at a time. */ cgroup_dbg("Parsing configuration file.\n"); - while ((itr = fgets(buff, sizeof(buff), fp)) != NULL) { + while (fgets(buff, sizeof(buff), fp) != NULL) { linenum++; - /* We ignore anything after a # sign as comments. */ - if ((itr = strchr(buff, '#'))) - *itr = '\0'; - - /* We also need to remove the newline character. */ - if ((itr = strchr(buff, '\n'))) - *itr = '\0'; - - /* Now, skip any leading tabs and spaces. */ - itr = buff; - while (itr && isblank(*itr)) - itr++; - - /* If there's nothing left, we can ignore this line. */ - if (!strlen(itr)) + itr = cg_skip_unused_charactors_in_rule(buff); + if (!itr) continue; /* -- 2.47.2