kernel32: add a stub for GetSystemDEPPolicy()

Austin English austinenglish at gmail.com
Sun Sep 12 13:29:02 CDT 2010


On Sun, Sep 12, 2010 at 1:26 PM, Paul Vriens <paul.vriens.wine at gmail.com> wrote:
> On 09/12/2010 08:04 PM, Marcus Meissner wrote:
>>
>> On Sun, Sep 12, 2010 at 12:50:39PM -0500, Austin English wrote:
>>>
>>> --
>>> -Austin
>>
>>> diff --git a/dlls/kernel32/kernel32.spec b/dlls/kernel32/kernel32.spec
>>> index 8a33013..43e339b 100644
>>> --- a/dlls/kernel32/kernel32.spec
>>> +++ b/dlls/kernel32/kernel32.spec
>>> @@ -624,6 +624,7 @@
>>>  @ stdcall GetSystemDefaultLCID()
>>>  @ stdcall GetSystemDefaultLangID()
>>>  @ stdcall GetSystemDefaultUILanguage()
>>> +@ stdcall GetSystemDEPPolicy()
>>>  @ stdcall GetSystemDirectoryA(ptr long)
>>>  @ stdcall GetSystemDirectoryW(ptr long)
>>>  @ stdcall GetSystemInfo(ptr)
>>> diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c
>>> index c1816a8..5990943 100644
>>> --- a/dlls/kernel32/process.c
>>> +++ b/dlls/kernel32/process.c
>>> @@ -3385,3 +3385,14 @@ DWORD WINAPI WTSGetActiveConsoleSessionId(void)
>>>      FIXME("stub\n");
>>>      return 0;
>>>  }
>>> +
>>> +/**********************************************************************
>>> + *           GetSystemDEPPolicy     (KERNEL32.@)
>>> + */
>>> +DEP_SYSTEM_POLICY_TYPE WINAPI GetSystemDEPPolicy(void)
>>> +{
>>> +    FIXME("stub\n");
>>> +    return 0;
>>
>> better return one of the Enum values here ... "AlwaysOff" probably,
>> instead
>> of 0 (AlwaysOn).
>>
>> Ciao, Marcus
>>
>>> +}
>>> +
>>> +
>>> diff --git a/include/winbase.h b/include/winbase.h
>>> index ea10696..d8dba1d 100644
>>> --- a/include/winbase.h
>>> +++ b/include/winbase.h
>>> @@ -1230,6 +1230,13 @@ typedef struct tagHW_PROFILE_INFOW {
>>>  DECL_WINELIB_TYPE_AW(HW_PROFILE_INFO)
>>>  DECL_WINELIB_TYPE_AW(LPHW_PROFILE_INFO)
>>>
>>> +typedef enum _DEP_SYSTEM_POLICY_TYPE {
>>> +    AlwaysOn,
>>> +    AlwaysOff,
>>> +    OptIn,
>>> +    OptOut
>>> +} DEP_SYSTEM_POLICY_TYPE;
>
> MSDN states that AlwaysOff is 0, that would suggest this enum is wrong, not?

Yes, AlwaysOff is 0. I'm working on a revised patch.

Thanks for the feedback!

-- 
-Austin



More information about the wine-devel mailing list