alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Merge branch 'bugfix/update_incorrect_refereing_of_function_esp_https_ota_get_img_desc' into 'master'

Browse Source 
fix(esp_https_ota): fix incorrect references of API esp_https_ota_get_img_desc()

See merge request espressif/esp-idf!28891
This commit is contained in:
Mahavir Jain 2024-02-07 12:05:58 +08:00
commit e0dd917d31
3 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
components/esp_https_ota/src/esp_https_ota.c
View File

@ -1,5 +1,5 @@
/* /*
* SPDX-FileCopyrightText: 2017-2023 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2017-2024 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: Apache-2.0 * SPDX-License-Identifier: Apache-2.0
*/ */
@ -451,11 +451,11 @@ esp_err_t esp_https_ota_get_img_desc(esp_https_ota_handle_t https_ota_handle, es
esp_https_ota_t *handle = (esp_https_ota_t *)https_ota_handle; esp_https_ota_t *handle = (esp_https_ota_t *)https_ota_handle;
if (handle == NULL || new_app_info == NULL) { if (handle == NULL || new_app_info == NULL) {
ESP_LOGE(TAG, "esp_https_ota_read_img_desc: Invalid argument"); ESP_LOGE(TAG, "esp_https_ota_get_img_desc: Invalid argument");
return ESP_ERR_INVALID_ARG; return ESP_ERR_INVALID_ARG;
} }
if (handle->state < ESP_HTTPS_OTA_BEGIN) { if (handle->state < ESP_HTTPS_OTA_BEGIN) {
ESP_LOGE(TAG, "esp_https_ota_read_img_desc: Invalid state"); ESP_LOGE(TAG, "esp_https_ota_get_img_desc: Invalid state");
return ESP_ERR_INVALID_STATE; return ESP_ERR_INVALID_STATE;
} }
if (read_header(handle) != ESP_OK) { if (read_header(handle) != ESP_OK) {
@ -508,7 +508,7 @@ esp_err_t esp_https_ota_perform(esp_https_ota_handle_t https_ota_handle)
return err; return err;
} }
handle->state = ESP_HTTPS_OTA_IN_PROGRESS; handle->state = ESP_HTTPS_OTA_IN_PROGRESS;
/* In case `esp_https_ota_read_img_desc` was invoked first, /* In case `esp_https_ota_get_img_desc` was invoked first,
then the image data read there should be written to OTA partition then the image data read there should be written to OTA partition
*/ */
int binary_file_len = 0; int binary_file_len = 0;

2
examples/system/ota/advanced_https_ota/main/advanced_https_ota_example.c
View File

@ -168,7 +168,7 @@ void advanced_ota_example_task(void *pvParameter)
esp_app_desc_t app_desc; esp_app_desc_t app_desc;
err = esp_https_ota_get_img_desc(https_ota_handle, &app_desc); err = esp_https_ota_get_img_desc(https_ota_handle, &app_desc);
if (err != ESP_OK) { if (err != ESP_OK) {
ESP_LOGE(TAG, "esp_https_ota_read_img_desc failed"); ESP_LOGE(TAG, "esp_https_ota_get_img_desc failed");
goto ota_end; goto ota_end;
} }
err = validate_image_header(&app_desc); err = validate_image_header(&app_desc);

2
examples/system/ota/advanced_https_ota/pytest_advanced_ota.py
View File

@ -225,7 +225,7 @@ def test_examples_protocol_advanced_https_ota_example_truncated_header(dut: Dut)
dut.expect('Starting Advanced OTA example', timeout=30) dut.expect('Starting Advanced OTA example', timeout=30)
print('writing to device: {}'.format('https://' + host_ip + ':' + str(server_port) + '/' + truncated_bin_name)) print('writing to device: {}'.format('https://' + host_ip + ':' + str(server_port) + '/' + truncated_bin_name))
dut.write('https://' + host_ip + ':' + str(server_port) + '/' + truncated_bin_name) dut.write('https://' + host_ip + ':' + str(server_port) + '/' + truncated_bin_name)
dut.expect('advanced_https_ota_example: esp_https_ota_read_img_desc failed', timeout=30) dut.expect('advanced_https_ota_example: esp_https_ota_get_img_desc failed', timeout=30)
try: try:
os.remove(binary_file) os.remove(binary_file)
except OSError: except OSError: