Browse Source

Merge pull request #40144 from lzhfromustc/GL_outputDone

integration/internal/container: fix a goroutine leak bug
Kirill Kolyshkin 5 years ago
parent
commit
154cf042fd
1 changed files with 1 additions and 1 deletions
  1. 1 1
      integration/internal/container/exec.go

+ 1 - 1
integration/internal/container/exec.go

@@ -57,7 +57,7 @@ func Exec(ctx context.Context, cli client.APIClient, id string, cmd []string) (E
 
 
 	// read the output
 	// read the output
 	var outBuf, errBuf bytes.Buffer
 	var outBuf, errBuf bytes.Buffer
-	outputDone := make(chan error)
+	outputDone := make(chan error, 1)
 
 
 	go func() {
 	go func() {
 		// StdCopy demultiplexes the stream into two buffers
 		// StdCopy demultiplexes the stream into two buffers