From 73ca459a56ef003ad0892ef46454bc98afd30a05 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Mon, 13 Jul 2015 18:17:24 -0600 Subject: os/exec: ignore pipe write errors when command completes successfully Fixes #9173 Change-Id: I83530533db84b07cb88dbf6ec690be48a06a9d7d Reviewed-on: https://go-review.googlesource.com/12152 Reviewed-by: Ian Lance Taylor --- src/os/exec/exec.go | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'src/os/exec/exec.go') diff --git a/src/os/exec/exec.go b/src/os/exec/exec.go index a263795814..e3c6fb62b1 100644 --- a/src/os/exec/exec.go +++ b/src/os/exec/exec.go @@ -180,6 +180,16 @@ func (c *Cmd) stdin() (f *os.File, err error) { c.closeAfterWait = append(c.closeAfterWait, pw) c.goroutine = append(c.goroutine, func() error { _, err := io.Copy(pw, c.Stdin) + + // Ignore EPIPE errors copying to stdin if the program + // completed successfully otherwise. + // See Issue 9173. + if pe, ok := err.(*os.PathError); ok && + pe.Op == "write" && pe.Path == "|1" && + pe.Err == syscall.EPIPE { + err = nil + } + if err1 := pw.Close(); err == nil { err = err1 } -- cgit v1.3