[v2 PATCH] ucrtbase: Added _sopen_dispatch/_wsopen_dispatch

[Piotr Caban]

On 10/28/16 13:41, Huw Davies wrote:
> On Fri, Oct 28, 2016 at 01:43:19PM +0300, Nikolay Sivov wrote:
>> diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c
>> index 432d5c7..37e5ece 100644
>> --- a/dlls/msvcrt/file.c
>> +++ b/dlls/msvcrt/file.c
>> @@ -2144,9 +2144,10 @@ static int check_bom(HANDLE h, int oflags, BOOL seek)
>>  }
>>
>>  /*********************************************************************
>> - *              _wsopen_s (MSVCRT.@)
>> + *              _wsopen_dispatch (UCRTBASE.@)
>>   */
>> -int CDECL MSVCRT__wsopen_s( int *fd, const MSVCRT_wchar_t* path, int oflags, int shflags, int pmode )
>> +int CDECL MSVCRT__wsopen_dispatch( const MSVCRT_wchar_t* path, int oflags, int shflags, int pmode,
>> +    int *fd, int secure )
>>  {
>>    DWORD access = 0, creation = 0, attrib;
>>    SECURITY_ATTRIBUTES sa;
>
> You probably want to add 'secure' to the TRACE() just below here and
> change the order of the parameters in that TRACE() to match the new
> function.

A FIXME message would be probably also useful since the secure parameter 
is ignored.

Thanks,
Piotr