Merge branch 'bugfix/ppp_netif_free' into 'master'

esp-modem: Remove esp-netif destroy from modem network glue code

Closes IDFGH-3534

See merge request espressif/esp-idf!9471

Author: david-cermak

examples/protocols/pppos_client/components/modem/src/esp_modem_netif.c

@@ -109,7 +109,6 @@ void *esp_modem_netif_setup(modem_dte_t *dte)
 void esp_modem_netif_teardown(void *h)
 {
     esp_modem_netif_driver_t *driver = h;
-    esp_netif_destroy(driver->base.netif);
     free(driver);
 }
 

examples/protocols/pppos_client/main/pppos_client_main.c

@@ -320,8 +320,10 @@ void app_main(void)
         vTaskDelay(pdMS_TO_TICKS(60000));
     }
 
-    /* Destroy the netif adapter withe events, which internally frees also the esp-netif instance */
+    /* Unregister events, destroy the netif adapter and destroy its esp-netif instance */
     esp_modem_netif_clear_default_handlers(modem_netif_adapter);
     esp_modem_netif_teardown(modem_netif_adapter);
+    esp_netif_destroy(esp_netif);
+
     ESP_ERROR_CHECK(dte->deinit(dte));
 }

tools/test_apps/protocols/pppos/main/pppos_client_main.c

@@ -258,9 +258,11 @@ void app_main(void)
     ESP_ERROR_CHECK(esp_modem_stop_ppp(dte));
     /* Wait for the PPP connection to terminate gracefully */
     xEventGroupWaitBits(event_group, DISCONNECT_BIT, pdTRUE, pdTRUE, portMAX_DELAY);
-    /* Destroy the netif adapter withe events, which internally frees also the esp-netif instance */
+    /* Unregister events, destroy the netif adapter and destroy its esp-netif instance */
     esp_modem_netif_clear_default_handlers(modem_netif_adapter);
     esp_modem_netif_teardown(modem_netif_adapter);
+    esp_netif_destroy(esp_netif);
+
     xEventGroupWaitBits(event_group, STOP_BIT, pdTRUE, pdTRUE, portMAX_DELAY);
     ESP_ERROR_CHECK(dce->deinit(dce));
     ESP_ERROR_CHECK(dte->deinit(dte));