Path: csiph.com!news.mixmin.net!news.linkpendium.com!news.linkpendium.com!panix!usenet.stanford.edu!not-for-mail
From: Webstrand <webstrand@gmail.com>
Newsgroups: gnu.bash.bug
Subject: Bug when using pipefail and lastpipe together in bash-4.4
Date: Tue, 8 Oct 2019 09:43:23 -0400
Lines: 69
Approved: bug-bash@gnu.org
Message-ID: <mailman.1368.1570545550.2651.bug-bash@gnu.org>
References: <CACm1TQebG+cnYuxqFSU7uu29XczQdRiO_M1RW-ne3isZkwbwsg@mail.gmail.com>
NNTP-Posting-Host: lists.gnu.org
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="000000000000d3d4c70594665a3f"
To: bug-bash@gnu.org
Envelope-to: bug-bash@gnu.org
Precedence: list
Xref: csiph.com gnu.bash.bug:15485

--000000000000d3d4c70594665a3f
Content-Type: text/plain; charset="UTF-8"

When running the following script under bash-4.4.0 to bash-4.4.23 it
seems to fail erroneously on read.

  set -x
  set -o pipefail
  shopt -s lastpipe

  main() {
    # Any input piped into while such that the condition
    # succeeds at least once.
    echo "x" | while read; do
      echo | read # anything piped to read
      sleep 0     # any non-builtin
    done
  }

  main || echo "main failed"

This produces the xtrace:

  + set -o pipefail
  + shopt -s lastpipe
  + main
  + read
  + echo x
  + read
  + echo
  + sleep 0
  + read
  + echo 'main failed'
  main failed

Under bash-4.3 and bash-5.0 the script does not output "main failed",
as expected.

I'm not sure if this is a known bug. One user on IRC reported finding
the same issue on debian stretch, but another could not reproduce the
issue for bash-4.4.12.

I have bisected the history of the devel git branch and found that the commit:
  85ec0778f9d778e1820fb8c0e3e996f2d1103b45 commit bash-20150417 snapshot
introduces the issue.

Applying the attached patch to a bash-4.4.23 source tree produces an
interpreter which does not output "main failed" when running the
aforementioned script.

--000000000000d3d4c70594665a3f
Content-Type: text/x-patch; charset="US-ASCII"; name="Revert-job-tracking-changes.patch"
Content-Disposition: attachment; 
	filename="Revert-job-tracking-changes.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_k1hw1gw30>
X-Attachment-Id: f_k1hw1gw30

ZGlmZiAtLWdpdCBhL2pvYnMuYyBiL2pvYnMuYwppbmRleCBkNjliOGQ4ZC4uZWMwMTRkMWYgMTAw
NjQ0Ci0tLSBhL2pvYnMuYworKysgYi9qb2JzLmMKQEAgLTEwNDMsOSArMTA0MywxMCBAQCBkZWxl
dGVfam9iIChqb2JfaW5kZXgsIGRmbGFncykKICAgaWYgKHRlbXAgPT0gMCkKICAgICByZXR1cm47
CiAKLSAgaWYgKChkZmxhZ3MgJiBERUxfTk9CR1BJRCkgPT0gMCAmJiAodGVtcC0+ZmxhZ3MgJiAo
Sl9BU1lOQ3xKX0ZPUkVHUk9VTkQpKSA9PSBKX0FTWU5DKQorICBpZiAoKGRmbGFncyAmIERFTF9O
T0JHUElEKSA9PSAwKQogICAgIHsKICAgICAgIHByb2MgPSBmaW5kX2xhc3RfcHJvYyAoam9iX2lu
ZGV4LCAwKTsKKyAgICAgIC8qIENvdWxkIGRvIHRoaXMganVzdCBmb3IgSl9BU1lOQyBqb2JzLCBi
dXQgd2Ugc2F2ZSBhbGwuICovCiAgICAgICBpZiAocHJvYykKIAliZ3BfYWRkIChwcm9jLT5waWQs
IHByb2Nlc3NfZXhpdF9zdGF0dXMgKHByb2MtPnN0YXR1cykpOwogICAgIH0K
--000000000000d3d4c70594665a3f--