Convert one more warning and add tests

Author: kamil-tekiela

ext/mysqli/tests/fake_server.inc

@@ -611,6 +611,13 @@ class my_mysqli_fake_server_conn
         $packets = $this->packet_generator->server_query_execute_data_response($field_name);
         $this->send($this->packets_to_bytes($packets), "Query execute data $field_name");
     }
+
+    public function send_too_short_server_ok()
+    {
+        $packet = $this->packet_generator->server_ok();
+        $packet->packet_length = "060000";
+        $this->send($packet->to_bytes(), "Server OK");
+    }
 }
 
 class my_mysqli_fake_server_process
@@ -807,6 +814,18 @@ function my_mysqli_test_query_response_row_read_two_fields(my_mysqli_fake_server
     }
 }
 
+function my_mysqli_test_invalid_greeting_packet(my_mysqli_fake_server_conn $conn): void
+{
+    // Send nothing at all to the client
+}
+
+function my_mysqli_test_ok_packet_too_short(my_mysqli_fake_server_conn $conn): void
+{
+    $conn->send_server_greetings();
+    $conn->read();
+    $conn->send_too_short_server_ok();
+}
+
 function run_fake_server(string $test_function, $port = 33305): void
 {
     $address = '127.0.0.1';

ext/mysqli/tests/wireprotocol_errors/AUTH_RESPONSE_packet_is_shorter_than_expected.phpt

@@ -0,0 +1,36 @@
+--TEST--
+AUTH_RESPONSE packet is shorter than expected
+--EXTENSIONS--
+mysqli
+--FILE--
+<?php
+require_once __DIR__ . '/../fake_server.inc';
+
+$port = 50001;
+$servername = "127.0.0.1";
+$username = "root";
+$password = "";
+
+$process = run_fake_server_in_background('ok_packet_too_short', $port);
+$process->wait();
+
+try {
+    $conn = new mysqli( $servername, $username, $password, "", $port );
+} catch (Exception $e) {
+    echo 'ERROR: ' . $e->getMessage() . PHP_EOL;
+}
+
+$process->terminate();
+
+print "done!";
+?>
+--EXPECTF--
+[*] Server started
+[*] Connection established
+[*] Sending - Server Greeting: 580000000a352e352e352d31302e352e31382d4d6172696144420003000000473e3f6047257c6700fef7080200ff81150000000000000f0000006c6b55463f49335f686c6431006d7973716c5f6e61746976655f70617373776f7264
+[*] Received: 6900000185a21a00000000c0080000000000000000000000000000000000000000000000726f6f7400006d7973716c5f6e61746976655f70617373776f7264002c0c5f636c69656e745f6e616d65076d7973716c6e640c5f7365727665725f686f7374093132372e302e302e31
+[*] Sending - Server OK: 0600000200000002000000
+
+Warning: mysqli::__construct(): Premature end of data (mysqlnd_wireprotocol.c:732) in %s
+ERROR: AUTH_RESPONSE packet is shorter than expected
+done!

ext/mysqli/tests/wireprotocol_errors/error_while_reading_greeting_packet.phpt

@@ -0,0 +1,31 @@
+--TEST--
+Error while reading greeting packet
+--EXTENSIONS--
+mysqli
+--FILE--
+<?php
+require_once __DIR__ . '/../fake_server.inc';
+
+$port = 50001;
+$servername = "127.0.0.1";
+$username = "root";
+$password = "";
+
+$process = run_fake_server_in_background('invalid_greeting_packet', $port);
+$process->wait();
+
+try {
+    $conn = new mysqli( $servername, $username, $password, "", $port );
+} catch (Exception $e) {
+    echo 'ERROR: ' . $e->getMessage() . PHP_EOL;
+}
+
+$process->terminate();
+
+print "done!";
+?>
+--EXPECTF--
+[*] Server started
+[*] Connection established
+ERROR: Error while reading greeting packet
+done!

ext/mysqlnd/mysqlnd_commands.c

@@ -594,8 +594,7 @@ MYSQLND_METHOD(mysqlnd_command, handshake)(MYSQLND_CONN_DATA * const conn, const
 	conn->payload_decoder_factory->m.init_greet_packet(&greet_packet);
 
 	if (FAIL == PACKET_READ(conn, &greet_packet)) {
-		DBG_ERR("Error while reading greeting packet");
-		php_error_docref(NULL, E_WARNING, "Error while reading greeting packet. PID=%d", getpid());
+		SET_CLIENT_ERROR(conn->error_info, CR_MALFORMED_PACKET, UNKNOWN_SQLSTATE, "Error while reading greeting packet");
 		goto err;
 	} else if (greet_packet.error_no) {
 		DBG_ERR_FMT("errorno=%u error=%s", greet_packet.error_no, greet_packet.error);