Re: [PATCH 0/2] MR7540: kernel32/tests: Add tests for maximum path length limitation.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/kernelbase/file.c:
> SetLastError( ERROR_INVALID_PARAMETER );
>          return FALSE;
>      }
> +
> +    if (wcsncmp(dest, L"\\\\?\\", 4) && wcslen(dest) >= MAX_PATH )
> +    {
> +        SetLastError( ERROR_PATH_NOT_FOUND );
> +        return FALSE;
> +    }
> +    if (wcsncmp(dest, L"\\\\?\\", 4) && wcslen(dest) >= MAX_PATH )
> +    {
> +        SetLastError( ERROR_PATH_NOT_FOUND );
> +        return FALSE;
> +    }

You're comparing the length twice.

And I don't see removing todo_wine in this patch.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_97367