ethernetif: fix potential memory leak

1. If L2_TO_L3_RX_BUF_MODE is not selected, we must assign l2_owner explictly before we call pbuf_free.
2. free intr resource in esp_eth_deinit

Closes https://github.com/espressif/esp-idf/issues/2670

diff --git a/components/ethernet/test/test_emac_deinit.c b/components/ethernet/test/test_emac_deinit.c
index 370f7975b06b45b9b1aafb8ddf2dce2071ab9778..3b540728a64433b75d6c6a3afe8292afbfc92a44 100644 (file)
--- a/components/ethernet/test/test_emac_deinit.c
+++ b/components/ethernet/test/test_emac_deinit.c
@@ -7,17 +7,19 @@
  * @author morris
  * @date 2018-08-24
  */
+#include <stdio.h>
 #include <string.h>
-#include "unity.h"
+
 #include "freertos/FreeRTOS.h"
 #include "freertos/task.h"
-
+#include "freertos/event_groups.h"
 #include "esp_system.h"
 #include "esp_err.h"
 #include "esp_event_loop.h"
 #include "esp_event.h"
 #include "esp_log.h"
 #include "esp_eth.h"
+#include "unity.h"
 
 #include "rom/gpio.h"
 
@@ -37,6 +39,9 @@ static const char *TAG = "eth_test_deinit";
 #define CONFIG_PHY_ADDRESS 31
 #define CONFIG_PHY_CLOCK_MODE 0
 
+static EventGroupHandle_t eth_event_group = NULL;
+static const int GOTIP_BIT = BIT0;
+
 static void phy_device_power_enable_via_gpio(bool enable)
 {
     if (!enable) {
@@ -67,6 +72,47 @@ static void eth_gpio_config_rmii(void)
     phy_rmii_smi_configure_pins(PIN_SMI_MDC, PIN_SMI_MDIO);
 }
 
+static esp_err_t eth_event_handler(void *ctx, system_event_t *event)
+{
+    tcpip_adapter_ip_info_t ip;
+
+    switch (event->event_id) {
+    case SYSTEM_EVENT_ETH_CONNECTED:
+        ESP_LOGI(TAG, "Ethernet Link Up");
+        break;
+    case SYSTEM_EVENT_ETH_DISCONNECTED:
+        ESP_LOGI(TAG, "Ethernet Link Down");
+        break;
+    case SYSTEM_EVENT_ETH_START:
+        ESP_LOGI(TAG, "Ethernet Started");
+        break;
+    case SYSTEM_EVENT_ETH_GOT_IP:
+        memset(&ip, 0, sizeof(tcpip_adapter_ip_info_t));
+        ESP_ERROR_CHECK(tcpip_adapter_get_ip_info(ESP_IF_ETH, &ip));
+        ESP_LOGI(TAG, "Ethernet Got IP Addr");
+        ESP_LOGI(TAG, "~~~~~~~~~~~");
+        ESP_LOGI(TAG, "ETHIP:" IPSTR, IP2STR(&ip.ip));
+        ESP_LOGI(TAG, "ETHMASK:" IPSTR, IP2STR(&ip.netmask));
+        ESP_LOGI(TAG, "ETHGW:" IPSTR, IP2STR(&ip.gw));
+        ESP_LOGI(TAG, "~~~~~~~~~~~");
+        xEventGroupSetBits(eth_event_group, GOTIP_BIT);
+        break;
+    case SYSTEM_EVENT_ETH_STOP:
+        ESP_LOGI(TAG, "Ethernet Stopped");
+        break;
+    default:
+        break;
+    }
+    return ESP_OK;
+}
+
+TEST_CASE("start event loop", "[ethernet][ignore]")
+{
+    eth_event_group = xEventGroupCreate();
+    tcpip_adapter_init();
+    ESP_ERROR_CHECK(esp_event_loop_init(eth_event_handler, NULL));
+}
+
 TEST_CASE("test emac deinit", "[ethernet][ignore]")
 {
     eth_config_t config = DEFAULT_ETHERNET_PHY_CONFIG;
@@ -79,7 +125,8 @@ TEST_CASE("test emac deinit", "[ethernet][ignore]")
     ESP_ERROR_CHECK(esp_eth_init(&config));
     ESP_ERROR_CHECK(esp_eth_enable());
 
-    vTaskDelay(2000 / portTICK_RATE_MS);
+    xEventGroupWaitBits(eth_event_group, GOTIP_BIT, true, true, portMAX_DELAY);
+    vTaskDelay(15000 / portTICK_RATE_MS);
 
     ESP_ERROR_CHECK(esp_eth_disable());
     ESP_ERROR_CHECK(esp_eth_deinit());