Mercurial > hg > toybox
view tests/bzcat.test @ 1613:96aa7ec74936 draft
Fix yet another sed bug.
The s/// command would copy the \ of substitutions before deciding what to
do with them (generally overwriting the \ with the new data). When the
substitution was A) at the very end of the new string, B) resolved to nothing,
it could leave a trailing \ that didn't belong there and didn't get overwritten
because the "copy trailing data" part that copies the original string's null
terminator already happened before the \ overwrote it.
The ghostwheel() function restarts regexes after embedded NUL bytes, but
if the string it's passed is _longer_ than the length it's told then it
gets confused (and it means we're off the end of our allocation so segfaults
are likely).
Fix: test for \ first and move the "copy byte" logic into an else case.
author | Rob Landley <rob@landley.net> |
---|---|
date | Mon, 15 Dec 2014 03:34:55 -0600 |
parents | 8700cbe1cb29 |
children |
line wrap: on
line source
#!/bin/bash # Copyright 2014 Divya Kothari <divya.s.kothari@gmail.com> # Copyright 2014 Naha Maggu <maggu.neha@gmail.com> [ -f testing.sh ] && . testing.sh #testing "name" "command" "result" "infile" "stdin" echo "hello" > file tar -cjf file.tar.bz2 file # Get system bzcat bzcatExe=`which bzcat` $bzcatExe file.tar.bz2 > bzcatOut testing "bzcat - decompresses a single file" "bzcat file.tar.bz2 > Tempfile && echo "yes"; diff Tempfile bzcatOut && echo "yes"; rm -rf file* bzcatOut Tempfile" "yes\nyes\n" "" "" #testing "name" "command" "result" "infile" "stdin" echo "hello" > file1 echo "hi" > file2 echo "Hi, Good morning !! I am a bzcat tester" > file3 tar -cjf file1.tar.bz2 file1 tar -cjf file2.tar.bz2 file2 tar -cjf file3.tar.bz2 file3 # Get system bzcat bzcatExe=`which bzcat` $bzcatExe file1.tar.bz2 file2.tar.bz2 file3.tar.bz2 > bzcatOut testing "bzcat - decompresses multiple files" "bzcat file1.tar.bz2 file2.tar.bz2 file3.tar.bz2 > Tempfile && echo "yes" ; diff Tempfile bzcatOut && echo "yes"; rm -rf file* bzcatOut Tempfile " "yes\nyes\n" "" ""