Bladeren bron

Merge branch 'bugfix/http_server_sess_count' into 'master'

http_server: Increment active socket count for SSL session

Closes IDFGH-8188

See merge request espressif/esp-idf!19991
Mahavir Jain 3 jaren geleden
bovenliggende
commit
38c0740509
1 gewijzigde bestanden met toevoegingen van 4 en 3 verwijderingen
  1. 4 3
      components/esp_http_server/src/httpd_sess.c

+ 4 - 3
components/esp_http_server/src/httpd_sess.c

@@ -204,6 +204,9 @@ esp_err_t httpd_sess_new(struct httpd_data *hd, int newfd)
     session->send_fn = httpd_default_send;
     session->recv_fn = httpd_default_recv;
 
+    // increment number of sessions
+    hd->hd_sd_active_count++;
+
     // Call user-defined session opening function
     if (hd->config.open_fn) {
         esp_err_t ret = hd->config.open_fn(hd, session->fd);
@@ -214,10 +217,8 @@ esp_err_t httpd_sess_new(struct httpd_data *hd, int newfd)
         }
     }
 
-    // increment number of sessions
-    hd->hd_sd_active_count++;
-    ESP_LOGD(TAG, LOG_FMT("active sockets: %d"), hd->hd_sd_active_count);
 
+    ESP_LOGD(TAG, LOG_FMT("active sockets: %d"), hd->hd_sd_active_count);
     return ESP_OK;
 }