From: Glenn L McGrath Date: Sun, 14 Sep 2003 16:28:08 +0000 (-0000) Subject: Memory cleanups and fix for `echo "foo" | sed 's/foo/bar/;H;q'` X-Git-Tag: 1_00_pre4~187 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=294d113adb6ff175ad1729e35888c36b0fc824d1;p=oweals%2Fbusybox.git Memory cleanups and fix for `echo "foo" | sed 's/foo/bar/;H;q'` --- diff --git a/editors/sed.c b/editors/sed.c index f0a3c028f..43395b392 100644 --- a/editors/sed.c +++ b/editors/sed.c @@ -865,6 +865,7 @@ static void process_file(FILE * file) case 'd': altered++; deleted = 1; + force_print = 0; break; case 's': @@ -1031,16 +1032,29 @@ static void process_file(FILE * file) break; case 'g': /* Replace pattern space with hold space */ free(pattern_space); - pattern_space = strdup(hold_space); + if (hold_space) { + pattern_space = strdup(hold_space); + } break; case 'G': { /* Append newline and hold space to pattern space */ - int pattern_space_size = 0; + int pattern_space_size = 2; + int hold_space_size = 0; + if (pattern_space) { - pattern_space_size = strlen(pattern_space); + pattern_space_size += strlen(pattern_space); + } + if (hold_space) { + hold_space_size = strlen(hold_space); + } + pattern_space = xrealloc(pattern_space, pattern_space_size + hold_space_size); + if (pattern_space_size == 2) { + strcat(pattern_space, "\n"); + } else { + strcpy(pattern_space, "\n"); + } + if (hold_space) { + strcat(pattern_space, hold_space); } - pattern_space = xrealloc(pattern_space, pattern_space_size + strlen(hold_space) + 2); - strcat(pattern_space, "\n"); - strcat(pattern_space, hold_space); break; } case 'h': /* Replace hold space with pattern space */