Bug 484348

Summary: Error parsing $(...)
Product: Red Hat Enterprise Linux 5 Reporter: Tomas Smetana <tsmetana>
Component: zshAssignee: James Antill <james.antill>
Status: CLOSED ERRATA QA Contact: James Antill <james.antill>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.1CC: mfabry, syeghiay, tao
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 496791 (view as bug list) Environment:
Last Closed: 2009-05-06 11:31:42 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Description Flags
Proposed patch
Improved patch
Even more improved patch... none

Description Tomas Smetana 2009-02-06 06:25:20 EST
Description of problem:
Consider the following script:

# Comment containing '
echo a
# Comment containing '
echo $VAR

The script is syntactically correct but zsh throws an error executing it.

Version-Release number of selected component (if applicable):
zsh-4.2.6-2 (reproduced also with Fedora's  zsh-4.3.6-5 version)

How reproducible:

Steps to Reproduce:
1. run the script above
Actual results:
parse error near `VAR=$('

Expected results:

Additional info:
ksh and bash are able to run the script without error.  No error appears when the $(...) command substitution is replaced with `...`.  I think there's a bug in parsing the content in the round brackets: the '\'' character is treated as special even when it appears in a comment.
Comment 1 Tomas Smetana 2009-02-06 06:31:22 EST
Created attachment 331118 [details]
Proposed patch

Change in the (...) parsing function: if the '#' is found, all the characters until the newline lose their special meaning.
Comment 7 Tomas Smetana 2009-02-26 04:01:02 EST
Patch has been sent upstream: http://www.zsh.org/mla/workers/2009/msg00396.html
Comment 8 Tomas Smetana 2009-02-27 04:34:21 EST
Created attachment 333448 [details]
Improved patch

The previous patch didn't really care about the context of the '#' character which might have caused problems in cases where it was not meant as beginning of a comment.

I'm attaching the version that was committed upstream: http://zsh.cvs.sourceforge.net/viewvc/zsh/zsh/Src/lex.c?r1=1.49&r2=1.50
Comment 11 Tomas Smetana 2009-03-02 06:09:41 EST
Created attachment 333724 [details]
Even more improved patch...

There was still a small problem with the latest patch:  http://www.zsh.org/mla/workers/2009/msg00408.html. (upstream lex.c: http://zsh.cvs.sourceforge.net/viewvc/zsh/zsh/Src/lex.c?revision=1.51&view=markup#l_1799)
Comment 16 errata-xmlrpc 2009-05-06 11:31:42 EDT
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.