From: Viktor Szakats Date: Sun, 25 Jan 2026 12:38:09 +0000 (+0100) Subject: build: constify `memchr()`/`strchr()`/etc result variables (cont.) X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4e5908306ad5febee88f7eae8ea3b0c41a6b7d84;p=thirdparty%2Fcurl.git build: constify `memchr()`/`strchr()`/etc result variables (cont.) Assisted-by: Rudi Heitbaum Fixes #20420 Follow-up to 7dc60bdb90c710c2e36b2d05aa3686ff491a9bbe #20425 Follow-up to 0e2507a3c65376d6bda860ff20bd94ada9bbb9fd #20421 Closes #20428 --- diff --git a/lib/dict.c b/lib/dict.c index 832da30ce4..fcdea918cf 100644 --- a/lib/dict.c +++ b/lib/dict.c @@ -136,7 +136,7 @@ static CURLcode sendf(struct Curl_easy *data, const char *fmt, ...) static CURLcode dict_do(struct Curl_easy *data, bool *done) { - const char *word; + char *word; char *eword = NULL; char *ppath; char *database = NULL; diff --git a/lib/http_digest.c b/lib/http_digest.c index ccc67f5efc..f5b20c5e9d 100644 --- a/lib/http_digest.c +++ b/lib/http_digest.c @@ -69,7 +69,7 @@ CURLcode Curl_output_digest(struct Curl_easy *data, { CURLcode result; unsigned char *path = NULL; - char *tmp = NULL; + const char *tmp = NULL; char *response; size_t len; bool have_chlg; diff --git a/src/tool_operhlp.c b/src/tool_operhlp.c index db3365e7f1..035f6b6318 100644 --- a/src/tool_operhlp.c +++ b/src/tool_operhlp.c @@ -116,7 +116,7 @@ CURLcode add_file_name_to_url(CURL *curl, char **inurlp, const char *filename) /* We only want the part of the local path that is on the right side of the rightmost slash and backslash. */ const char *filep = strrchr(filename, '/'); - char *file2 = strrchr(filep ? filep : filename, '\\'); + const char *file2 = strrchr(filep ? filep : filename, '\\'); char *encfile; if(file2) diff --git a/src/tool_urlglob.c b/src/tool_urlglob.c index 6cd8ec176e..4ab37c935f 100644 --- a/src/tool_urlglob.c +++ b/src/tool_urlglob.c @@ -352,7 +352,7 @@ static CURLcode peek_ipv6(const char *str, size_t *skip, bool *ipv6p) */ char hostname[MAX_IP6LEN]; CURLU *u; - char *endbr = strchr(str, ']'); + const char *endbr = strchr(str, ']'); size_t hlen; CURLUcode rc; CURLcode result = CURLE_OK; diff --git a/src/tool_writeout.c b/src/tool_writeout.c index f4de98a207..fd0224c32a 100644 --- a/src/tool_writeout.c +++ b/src/tool_writeout.c @@ -734,10 +734,10 @@ void ourWriteOut(struct OperationConfig *config, struct per_transfer *per, } else { /* this is meant as a variable to output */ - char *end; + const char *end; size_t vlen; if('{' == ptr[1]) { - struct writeoutvar *wv = NULL; + const struct writeoutvar *wv = NULL; struct writeoutvar find = { 0 }; end = strchr(ptr, '}'); ptr += 2; /* pass the % and the { */ diff --git a/tests/libtest/lib1560.c b/tests/libtest/lib1560.c index deb24be9dc..c880902940 100644 --- a/tests/libtest/lib1560.c +++ b/tests/libtest/lib1560.c @@ -1185,7 +1185,7 @@ static CURLUcode updateurl(CURLU *u, const char *cmd, unsigned int setflags) /* make sure the last command ends with a comma too! */ while(p) { - char *e = strchr(p, ','); + const char *e = strchr(p, ','); if(e) { size_t n = (size_t)(e - p); char buf[80]; diff --git a/tests/server/sws.c b/tests/server/sws.c index 43813c95f8..297cd58174 100644 --- a/tests/server/sws.c +++ b/tests/server/sws.c @@ -491,7 +491,7 @@ static int sws_ProcessRequest(struct sws_httprequest *req) if(req->testno == DOCNUMBER_NOTHING) { /* check for a Testno: header with the test case number */ - char *testno = strstr(line, "\nTestno: "); + const char *testno = strstr(line, "\nTestno: "); if(testno) { pval = &testno[9]; if(!curlx_str_number(&pval, &num, INT_MAX)) { @@ -596,7 +596,7 @@ static int sws_ProcessRequest(struct sws_httprequest *req) return 1; /* done */ } else if(strstr(req->reqbuf, "\r\n0\r\n")) { - char *last_crlf_char = strstr(req->reqbuf, "\r\n\r\n"); + const char *last_crlf_char = strstr(req->reqbuf, "\r\n\r\n"); while(TRUE) { if(!strstr(last_crlf_char + 4, "\r\n\r\n")) break;