From 0a1ac00ad6b0e5b82ea70eca121c92bdc27677fd Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Thu, 31 Oct 1996 23:49:42 +0000 Subject: (es_free): New function. (parse_str): Use it to plug small memory leak. --- src/tr.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) (limited to 'src/tr.c') diff --git a/src/tr.c b/src/tr.c index 95b1ad939..65868975d 100644 --- a/src/tr.c +++ b/src/tr.c @@ -479,6 +479,13 @@ is_char_class_member (enum Char_class char_class, unsigned int c) return result; } +static void +es_free (struct E_string *es) +{ + free (es->s); + free (es->escaped); +} + /* Perform the first pass over each range-spec argument S, converting all \c and \ddd escapes to their one-byte representations. The conversion is done in-place, so S must point to writable storage. If an invalid @@ -1441,12 +1448,13 @@ static int parse_str (const unsigned char *s, struct Spec_list *spec_list) { struct E_string es; + int fail; - if (unquote (s, &es)) - return 1; - if (build_spec_list (&es, spec_list)) - return 1; - return 0; + fail = unquote (s, &es); + if (!fail) + fail = build_spec_list (&es, spec_list); + es_free (&es); + return fail; } /* Given two specification lists, S1 and S2, and assuming that -- cgit v1.2.3-54-g00ecf