Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination

[Akihiro Sagawa (＠sgwaki)]

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c:
> +        char dll_name_a[MAX_PATH];
> +        HMODULE mod;
> +
> +        nt_header.FileHeader.NumberOfSections = 1;
> +        nt_header.FileHeader.SizeOfOptionalHeader = sizeof(IMAGE_OPTIONAL_HEADER);
> +
> +        nt_header.OptionalHeader.SectionAlignment = page_size;
> +        nt_header.OptionalHeader.DllCharacteristics = IMAGE_DLLCHARACTERISTICS_NX_COMPAT | IMAGE_DLLCHARACTERISTICS_DYNAMIC_BASE;
> +        nt_header.OptionalHeader.FileAlignment = page_size;
> +        nt_header.OptionalHeader.SizeOfHeaders = sizeof(dos_header) + sizeof(nt_header) + sizeof(IMAGE_SECTION_HEADER);
> +        nt_header.OptionalHeader.SizeOfImage = sizeof(dos_header) + sizeof(nt_header) + sizeof(IMAGE_SECTION_HEADER) + page_size;
> +
> +        create_test_dll( &dos_header, sizeof(dos_header), &nt_header, dll_name_a );
> +        mod = LoadLibraryA( dll_name_a );
> +        ok(mod != NULL, "can't load test dll\n");
> +        if (mod != NULL)
> This `if` is redundant; the previous test already covers this.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132023