Merge pull request #11 from VadimIpatov/master

saschat · web-flow · commit ae22d23ca7fa · 2020-08-11T12:18:25.000+02:00
fixed possible crash when establishing a connection
diff --git a/server_conn.go b/server_conn.go
@@ -102,8 +102,10 @@ func (sc *serverConn) connect() error {
 		// Error, except if the server doesn't support authentication
 		mErr := err.(*Error)
 		if mErr.Status != StatusUnknownCommand {
-			sc.conn.Close()
-			sc.conn = nil
+			if sc.conn != nil {
+				sc.conn.Close()
+				sc.conn = nil
+			}
 			return err
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -102,8 +102,10 @@ func (sc *serverConn) connect() error {`
`102`	`102`	`// Error, except if the server doesn't support authentication`
`103`	`103`	`mErr := err.(*Error)`
`104`	`104`	`if mErr.Status != StatusUnknownCommand {`
`105`		`- sc.conn.Close()`
`106`		`- sc.conn = nil`
	`105`	`+ if sc.conn != nil {`
	`106`	`+ sc.conn.Close()`
	`107`	`+ sc.conn = nil`
	`108`	`+ }`
`107`	`109`	`return err`
`108`	`110`	`}`
`109`	`111`	`}`