From 77f60304f5d81e9ba1a9c91d4f6da8e7b6b6513c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?King=20K=C3=A9vin?= Date: Mon, 24 Oct 2022 18:27:47 +0200 Subject: [PATCH] fix complete_task exit --- examples/device/dfu_freertos/src/main.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/examples/device/dfu_freertos/src/main.c b/examples/device/dfu_freertos/src/main.c index a2ada19a4..3918a9ead 100644 --- a/examples/device/dfu_freertos/src/main.c +++ b/examples/device/dfu_freertos/src/main.c @@ -186,7 +186,7 @@ static void complete_task(void* arg) if (ESP_OK != rc) { ESP_LOGE(TAG, "close OTA failed"); tud_dfu_finish_flashing(DFU_STATUS_ERR_PROG); - return; + vTaskDelete(NULL); // close task } } @@ -197,16 +197,13 @@ static void complete_task(void* arg) if (ESP_APP_DESC_MAGIC_WORD == ota_desc.magic_word) { ESP_LOGI(TAG, "set boot to valid app"); esp_ota_set_boot_partition(ota); + xTimerChangePeriod(blinky_tm, pdMS_TO_TICKS(BLINK_IDLE), 0); + tud_dfu_finish_flashing(DFU_STATUS_OK); } else { ESP_LOGI(TAG, "no valid app"); tud_dfu_finish_flashing(DFU_STATUS_ERR_VERIFY); - return; } - // flashing op for manifest is complete without error - xTimerChangePeriod(blinky_tm, pdMS_TO_TICKS(BLINK_IDLE), 0); - tud_dfu_finish_flashing(DFU_STATUS_OK); - vTaskDelete(NULL); // close task }