From 81b096feef3773dfd17e384ffb9328287cdcea94 Mon Sep 17 00:00:00 2001 From: Wayne Davison Date: Mon, 30 Jan 2006 17:53:13 +0000 Subject: [PATCH] Changed the way the parse_chmod() function is called. --- chmod.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/chmod.c b/chmod.c index 3f9c8b43..8a6adfd8 100644 --- a/chmod.c +++ b/chmod.c @@ -23,8 +23,7 @@ struct chmod_mode_struct { /* Parse a chmod-style argument, and break it down into one or more AND/OR * pairs in a linked list. We use a state machine to walk through the * options. */ -struct chmod_mode_struct *parse_chmod(const char *modestr, - struct chmod_mode_struct *append_to) +int parse_chmod(const char *modestr, struct chmod_mode_struct **root_mode_ptr) { int state = STATE_1ST_HALF; int where = 0, what = 0, op = 0, topbits = 0, topoct = 0, flags = 0; @@ -154,17 +153,18 @@ struct chmod_mode_struct *parse_chmod(const char *modestr, if (state == STATE_ERROR) { free_chmod_mode(first_mode); - return NULL; + return 0; } - if (append_to) { - for (prev_mode = append_to; prev_mode->next; ) - prev_mode = prev_mode->next; - prev_mode->next = first_mode; - return append_to; + if (!(curr_mode = *root_mode_ptr)) + *root_mode_ptr = first_mode; + else { + while (curr_mode->next) + curr_mode = curr_mode->next; + curr_mode->next = first_mode; } - return first_mode; + return 1; } -- 2.34.1