From 8025afad8af99c770d8be3aeca0dbe6329e64d61 Mon Sep 17 00:00:00 2001 From: Bernhard Reutner-Fischer Date: Mon, 2 Apr 2007 16:54:41 +0000 Subject: [PATCH] - use xregcomp. Untested. vda, isn't there a regfree missing here? text data bss dec hex filename 2522 0 0 2522 9da expr.o.r18305 2478 0 0 2478 9ae expr.o --- coreutils/expr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/coreutils/expr.c b/coreutils/expr.c index 8eda36739..8401594b5 100644 --- a/coreutils/expr.c +++ b/coreutils/expr.c @@ -226,8 +226,7 @@ of a basic regular expression is not portable; it is being ignored", pv->u.s); memset(&re_buffer, 0, sizeof(re_buffer)); memset(re_regs, 0, sizeof(*re_regs)); - if (regcomp(&re_buffer, pv->u.s, 0) != 0) - bb_error_msg_and_die("invalid regular expression"); + xregcomp(&re_buffer, pv->u.s, 0); /* expr uses an anchored pattern match, so check that there was a * match and that the match starts at offset 0. */ @@ -246,6 +245,7 @@ of a basic regular expression is not portable; it is being ignored", pv->u.s); else v = int_value(0); } +//FIXME: sounds like here is a bit missing: regfree(&re_buffer); return v; }