From 4898135c1572a17574615a856c4818820538f46f Mon Sep 17 00:00:00 2001 From: chenwu Date: Thu, 28 Jun 2018 17:27:10 +0800 Subject: [PATCH] coap: pass null-terminated string to gethostbyname coap client parse a string by `coap_split_uri`,fetch host should by `host.length` otherwise, would cause `gethostbyname` failed because of inappropriate parameter. --- .../coap_client/main/coap_client_example_main.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/examples/protocols/coap_client/main/coap_client_example_main.c b/examples/protocols/coap_client/main/coap_client_example_main.c index 98f003e1ad..e2d1b96845 100644 --- a/examples/protocols/coap_client/main/coap_client_example_main.c +++ b/examples/protocols/coap_client/main/coap_client_example_main.c @@ -79,7 +79,7 @@ static void coap_example_task(void *p) coap_pdu_t* request = NULL; const char* server_uri = COAP_DEFAULT_DEMO_URI; uint8_t get_method = 1; - + char* phostname = NULL; while (1) { /* Wait for the callback to set the CONNECTED_BIT in the event group. @@ -93,7 +93,16 @@ static void coap_example_task(void *p) break; } - hp = gethostbyname((const char *)uri.host.s); + phostname = (char *)calloc(1, uri.host.length + 1); + + if (phostname == NULL) { + ESP_LOGE(TAG, "calloc failed"); + continue; + } + + memcpy(phostname, uri.host.s, uri.host.length); + hp = gethostbyname(phostname); + free(phostname); if (hp == NULL) { ESP_LOGE(TAG, "DNS lookup failed");