[PATCH] kernel32/tests: Load GetSystemFirmwareTable() dynamically.

Zebediah Figura z.figura12 at gmail.com
Fri Jul 13 03:01:29 CDT 2018


On 13/07/18 09:59, Dmitry Timoshkov wrote:
> Zebediah Figura <z.figura12 at gmail.com> wrote:
> 
>>      expected_len -= min_sfti_len;
>>      smbios_table = HeapAlloc(GetProcessHeap(), 0, expected_len);
>> -    len = GetSystemFirmwareTable(RSMB, 0, smbios_table, expected_len);
>> +    len = pGetSystemFirmwareTable(RSMB, 0, smbios_table, expected_len);
> 
> I guess that an earlier call fails and the test is skipped but still
> it would be better to check the pGetSystemFirmwareTable for NULL.
> 

Fair enough; I'll send an updated patch.



More information about the wine-devel mailing list