diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2003-11-13 06:01:46 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2003-11-13 06:01:46 +0000 |
| commit | 06e8303a288f626929d69e57eb4128b2e40b1313 (patch) | |
| tree | 992c00ec8ac45cd0d97a5778bb1b947799588469 /posix | |
| parent | 813ec65a4d69dcf9b2ecaf42395a4ba62dd2d33c (diff) | |
| download | glibc-06e8303a288f626929d69e57eb4128b2e40b1313.tar.gz glibc-06e8303a288f626929d69e57eb4128b2e40b1313.tar.xz glibc-06e8303a288f626929d69e57eb4128b2e40b1313.zip | |
Update.
* posix/regcomp.c (parse_expression): If token is OP_OPEN_DUP_NUM and RE_CONTEXT_INVALID_DUP syntax flag is set, fail. * posix/regex.h (RE_CONTEXT_INVALUD_OPS): New macro. (RE_SYNTAX_POSIX_BASIC): Use RE_CONTEXT_INVALUD_OPS. * posix/regcomp.c (parse_sub_exp): In case of not-matching ( ) return REG_EPAREN.
Diffstat (limited to 'posix')
| -rw-r--r-- | posix/regcomp.c | 12 | ||||
| -rw-r--r-- | posix/regex.h | 6 |
2 files changed, 14 insertions, 4 deletions
diff --git a/posix/regcomp.c b/posix/regcomp.c index ce91ef6807..c404929811 100644 --- a/posix/regcomp.c +++ b/posix/regcomp.c @@ -122,7 +122,7 @@ static reg_errcode_t build_charclass (RE_TRANSLATE_TYPE trans, reg_syntax_t syntax); #endif /* not RE_ENABLE_I18N */ static bin_tree_t *build_charclass_op (re_dfa_t *dfa, RE_TRANSLATE_TYPE trans, - const unsigned char *class_name, + const unsigned char *class_name, const unsigned char *extra, int not, reg_errcode_t *err); static void free_bin_tree (bin_tree_t *tree); @@ -2095,10 +2095,16 @@ parse_expression (regexp, preg, token, syntax, nest, err) ++dfa->nbackref; dfa->has_mb_node = 1; break; + case OP_OPEN_DUP_NUM: + if (syntax & RE_CONTEXT_INVALID_DUP) + { + *err = REG_BADRPT; + return NULL; + } + /* FALLTHROUGH */ case OP_DUP_ASTERISK: case OP_DUP_PLUS: case OP_DUP_QUESTION: - case OP_OPEN_DUP_NUM: if (syntax & RE_CONTEXT_INVALID_OPS) { *err = REG_BADRPT; @@ -2292,7 +2298,7 @@ parse_sub_exp (regexp, preg, token, syntax, nest, err) if (BE (token->type != OP_CLOSE_SUBEXP, 0)) { free_bin_tree (tree); - *err = REG_BADPAT; + *err = REG_EPAREN; return NULL; } new_idx = re_dfa_add_node (dfa, *token, 0); diff --git a/posix/regex.h b/posix/regex.h index 1ab3e243a7..c76005332d 100644 --- a/posix/regex.h +++ b/posix/regex.h @@ -175,6 +175,10 @@ typedef unsigned long int reg_syntax_t; whether ^ should be special. */ #define RE_CARET_ANCHORS_HERE (RE_ICASE << 1) +/* If this bit is set, then \{ cannot be first in an bre or + immediately after an alternation or begin-group operator. */ +#define RE_CONTEXT_INVALID_DUP (RE_CARET_ANCHORS_HERE << 1) + /* This global variable defines the particular regexp syntax to use (for some interfaces). When a regexp is compiled, the syntax used is stored in the pattern buffer, so changing this does not affect @@ -229,7 +233,7 @@ extern reg_syntax_t re_syntax_options; | RE_INTERVALS | RE_NO_EMPTY_RANGES) #define RE_SYNTAX_POSIX_BASIC \ - (_RE_SYNTAX_POSIX_COMMON | RE_BK_PLUS_QM) + (_RE_SYNTAX_POSIX_COMMON | RE_BK_PLUS_QM | RE_CONTEXT_INVALID_DUP) /* Differs from ..._POSIX_BASIC only in that RE_BK_PLUS_QM becomes RE_LIMITED_OPS, i.e., \? \+ \| are not recognized. Actually, this |