Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
MIME-Version: 1.0
From: Sebastian Gniazdowski <sgniazdowski@gmail.com>
Date: Tue, 16 Feb 2016 08:53:05 +0100
Message-ID: <CAKc7PVDKd1HWgausNPpqzsi0DK+CJp=+LC0tcEa0cDi-4BJmbQ@mail.gmail.com>
Subject: Quoting problems
To: Zsh hackers list <zsh-workers@zsh.org>
Content-Type: multipart/mixed; boundary=001a113db2f69965ef052bde6d79

--001a113db2f69965ef052bde6d79
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hello,
# source test_script > a
# source ./a
./a:156: unmatched "
./a:157: invalid function definition


It's about following line:
[[ $#exp -eq 1 && "${exp[1]//\}" =3D "${word//\}"(|\(N\)) ]]

The first and second backslash aren't quoted and cause the two '{' to
be unbound. Other quoting styles (q, qqq, qqqq) also fail in general
(with e.g. parse errors, also with an 'unmatched "'). I think that
quoting means: content can be anything, special characters will be
taken care of =E2=80=93 so this is a problem.

The test script is:

# To be sourced
for k in _expand; do
    echo "if [[ \"\${functions[$k]}\" !=3D ${(qq)functions[$k]} ]]; then
        functions[$k]=3D${(qq)functions[$k]}
    fi"
done

I also attach short version of the result:
if [[ "${functions[_expand]}" !=3D '      setopt localoptions nonomatch' ]]=
; then
        functions[_expand]=3D'[[ $#exp -eq 1 && "${exp[1]//\}" =3D
"${word//\}"(|\(N\)) ]]'
fi

- sourcing this will fail because of the unquoted backslash.


Best regards,
Sebastian Gniazdowski

--001a113db2f69965ef052bde6d79
Content-Type: application/octet-stream; name=test_script
Content-Disposition: attachment; filename=test_script
Content-Transfer-Encoding: base64
X-Attachment-Id: f_ikp3qzq40

IyBUbyBiZSBzb3VyY2VkCmZvciBrIGluIF9leHBhbmQ7IGRvCiAgICBlY2hvICJpZiBbWyBcIlwk
e2Z1bmN0aW9uc1ska119XCIgIT0gJHsocXEpZnVuY3Rpb25zWyRrXX0gXV07IHRoZW4KICAgICAg
ICBmdW5jdGlvbnNbJGtdPSR7KHFxKWZ1bmN0aW9uc1ska119CiAgICBmaSIKZG9uZQo=
--001a113db2f69965ef052bde6d79
Content-Type: application/octet-stream; name="snap-2016.02.16_07:52:44"
Content-Disposition: attachment; filename="snap-2016.02.16_07:52:44"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_ikp3qzqo1

aWYgW1sgIiR7ZnVuY3Rpb25zW19leHBhbmRdfSIgIT0gJwlzZXRvcHQgbG9jYWxvcHRpb25zIG5v
bm9tYXRjaCcgXV07IHRoZW4KICAgICAgICBmdW5jdGlvbnNbX2V4cGFuZF09J1tbICQjZXhwIC1l
cSAxICYmICIke2V4cFsxXS8vXH0iID0gIiR7d29yZC8vXH0iKHxcKE5cKSkgXV0nCiAgICBmaQo=
--001a113db2f69965ef052bde6d79--