]> Cypherpunks repositories - gostls13.git/commitdiff
exec: closeAfterWait append bug
authorBrad Fitzpatrick <bradfitz@golang.org>
Mon, 11 Jul 2011 21:46:46 +0000 (14:46 -0700)
committerBrad Fitzpatrick <bradfitz@golang.org>
Mon, 11 Jul 2011 21:46:46 +0000 (14:46 -0700)
Wasn't actually eager closing the fds as a result of the
copy/paste bug. (GC was doing it instead)

R=rsc
CC=golang-dev
https://golang.org/cl/4671057

src/pkg/exec/exec.go

index 5b988d5eb41e113d0f03379749fbbea23480989b..4ddefae24ea2a584c37aa2fc3ea1cba688b34bd0 100644 (file)
@@ -332,7 +332,7 @@ func (c *Cmd) StdinPipe() (io.WriteCloser, os.Error) {
        }
        c.Stdin = pr
        c.closeAfterStart = append(c.closeAfterStart, pr)
-       c.closeAfterWait = append(c.closeAfterStart, pw)
+       c.closeAfterWait = append(c.closeAfterWait, pw)
        return pw, nil
 }
 
@@ -351,7 +351,7 @@ func (c *Cmd) StdoutPipe() (io.Reader, os.Error) {
        }
        c.Stdout = pw
        c.closeAfterStart = append(c.closeAfterStart, pw)
-       c.closeAfterWait = append(c.closeAfterStart, pr)
+       c.closeAfterWait = append(c.closeAfterWait, pr)
        return pr, nil
 }
 
@@ -370,6 +370,6 @@ func (c *Cmd) StderrPipe() (io.Reader, os.Error) {
        }
        c.Stderr = pw
        c.closeAfterStart = append(c.closeAfterStart, pw)
-       c.closeAfterWait = append(c.closeAfterStart, pr)
+       c.closeAfterWait = append(c.closeAfterWait, pr)
        return pr, nil
 }