Jelajahi Sumber

Merge pull request #8877 from erikh/proxy_error_check

proxy: Fix a potential panic handling error states.
Tibor Vass 10 tahun lalu
induk
melakukan
fa59fe997c
1 mengubah file dengan 6 tambahan dan 1 penghapusan
  1. 6 1
      daemon/networkdriver/portmapper/proxy.go

+ 6 - 1
daemon/networkdriver/portmapper/proxy.go

@@ -130,7 +130,12 @@ func (p *proxyCommand) Start() error {
 		r.Read(buf)
 
 		if string(buf) != "0\n" {
-			errStr, _ := ioutil.ReadAll(r)
+			errStr, err := ioutil.ReadAll(r)
+			if err != nil {
+				errchan <- fmt.Errorf("Error reading exit status from userland proxy: %v", err)
+				return
+			}
+
 			errchan <- fmt.Errorf("Error starting userland proxy: %s", errStr)
 			return
 		}