Skip to content

unit2604: avoid UNCONST()#18143

Closed
vszakats wants to merge 8 commits into
curl:masterfrom
vszakats:unconstmore
Closed

unit2604: avoid UNCONST()#18143
vszakats wants to merge 8 commits into
curl:masterfrom
vszakats:unconstmore

Conversation

@vszakats

@vszakats vszakats commented Aug 2, 2025

Copy link
Copy Markdown
Member

No description provided.

tests\unit\unit2604.c(112,1): error C2220: the following warning is treated as an error
tests\unit\unit2604.c(112,1): warning C4701: potentially uninitialized local variable 'cp0' used
tests\unit\unit2604.c(112,1): warning C4703: potentially uninitialized local pointer variable 'cp0' used
@vszakats vszakats changed the title unit2604: update to avoid UNCONST() unit2604: avoid UNCONST() Aug 2, 2025
```
 valgrind ERROR ==118247== Conditional jump or move depends on uninitialised value(s)
==118247==    at 0x485400A: strpbrk (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==118247==    by 0x15D9BC: Curl_get_pathname (curl_path.c:180)
==118247==    by 0x126263: test_unit2604 (unit2604.c:85)
==118247==    by 0x124905: main (first.c:281)
==118247==
==118247== Conditional jump or move depends on uninitialised value(s)
==118247==    at 0x484F238: strlen (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==118247==    by 0x15DB07: Curl_get_pathname (curl_path.c:182)
==118247==    by 0x126263: test_unit2604 (unit2604.c:85)
==118247==    by 0x124905: main (first.c:281)
==118247==
==118247== Conditional jump or move depends on uninitialised value(s)
==118247==    at 0x4854136: strspn (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==118247==    by 0x15D9D7: Curl_get_pathname (curl_path.c:185)
==118247==    by 0x126263: test_unit2604 (unit2604.c:85)
==118247==    by 0x124905: main (first.c:281)
==118247==
```
https://github.com/curl/curl/actions/runs/16693117599/job/47253533140?pr=18143#step:39:3585
@vszakats vszakats closed this in 37ac449 Aug 4, 2025
@vszakats vszakats deleted the unconstmore branch August 4, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant