fix(esp_http_server): updated condition to verify http version

Closes https://github.com/espressif/esp-idf/issues/14723
This commit is contained in:
nilesh.kale
2024-10-15 17:25:03 +05:30
parent e546409ae5
commit 63afc49584

View File

@ -1,5 +1,5 @@
/*
* SPDX-FileCopyrightText: 2018-2021 Espressif Systems (Shanghai) CO LTD
* SPDX-FileCopyrightText: 2018-2024 Espressif Systems (Shanghai) CO LTD
*
* SPDX-License-Identifier: Apache-2.0
*/
@ -83,7 +83,7 @@ static esp_err_t verify_url (http_parser *parser)
ESP_LOGD(TAG, LOG_FMT("received URI = %s"), r->uri);
/* Make sure version is HTTP/1.1 */
if ((parser->http_major != 1) && (parser->http_minor != 1)) {
if (!((parser->http_major == 1) && (parser->http_minor == 1))) {
ESP_LOGW(TAG, LOG_FMT("unsupported HTTP version = %d.%d"),
parser->http_major, parser->http_minor);
parser_data->error = HTTPD_505_VERSION_NOT_SUPPORTED;
@ -103,7 +103,7 @@ static esp_err_t verify_url (http_parser *parser)
}
/* http_parser callback on finding url in HTTP request
* Will be invoked ATLEAST once every packet
* Will be invoked AT LEAST once every packet
*/
static esp_err_t cb_url(http_parser *parser,
const char *at, size_t length)
@ -188,7 +188,7 @@ static size_t continue_parsing(http_parser *parser, size_t length)
}
/* http_parser callback on header field in HTTP request
* May be invoked ATLEAST once every header field
* May be invoked AT LEAST once every header field
*/
static esp_err_t cb_header_field(http_parser *parser, const char *at, size_t length)
{
@ -247,7 +247,7 @@ static esp_err_t cb_header_field(http_parser *parser, const char *at, size_t len
}
/* http_parser callback on header value in HTTP request.
* May be invoked ATLEAST once every header value
* May be invoked AT LEAST once every header value
*/
static esp_err_t cb_header_value(http_parser *parser, const char *at, size_t length)
{