소스 검색

Merge branch 'bugfix/esp_tls_coverity_warning' into 'master'

Fix coverity warning in esp-tls component

See merge request espressif/esp-idf!22934
Aditya Patwardhan 2 년 전
부모
커밋
7c8e8557a6
1개의 변경된 파일4개의 추가작업 그리고 2개의 파일을 삭제
  1. 4 2
      components/esp-tls/esp_tls.c

+ 4 - 2
components/esp-tls/esp_tls.c

@@ -176,7 +176,7 @@ static esp_err_t esp_tls_hostname_to_fd(const char *host, size_t hostlen, int po
     struct addrinfo hints;
     memset(&hints, 0, sizeof(hints));
 
-    switch(addr_family) {
+    switch (addr_family) {
         case ESP_TLS_AF_INET:
             hints.ai_family = AF_INET;
             break;
@@ -331,7 +331,9 @@ static inline esp_err_t tcp_connect(const char *host, int hostlen, int port, con
 {
     struct sockaddr_storage address;
     int fd;
-    esp_err_t ret = esp_tls_hostname_to_fd(host, hostlen, port, cfg->addr_family, &address, &fd);
+
+    esp_tls_addr_family_t addr_family = (cfg != NULL) ? cfg->addr_family : ESP_TLS_AF_UNSPEC;
+    esp_err_t ret = esp_tls_hostname_to_fd(host, hostlen, port, addr_family, &address, &fd);
     if (ret != ESP_OK) {
         ESP_INT_EVENT_TRACKER_CAPTURE(error_handle, ESP_TLS_ERR_TYPE_SYSTEM, errno);
         return ret;