From: Rich Felker <dalias@aerifal.cx>
Date: Sun, 13 May 2012 21:20:01 +0000 (-0400)
Subject: remove some no-op end of string tests from regex parser
X-Git-Tag: v0.9.1~50
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=13b2945a3c3de7c309ca2e44ab4cf76236ff1fdb;p=oweals%2Fmusl.git

remove some no-op end of string tests from regex parser

these are cruft from the original code which used an explicit string
length rather than null termination. i blindly converted all the
checks to null terminator checks, without noticing that in several
cases, the subsequent switch statement would automatically handle the
null byte correctly.
---

diff --git a/src/regex/regcomp.c b/src/regex/regcomp.c
index 712ca80d..322a5e67 100644
--- a/src/regex/regcomp.c
+++ b/src/regex/regcomp.c
@@ -1052,8 +1052,6 @@ tre_parse(tre_parse_ctx_t *ctx)
 	  }
 
 	case PARSE_UNION:
-	  if (!*ctx->re)
-	    break;
 	  switch (*ctx->re)
 	    {
 	    case CHAR_PIPE:
@@ -1086,8 +1084,6 @@ tre_parse(tre_parse_ctx_t *ctx)
 
 	case PARSE_POSTFIX:
 	  /* Parse postfix operators. */
-	  if (!*ctx->re)
-	    break;
 	  switch (*ctx->re)
 	    {
 	    case CHAR_PLUS: