Rewriting some _old_ code around the handlePASS method

It seems to me it never worked like it should have.
fclairamb · Feb 25, 2024 · c0d4f26 · c0d4f26
1 parent 4054ae2
commit c0d4f26
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/handle_auth.go b/handle_auth.go
@@ -66,22 +66,22 @@ func (c *clientHandler) handlePASS(param string) error {
 	}
 
 	switch {
-	case err == nil:
-		if msg == "" {
-			msg = "Password ok, continue"
-		}
-
-		c.writeMessage(StatusUserLoggedIn, msg)
+	case err == nil && c.driver == nil:
+		c.writeMessage(StatusNotLoggedIn, "Unexpected exception (driver is nil)")
+		c.disconnect()
 	case err != nil:
 		if msg == "" {
 			msg = fmt.Sprintf("Authentication error: %v", err)
 		}
 
 		c.writeMessage(StatusNotLoggedIn, msg)
 		c.disconnect()
-	default:
-		c.writeMessage(StatusNotLoggedIn, "Unexpected exception (driver is nil)")
-		c.disconnect()
+	case err == nil:
+		if msg == "" {
+			msg = "Password ok, continue"
+		}
+
+		c.writeMessage(StatusUserLoggedIn, msg)
 	}
 
 	return nil