From 508d9ff8ac32ca869ed0f5b4139db1e9d877a462 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Mon, 27 Sep 1999 16:47:01 +0000 Subject: [PATCH] Update. * locale/programs/ld-collate.c (insert_value): Add hack to not crash in incomplete collate implementation. --- ChangeLog | 3 +++ locale/programs/ld-collate.c | 20 ++++++++++++++++---- 2 files changed, 19 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index 200bf93..b6a42b8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 1999-09-27 Ulrich Drepper + * locale/programs/ld-collate.c (insert_value): Add hack to not + crash in incomplete collate implementation. + * locale/programs/ld-collate.c (read_directions): Read new token after accepting sorting rule. diff --git a/locale/programs/ld-collate.c b/locale/programs/ld-collate.c index 070c65a..fcfdc81 100644 --- a/locale/programs/ld-collate.c +++ b/locale/programs/ld-collate.c @@ -423,7 +423,7 @@ insert_value (struct linereader *ldfile, struct token *arg, /* Try to find the character in the charmap. */ seq = charmap_find_value (charmap, arg->val.str.startmb, arg->val.str.lenmb); - if (wc == ILLEGAL_CHAR_VALUE) + if (wc == ILLEGAL_CHAR_VALUE && seq == NULL) { /* It's no character, so look through the collation elements and symbol list. */ @@ -435,17 +435,29 @@ insert_value (struct linereader *ldfile, struct token *arg, /* It's a collation symbol. */ struct symbol_t *sym = (struct symbol_t *) result; elem = sym->order; + + if (elem == NULL) + elem = sym->order = new_element (collate, arg->val.str.startmb, + arg->val.str.startwc); } else if (find_entry (&collate->elem_table, arg->val.str.startmb, - arg->val.str.lenmb, &result) != 0) + arg->val.str.lenmb, &elem) != 0) /* It's also no collation element. Therefore ignore it. */ return; } + else + { + /* Otherwise the symbols stands for an character. Make sure it is + not already in the table. */ + + } - /* XXX elem must be defined. */ + if (elem == NULL) + /* XXX HACK HACK HACK */ + return; /* Test whether this element is not already in the list. */ - if (elem->next != NULL) + if (elem->next != NULL || elem->next == collate->cursor) { lr_error (ldfile, _("order for `%.*s' already defined at %s:%Z"), arg->val.str.startmb, arg->val.str.lenmb, -- 2.7.4