From 4e3f71a68219ccf12dfa213a4afbacf684873812 Mon Sep 17 00:00:00 2001 From: "wangtao@espressif.com" Date: Thu, 7 May 2026 16:38:07 +0800 Subject: [PATCH] fix(lwip): improve lost ip event process --- components/esp_netif/lwip/esp_netif_lwip.c | 19 ++++++++++++------- components/esp_wifi/src/wifi_default_ap.c | 3 --- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/components/esp_netif/lwip/esp_netif_lwip.c b/components/esp_netif/lwip/esp_netif_lwip.c index a7188633e97..5a78b9de273 100644 --- a/components/esp_netif/lwip/esp_netif_lwip.c +++ b/components/esp_netif/lwip/esp_netif_lwip.c @@ -1281,7 +1281,7 @@ esp_err_t esp_netif_receive(esp_netif_t *esp_netif, void *buffer, size_t len, vo } #if CONFIG_LWIP_IPV4 -static esp_err_t esp_netif_start_ip_lost_timer(esp_netif_t *esp_netif); +static esp_err_t esp_netif_start_ip_lost_timer(esp_netif_t *esp_netif, bool renew); // // DHCP: @@ -1344,7 +1344,7 @@ static void esp_netif_internal_dhcpc_cb(struct netif *netif) } } else { if (!ip4_addr_cmp(&ip_info->ip, IP4_ADDR_ANY4)) { - esp_netif_start_ip_lost_timer(esp_netif); + esp_netif_start_ip_lost_timer(esp_netif, false); } } } @@ -1384,7 +1384,7 @@ static void esp_netif_ip_lost_timer(void *arg) } } -static esp_err_t esp_netif_start_ip_lost_timer(esp_netif_t *esp_netif) +static esp_err_t esp_netif_start_ip_lost_timer(esp_netif_t *esp_netif, bool renew) { esp_netif_ip_info_t *ip_info_old = esp_netif->ip_info; struct netif *netif = esp_netif->lwip_netif; @@ -1393,7 +1393,12 @@ static esp_err_t esp_netif_start_ip_lost_timer(esp_netif_t *esp_netif) if ( netif && (CONFIG_ESP_NETIF_IP_LOST_TIMER_INTERVAL > 0)) { if (esp_netif->timer_running) { - sys_untimeout(esp_netif_ip_lost_timer, (void *)esp_netif); + if (renew) { + sys_untimeout(esp_netif_ip_lost_timer, (void *)esp_netif); + } else{ + ESP_LOGD(TAG, "if%p start ip lost tmr: already started", esp_netif); + return ESP_OK; + } } esp_netif->timer_running = true; sys_timeout(CONFIG_ESP_NETIF_IP_LOST_TIMER_INTERVAL * 1000, esp_netif_ip_lost_timer, (void *)esp_netif); @@ -1424,7 +1429,7 @@ static esp_err_t esp_netif_dhcpc_stop_api(esp_netif_api_msg_t *msg) if (p_netif != NULL) { dhcp_stop(p_netif); esp_netif_reset_ip_info(esp_netif); - esp_netif_start_ip_lost_timer(esp_netif); + esp_netif_start_ip_lost_timer(esp_netif, false); } else { ESP_LOGD(TAG, "dhcp client if not ready"); return ESP_ERR_ESP_NETIF_IF_NOT_READY; @@ -1487,7 +1492,7 @@ static esp_err_t esp_netif_dhcpc_start_api(esp_netif_api_msg_t *msg) ip_addr_set_zero(&p_netif->ip_addr); ip_addr_set_zero(&p_netif->netmask); ip_addr_set_zero(&p_netif->gw); - esp_netif_start_ip_lost_timer(esp_netif); + esp_netif_start_ip_lost_timer(esp_netif, true); } else { ESP_LOGD(TAG, "dhcp client re init"); esp_netif->dhcpc_status = ESP_NETIF_DHCP_INIT; @@ -1738,7 +1743,7 @@ static esp_err_t esp_netif_down_api(esp_netif_api_msg_t *msg) if (esp_netif->flags & ESP_NETIF_DHCP_CLIENT) { #if CONFIG_LWIP_IPV4 - esp_netif_start_ip_lost_timer(esp_netif); + esp_netif_start_ip_lost_timer(esp_netif, false); #endif } diff --git a/components/esp_wifi/src/wifi_default_ap.c b/components/esp_wifi/src/wifi_default_ap.c index ec068acbbd4..9e1be32822e 100644 --- a/components/esp_wifi/src/wifi_default_ap.c +++ b/components/esp_wifi/src/wifi_default_ap.c @@ -36,7 +36,4 @@ esp_err_t esp_wifi_ap_get_sta_list_with_ip(const wifi_sta_list_t *wifi_sta_list, } } return ESP_OK; -#else - return ESP_ERR_NOT_SUPPORTED; -#endif }