Bug 1241014

Summary: case in a for loop inside subshell causes syntax error
Product: Red Hat Enterprise Linux 7 Reporter: Martin Kyral <mkyral>
Component: kshAssignee: Siteshwar Vashisht <svashisht>
Status: CLOSED ERRATA QA Contact: Martin Kyral <mkyral>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.1CC: isenfeld, jkejda, ovasik, qe-baseos-apps
Target Milestone: rcKeywords: Patch
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: ksh-20120801-26.el7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1212775 Environment:
Last Closed: 2016-11-04 07:46:19 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1240994    
Attachments:
Description Flags
Proposed patch none

Description Martin Kyral 2015-07-08 10:09:15 UTC
ksh in RHEL 7 has the same issue

Version-Release number of selected component (if applicable):
ksh-20120801-22.el7

+++ This bug was initially created as a clone of Bug #1212775 +++

Description of problem:

# this works
$ x=$(for i in test; do echo $i; done)
$ echo $x
test

# this too, works
$ i=test
$ x=$(case $i in test) echo test;; esac)
$ echo $x
test

# this does not
$ x=$(for i in test; do case $i in test) echo test;; esac; done)
-bash: syntax error near unexpected token `;;'

# however removing the output-into-variable makes it work
$ (for i in test; do case $i in test) echo test;; esac; done)
test


Version-Release number of selected component (if applicable):
bash-4.2.46-12.el7

How reproducible:
always

Actual results:
expression fails on syntax error

Expected results:
the subshell output is assigned to a variable

Additional info:
This *works* on my bash-4.3.30-2.fc21, so there might have been an upstream fix.

--- Additional comment from Ondrej Oprala on 2015-04-18 09:36:45 EDT ---

Hi Jiri,
Yes, this was fixed in 4.3. I have yet to look at the size of the change.

Meanwhile, the more verbose way of writing a case statement still seems to work:
x=$(for i in test; do case $i in (test) echo test;; esac; done)
#                                ^^

Comment 2 Martin Kyral 2015-07-21 08:09:16 UTC
*** Bug 1245094 has been marked as a duplicate of this bug. ***

Comment 3 Ondrej Oprala 2015-08-19 08:32:34 UTC
Created attachment 1064679 [details]
Proposed patch

Proposed patch backported from ksh-20130409 along with a test. Code is the same in ksh-20140929.

Comment 7 errata-xmlrpc 2016-11-04 07:46:19 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2016-2510.html