Add some assertions

Girgias · Girgias · commit 8f4c2c50e97d · 2022-05-11T23:00:15.000+01:00
diff --git a/sapi/cli/php_cli_server.c b/sapi/cli/php_cli_server.c
@@ -1898,7 +1898,11 @@ static void php_cli_server_client_dtor(php_cli_server_client *client) /* {{{ */
 	}
 	pefree(client->addr, 1);
 	zend_string_release_ex(client->addr_str, /* persistent */ true);
-	// TODO release client->current_header_name ?
+
+	/* Headers must be sent */
+	assert(client->current_header_name == NULL);
+	assert(client->current_header_value == NULL);
+
 	if (client->content_sender_initialized) {
 		php_cli_server_content_sender_dtor(&client->content_sender);
 	}

Original file line number	Diff line number	Diff line change
`@@ -1898,7 +1898,11 @@ static void php_cli_server_client_dtor(php_cli_server_client client) / {{{ */`
`1898`	`1898`	`}`
`1899`	`1899`	`pefree(client->addr, 1);`
`1900`	`1900`	`zend_string_release_ex(client->addr_str, /* persistent */ true);`
`1901`		`- // TODO release client->current_header_name ?`
	`1901`	`+`
	`1902`	`+ /* Headers must be sent */`
	`1903`	`+ assert(client->current_header_name == NULL);`
	`1904`	`+ assert(client->current_header_value == NULL);`
	`1905`	`+`
`1902`	`1906`	`if (client->content_sender_initialized) {`
`1903`	`1907`	`php_cli_server_content_sender_dtor(&client->content_sender);`
`1904`	`1908`	`}`