From f7d27500c909538d499dcf5e9386bf1d11559af1 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Fri, 15 Oct 2021 16:29:52 +0200 Subject: urlapi: skip a strlen(), pass in zero ... to let curl_easy_escape() itself do the strlen. This avoids a (false positive) Coverity warning and it avoids us having to store the strlen() return value in an int variable. --- lib/urlapi.c | 3 +-- tests/data/test1560 | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/urlapi.c b/lib/urlapi.c index 5c943c52c..6d116b61b 100644 --- a/lib/urlapi.c +++ b/lib/urlapi.c @@ -1295,8 +1295,7 @@ CURLUcode curl_url_get(CURLU *u, CURLUPart what, } } else if(urlencode) { - int hostlen = (int)strlen(u->host); - allochost = curl_easy_escape(NULL, u->host, hostlen); + allochost = curl_easy_escape(NULL, u->host, 0); if(!allochost) return CURLUE_OUT_OF_MEMORY; } diff --git a/tests/data/test1560 b/tests/data/test1560 index e8767b27b..f6722fd41 100644 --- a/tests/data/test1560 +++ b/tests/data/test1560 @@ -2,7 +2,7 @@ unittest -URL API +URLAPI -- cgit v1.2.1