diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2003-12-29 00:42:16 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2003-12-29 00:42:16 +0000 |
| commit | a96c63edd2c1b291741d443ba4e93144168eca72 (patch) | |
| tree | 26d69b839c0dc35c51b7b40ba896c490d6f430a3 /posix/regexec.c | |
| parent | 5a299c962e5e096919470ba77a92522c278ad63a (diff) | |
Update.
* posix/regcomp.c (mark_opt_subexp_iter): Declare IDX as int.
* posix/regexec.c (re_copy_regs): Fix testing for failed allocation.
_IO_peekc_unlocked, _IO_putc_unlocked, _IO_getwc_unlocked, and
Diffstat (limited to 'posix/regexec.c')
| -rw-r--r-- | posix/regexec.c | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/posix/regexec.c b/posix/regexec.c index 8f1f8c6b1f..9d1af80c0d 100644 --- a/posix/regexec.c +++ b/posix/regexec.c @@ -455,10 +455,10 @@ re_copy_regs (regs, pmatch, nregs, regs_allocated) if (regs_allocated == REGS_UNALLOCATED) { /* No. So allocate them with malloc. We allocate the arrays for the start and end in one block. */ - regs->start = re_malloc (regoff_t, 2 * need_regs); - if (BE (regs->start == NULL, 0)) + regs->start = re_malloc (regoff_t, need_regs); + regs->end = re_malloc (regoff_t, need_regs); + if (BE (regs->start == NULL, 0) || BE (regs->end == NULL, 0)) return REGS_UNALLOCATED; - regs->end = regs->start + need_regs; regs->num_regs = need_regs; } else if (regs_allocated == REGS_REALLOCATE) @@ -467,13 +467,10 @@ re_copy_regs (regs, pmatch, nregs, regs_allocated) leave it alone. */ if (BE (need_regs > regs->num_regs, 0)) { - regs->start = re_realloc (regs->start, regoff_t, 2 * need_regs); - if (BE (regs->start == NULL, 0)) - { - regs->end = NULL; - return REGS_UNALLOCATED; - } - regs->end = regs->start + need_regs; + regs->start = re_realloc (regs->start, regoff_t, need_regs); + regs->end = re_realloc (regs->end, regoff_t, need_regs); + if (BE (regs->start == NULL, 0) || BE (regs->end == NULL, 0)) + return REGS_UNALLOCATED; regs->num_regs = need_regs; } } |