Skip to content

Commit 4470e28

Browse files
committed
WiFiClientSecure fail if gethostbyname returns null
1 parent 0c038b4 commit 4470e28

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

Diff for: libraries/WiFiClientSecure/src/ssl_client.cpp

+5-4
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ void ssl_init(sslclient_context *ssl_client)
4343
int start_ssl_client(sslclient_context *ssl_client, const char *host, uint32_t port, const char *rootCABuff, const char *cli_cert, const char *cli_key)
4444
{
4545
char buf[512];
46-
int ret, flags, len, timeout;
46+
int ret, flags, timeout;
4747
int enable = 1;
4848
log_v("Free heap before TLS %u", xPortGetFreeHeapSize());
4949

@@ -59,7 +59,8 @@ int start_ssl_client(sslclient_context *ssl_client, const char *host, uint32_t p
5959
struct hostent *server;
6060
server = gethostbyname(host);
6161
if (server == NULL) {
62-
return 0;
62+
log_e("gethostbyname failed");
63+
return -1;
6364
}
6465
IPAddress srv((const uint8_t *)(server->h_addr));
6566

@@ -245,7 +246,7 @@ int send_ssl_data(sslclient_context *ssl_client, const uint8_t *data, uint16_t l
245246
}
246247

247248
len = ret;
248-
log_v("%d bytes written", len); //for low level debug
249+
//log_v("%d bytes written", len); //for low level debug
249250
return ret;
250251
}
251252

@@ -257,6 +258,6 @@ int get_ssl_receive(sslclient_context *ssl_client, uint8_t *data, int length)
257258

258259
ret = mbedtls_ssl_read(&ssl_client->ssl_ctx, data, length);
259260

260-
log_v( "%d bytes read", ret); //for low level debug
261+
//log_v( "%d bytes read", ret); //for low level debug
261262
return ret;
262263
}

0 commit comments

Comments
 (0)