Re: [Wget-dev] wget2 | * tests/libtest.c: Create directories if needed when creating WGET_TEST_EXISTING_FILES (!440)

[DalmeGNU]

DalmeGNU commented on a discussion on tests/libtest.c: 
https://gitlab.com/gnuwget/wget2/merge_requests/440#note_206825008

>               // create files
>               if (existing_files) {
>                       for (it = 0; existing_files[it].name; it++) {
> +                             const char *namecpy = 
> wget_strdup(existing_files[it].name);

Yes, but it modifies the string changing '/' to character 0 and then it turns 
it back to '/'. And existing_files[it].name is a pointer got from a va_list 
using va_arg(), thus the memory it points to cannot be modified and if you try 
you'll get a segmention fault

-- 
Reply to this email directly or view it on GitLab: 
https://gitlab.com/gnuwget/wget2/merge_requests/440#note_206825008
You're receiving this email because of your account on gitlab.com.