diff options
author | Jeremy Kerr <jk@ozlabs.org> | 2013-09-13 15:25:33 +0800 |
---|---|---|
committer | Jeremy Kerr <jk@ozlabs.org> | 2013-09-24 13:14:59 +0800 |
commit | c12f6bacbd96fda27252bdb79781e5743c01da60 (patch) | |
tree | a334291b6a860e4d5dad6851fee85d8c28c9c9b5 /discover/grub2 | |
parent | fb5a2b71f85614114663c86dbba62c5c491a11b9 (diff) | |
download | talos-petitboot-c12f6bacbd96fda27252bdb79781e5743c01da60.tar.gz talos-petitboot-c12f6bacbd96fda27252bdb79781e5743c01da60.zip |
discover/grub2: Fix word append
The parser was setting word->next on the top-level word, resulting in
word chains of a maximum length of 2. Instead, use a last pointer to
always append to the last word.
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Diffstat (limited to 'discover/grub2')
-rw-r--r-- | discover/grub2/grub2.h | 1 | ||||
-rw-r--r-- | discover/grub2/parser-api.c | 5 |
2 files changed, 5 insertions, 1 deletions
diff --git a/discover/grub2/grub2.h b/discover/grub2/grub2.h index cbd69dc..970c500 100644 --- a/discover/grub2/grub2.h +++ b/discover/grub2/grub2.h @@ -19,6 +19,7 @@ struct grub2_word { } var; }; struct grub2_word *next; + struct grub2_word *last; struct list_item argv_list; }; diff --git a/discover/grub2/parser-api.c b/discover/grub2/parser-api.c index 765bd87..f8915a1 100644 --- a/discover/grub2/parser-api.c +++ b/discover/grub2/parser-api.c @@ -75,6 +75,7 @@ struct grub2_word *create_word_text(struct grub2_parser *parser, word->type = GRUB2_WORD_TEXT; word->text = talloc_strdup(word, text); word->next = NULL; + word->last = word; return word; } @@ -86,6 +87,7 @@ struct grub2_word *create_word_var(struct grub2_parser *parser, word->var.name = talloc_strdup(word, name); word->var.split = split; word->next = NULL; + word->last = word; return word; } @@ -103,5 +105,6 @@ void argv_append(struct grub2_argv *argv, struct grub2_word *word) void word_append(struct grub2_word *w1, struct grub2_word *w2) { - w1->next = w2; + w1->last->next = w2; + w1->last = w2; } |