winedump: Cast-qual warnings fix

Eric Pouech eric.pouech at wanadoo.fr
Wed Oct 11 14:19:52 CDT 2006


Dmitry Timoshkov wrote:

> "Andrew Talbot" <Andrew.Talbot at talbotville.com> wrote:
>
>> #define IMAGE_FIRST_SECTION(ntheader) \
>> -  
>> ((PIMAGE_SECTION_HEADER)((LPBYTE)&((PIMAGE_NT_HEADERS)(ntheader))->OptionalHeader 
>> + \
>> -                           
>> ((PIMAGE_NT_HEADERS)(ntheader))->FileHeader.SizeOfOptionalHeader))
>> +  
>> ((PIMAGE_SECTION_HEADER)((LPBYTE)&((PIMAGE_NT_HEADERS)(UINT32)(ntheader))->OptionalHeader 
>> + \
>> +                           
>> ((PIMAGE_NT_HEADERS)(UINT32)(ntheader))->FileHeader.SizeOfOptionalHeader)) 
>>
>
>
> UINT32 is not a 64-bit safe type. Please use ULONG_PTR instead.
>
actually, this would be better written as:
((PIMAGE_SECTION_HEADER)(DWORD_PTR)((LPCBYTE)&((const 
IMAGE_NT_HEADERS*)(ntheader))->OptionalHeader + \
                           ((const 
IMAGE_NT_HEADERS*)(ntheader))->FileHeader.SizeOfOptionalHeader))



More information about the wine-devel mailing list