January 2017 - Wine-Commits

Hans Leidekker : wbemprox: Only include matching rows in the table row count.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: f2c005464ef4c0ec5f424d094eb6300e3514b288 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f2c005464ef4c0ec5f424d094… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Jan 27 09:16:55 2017 +0100 wbemprox: Only include matching rows in the table row count. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wbemprox/builtin.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c index cf9ae5c..14e819e 100644 --- a/dlls/wbemprox/builtin.c +++ b/dlls/wbemprox/builtin.c @@ -2350,7 +2350,7 @@ static enum fill_status fill_printer( struct table *table, const struct expr *co struct record_printer *rec; enum fill_status status = FILL_STATUS_UNFILTERED; PRINTER_INFO_2W *info; - DWORD i, offset = 0, count = 0, size = 0; + DWORD i, offset = 0, count = 0, size = 0, num_rows = 0; EnumPrintersW( PRINTER_ENUM_LOCAL, NULL, 2, NULL, 0, &size, &count ); if (GetLastError() != ERROR_INSUFFICIENT_BUFFER) return FILL_STATUS_FAILED; @@ -2382,9 +2382,10 @@ static enum fill_status fill_printer( struct table *table, const struct expr *co continue; } offset += sizeof(*rec); + num_rows++; } - TRACE("created %u rows\n", count); - table->num_rows = count; + TRACE("created %u rows\n", num_rows); + table->num_rows = num_rows; heap_free( info ); return status; @@ -2567,7 +2568,7 @@ static enum fill_status fill_processor( struct table *table, const struct expr * static const WCHAR fmtW[] = {'C','P','U','%','u',0}; WCHAR caption[100], device_id[14], processor_id[17], manufacturer[13], name[49] = {0}, version[50]; struct record_processor *rec; - UINT i, offset = 0, num_cores, num_logical_processors, count = get_processor_count(); + UINT i, offset = 0, num_rows = 0, num_cores, num_logical_processors, count = get_processor_count(); enum fill_status status = FILL_STATUS_UNFILTERED; if (!resize_table( table, count, sizeof(*rec) )) return FILL_STATUS_FAILED; @@ -2608,10 +2609,11 @@ static enum fill_status fill_processor( struct table *table, const struct expr * continue; } offset += sizeof(*rec); + num_rows++; } - TRACE("created %u rows\n", count); - table->num_rows = count; + TRACE("created %u rows\n", num_rows); + table->num_rows = num_rows; return status; }

1 0

Hans Leidekker : mapi32: Add a stub implementation of HrDispatchNotifications.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 454bccd63ebde54c88395e3917db8d7b58ff33aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=454bccd63ebde54c88395e391… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Jan 27 10:28:40 2017 +0100 mapi32: Add a stub implementation of HrDispatchNotifications. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mapi32/mapi32.spec | 2 +- dlls/mapi32/util.c | 9 +++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/dlls/mapi32/mapi32.spec b/dlls/mapi32/mapi32.spec index f5a1bef..560784c 100644 --- a/dlls/mapi32/mapi32.spec +++ b/dlls/mapi32/mapi32.spec @@ -175,7 +175,7 @@ 236 stub cmc_read 237 stub cmc_send 238 stub cmc_send_documents -239 stub HrDispatchNotifications(a)4 +239 stdcall HrDispatchNotifications(a)4(long) HrDispatchNotifications 241 stub HrValidateParameters(a)8 244 stub ScCreateConversationIndex(a)16 246 stub HrGetOmiProvidersFlags diff --git a/dlls/mapi32/util.c b/dlls/mapi32/util.c index 1d16a33..13e4aff 100644 --- a/dlls/mapi32/util.c +++ b/dlls/mapi32/util.c @@ -239,6 +239,15 @@ HRESULT WINAPI WrapProgress(PVOID unk1, PVOID unk2, PVOID unk3, PVOID unk4, PVOI } /************************************************************************* + * HrDispatchNotifications(a)4 (MAPI32.239) + */ +HRESULT WINAPI HrDispatchNotifications(ULONG flags) +{ + FIXME("(%08x)\n", flags); + return S_OK; +} + +/************************************************************************* * HrThisThreadAdviseSink(a)8 (MAPI32.42) * * Ensure that an advise sink is only notified in its originating thread.

1 0

Hans Leidekker : winhttp: Accept NULL buffer for size queries in WinHttpCreateUrl.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 542998ee24fe05cf230288c47538ccf18807ec64 URL: http://source.winehq.org/git/wine.git/?a=commit;h=542998ee24fe05cf230288c47… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Jan 27 10:28:43 2017 +0100 winhttp: Accept NULL buffer for size queries in WinHttpCreateUrl. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winhttp/tests/url.c | 27 +++++++++++++++++++-------- dlls/winhttp/url.c | 8 ++++++-- 2 files changed, 25 insertions(+), 10 deletions(-) diff --git a/dlls/winhttp/tests/url.c b/dlls/winhttp/tests/url.c index b76e82c..6bde1de 100644 --- a/dlls/winhttp/tests/url.c +++ b/dlls/winhttp/tests/url.c @@ -118,7 +118,7 @@ static void WinHttpCreateUrl_test( void ) { URL_COMPONENTS uc; WCHAR *url; - DWORD len; + DWORD len, err; BOOL ret; /* NULL components */ @@ -144,22 +144,33 @@ static void WinHttpCreateUrl_test( void ) ok( !ret, "expected failure\n" ); ok( GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER got %u\n", GetLastError() ); - /* valid components, NULL url */ + /* valid components, NULL url, insufficient length */ + len = 0; SetLastError( 0xdeadbeef ); ret = WinHttpCreateUrl( &uc, 0, NULL, &len ); ok( !ret, "expected failure\n" ); - ok( GetLastError() == ERROR_INSUFFICIENT_BUFFER || - GetLastError() == ERROR_INVALID_PARAMETER, - "expected ERROR_INSUFFICIENT_BUFFER or ERROR_INVALID_PARAMETER got %u\n", GetLastError() ); + ok( GetLastError() == ERROR_INSUFFICIENT_BUFFER, "expected ERROR_INSUFFICIENT_BUFFER got %u\n", GetLastError() ); + ok( len == 57, "expected len 57 got %u\n", len ); + + /* valid components, NULL url, sufficient length */ + SetLastError( 0xdeadbeef ); + len = 256; + ret = WinHttpCreateUrl( &uc, 0, NULL, &len ); + err = GetLastError(); + ok( !ret, "expected failure\n" ); + ok( err == ERROR_INVALID_PARAMETER || broken(err == ERROR_INSUFFICIENT_BUFFER) /* < win7 */, + "expected ERROR_INVALID_PARAMETER got %u\n", GetLastError() ); + ok( len == 256 || broken(len == 57) /* < win7 */, "expected len 256 got %u\n", len ); /* correct size, NULL url */ fill_url_components( &uc ); SetLastError( 0xdeadbeef ); ret = WinHttpCreateUrl( &uc, 0, NULL, &len ); + err = GetLastError(); ok( !ret, "expected failure\n" ); - ok( GetLastError() == ERROR_INSUFFICIENT_BUFFER || - GetLastError() == ERROR_INVALID_PARAMETER, - "expected ERROR_INSUFFICIENT_BUFFER or ERROR_INVALID_PARAMETER got %u\n", GetLastError() ); + ok( err == ERROR_INVALID_PARAMETER || broken(err == ERROR_INSUFFICIENT_BUFFER) /* < win7 */, + "expected ERROR_INVALID_PARAMETER got %u\n", GetLastError() ); + ok( len == 256 || broken(len == 57) /* < win7 */, "expected len 256 got %u\n", len ); /* valid components, allocated url, short length */ SetLastError( 0xdeadbeef ); diff --git a/dlls/winhttp/url.c b/dlls/winhttp/url.c index e7bb2a5..32b30da 100644 --- a/dlls/winhttp/url.c +++ b/dlls/winhttp/url.c @@ -417,13 +417,12 @@ BOOL WINAPI WinHttpCreateUrl( LPURL_COMPONENTS uc, DWORD flags, LPWSTR url, LPDW { static const WCHAR formatW[] = {'%','u',0}; static const WCHAR twoslashW[] = {'/','/'}; - DWORD len; INTERNET_SCHEME scheme; TRACE("%p, 0x%08x, %p, %p\n", uc, flags, url, required); - if (!uc || uc->dwStructSize != sizeof(URL_COMPONENTS) || !required || !url) + if (!uc || uc->dwStructSize != sizeof(URL_COMPONENTS) || !required) { set_last_error( ERROR_INVALID_PARAMETER ); return FALSE; @@ -437,6 +436,11 @@ BOOL WINAPI WinHttpCreateUrl( LPURL_COMPONENTS uc, DWORD flags, LPWSTR url, LPDW set_last_error( ERROR_INSUFFICIENT_BUFFER ); return FALSE; } + if (!url) + { + set_last_error( ERROR_INVALID_PARAMETER ); + return FALSE; + } url[0] = 0; *required = len;

1 0

Henri Verbeet : wined3d: Introduce wined3d_buffer_upload_ranges().
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: a11e301566b4f787b6e00fb1774ac52d8fa229dd URL: http://source.winehq.org/git/wine.git/?a=commit;h=a11e301566b4f787b6e00fb17… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jan 27 11:36:32 2017 +0100 wined3d: Introduce wined3d_buffer_upload_ranges(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/buffer.c | 46 +++++++++++++++++++--------------------------- 1 file changed, 19 insertions(+), 27 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index 1f7292e..a597f18 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -804,31 +804,27 @@ drop_query: This->flags &= ~WINED3D_BUFFER_APPLESYNC; } -/* The caller provides a GL context */ -static void buffer_direct_upload(struct wined3d_buffer *This, struct wined3d_context *context) +/* Context activation is done by the caller. */ +static void wined3d_buffer_upload_ranges(struct wined3d_buffer *buffer, struct wined3d_context *context, + const void *data, unsigned int range_count, const struct wined3d_map_range *ranges) { const struct wined3d_gl_info *gl_info = context->gl_info; - unsigned int start, len; + const struct wined3d_map_range *range; - buffer_bind(This, context); + buffer_bind(buffer, context); - while (This->modified_areas) + while (range_count--) { - This->modified_areas--; - start = This->maps[This->modified_areas].offset; - len = This->maps[This->modified_areas].size; - - GL_EXTCALL(glBufferSubData(This->buffer_type_hint, start, len, (BYTE *)This->resource.heap_memory + start)); - checkGLcall("glBufferSubData"); + range = &ranges[range_count]; + GL_EXTCALL(glBufferSubData(buffer->buffer_type_hint, + range->offset, range->size, (BYTE *)data + range->offset)); } - - wined3d_buffer_validate_location(This, WINED3D_LOCATION_BUFFER); + checkGLcall("glBufferSubData"); } static void buffer_conversion_upload(struct wined3d_buffer *buffer, struct wined3d_context *context) { - const struct wined3d_gl_info *gl_info = context->gl_info; - unsigned int i, j, start, end, len, vertex_count; + unsigned int i, j, range_idx, start, end, vertex_count; BYTE *data; wined3d_buffer_load_location(buffer, context, WINED3D_LOCATION_SYSMEM); @@ -842,13 +838,10 @@ static void buffer_conversion_upload(struct wined3d_buffer *buffer, struct wined return; } - buffer_bind(buffer, context); - while (buffer->modified_areas) + for (range_idx = 0; range_idx < buffer->modified_areas; ++range_idx) { - buffer->modified_areas--; - start = buffer->maps[buffer->modified_areas].offset; - len = buffer->maps[buffer->modified_areas].size; - end = start + len; + start = buffer->maps[range_idx].offset; + end = start + buffer->maps[range_idx].size; memcpy(data + start, (BYTE *)buffer->resource.heap_memory + start, end - start); for (i = start / buffer->stride; i < min((end / buffer->stride) + 1, vertex_count); ++i) @@ -873,14 +866,11 @@ static void buffer_conversion_upload(struct wined3d_buffer *buffer, struct wined } } } - - GL_EXTCALL(glBufferSubData(buffer->buffer_type_hint, start, len, data + start)); - checkGLcall("glBufferSubData"); } - HeapFree(GetProcessHeap(), 0, data); + wined3d_buffer_upload_ranges(buffer, context, data, buffer->modified_areas, buffer->maps); - wined3d_buffer_validate_location(buffer, WINED3D_LOCATION_BUFFER); + HeapFree(GetProcessHeap(), 0, data); } void buffer_mark_used(struct wined3d_buffer *buffer) @@ -1002,12 +992,14 @@ void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context * if (!(buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER)) return; - buffer_direct_upload(buffer, context); + wined3d_buffer_upload_ranges(buffer, context, buffer->resource.heap_memory, + buffer->modified_areas, buffer->maps); } else { buffer_conversion_upload(buffer, context); } + wined3d_buffer_validate_location(buffer, WINED3D_LOCATION_BUFFER); } struct wined3d_resource * CDECL wined3d_buffer_get_resource(struct wined3d_buffer *buffer)

1 0

Hans Leidekker : secur32: Implement SspiPrepareForCredWrite.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 7c6fbebf68cb36c9b889689f832e6b3e5b85c2d0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c6fbebf68cb36c9b889689f8… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Jan 27 10:28:41 2017 +0100 secur32: Implement SspiPrepareForCredWrite. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/secur32/secur32.spec | 1 + dlls/secur32/tests/secur32.c | 61 +++++++++++++++++++++++++++++++++++++++++++- dlls/sspicli/main.c | 57 +++++++++++++++++++++++++++++++++++++++++ dlls/sspicli/sspicli.spec | 2 +- include/sspi.h | 3 +++ 5 files changed, 122 insertions(+), 2 deletions(-) diff --git a/dlls/secur32/secur32.spec b/dlls/secur32/secur32.spec index 8135958..0d6895a 100644 --- a/dlls/secur32/secur32.spec +++ b/dlls/secur32/secur32.spec @@ -73,6 +73,7 @@ @ stdcall SspiEncodeStringsAsAuthIdentity(wstr wstr wstr ptr) sspicli.SspiEncodeStringsAsAuthIdentity @ stdcall SspiFreeAuthIdentity(ptr) sspicli.SspiFreeAuthIdentity @ stdcall SspiLocalFree(ptr) sspicli.SspiLocalFree +@ stdcall SspiPrepareForCredWrite(ptr wstr ptr ptr ptr ptr ptr) sspicli.SspiPrepareForCredWrite @ stdcall SspiZeroAuthIdentity(ptr) sspicli.SspiZeroAuthIdentity @ stdcall TranslateNameA(str long long ptr ptr) @ stdcall TranslateNameW(wstr long long ptr ptr) diff --git a/dlls/secur32/tests/secur32.c b/dlls/secur32/tests/secur32.c index 49103bd..df9e627 100644 --- a/dlls/secur32/tests/secur32.c +++ b/dlls/secur32/tests/secur32.c @@ -27,6 +27,7 @@ #define SECURITY_WIN32 #include <security.h> #include <schannel.h> +#include <wincred.h> #include "wine/test.h" @@ -40,6 +41,8 @@ static void (SEC_ENTRY *pSspiFreeAuthIdentity) (PSEC_WINNT_AUTH_IDENTITY_OPAQUE); static void (SEC_ENTRY *pSspiLocalFree) (void *); +static SECURITY_STATUS (SEC_ENTRY *pSspiPrepareForCredWrite) + (PSEC_WINNT_AUTH_IDENTITY_OPAQUE, PCWSTR, PULONG, PCWSTR *, PCWSTR *, PUCHAR *, PULONG); static void (SEC_ENTRY *pSspiZeroAuthIdentity) (PSEC_WINNT_AUTH_IDENTITY_OPAQUE); @@ -317,6 +320,60 @@ static void test_SspiEncodeStringsAsAuthIdentity(void) pSspiFreeAuthIdentity( id ); } +static void test_SspiPrepareForCredWrite(void) +{ + static const WCHAR usernameW[] = + {'u','s','e','r','n','a','m','e',0}; + static const WCHAR domainnameW[] = + {'d','o','m','a','i','n','n','a','m','e',0}; + static const WCHAR passwordW[] = + {'p','a','s','s','w','o','r','d',0}; + static const WCHAR targetW[] = + {'d','o','m','a','i','n','n','a','m','e','\\','u','s','e','r','n','a','m','e',0}; + static const WCHAR target2W[] = + {'d','o','m','a','i','n','n','a','m','e','2','\\','u','s','e','r','n','a','m','e','2',0}; + const WCHAR *target, *username; + PSEC_WINNT_AUTH_IDENTITY_OPAQUE id; + SECURITY_STATUS status; + ULONG type, size; + UCHAR *blob; + + if (!pSspiPrepareForCredWrite) + { + win_skip( "SspiPrepareForCredWrite not exported by secur32.dll\n" ); + return; + } + + status = pSspiEncodeStringsAsAuthIdentity( usernameW, domainnameW, passwordW, &id ); + ok( status == SEC_E_OK, "got %08x\n", status ); + + type = size = 0; + status = pSspiPrepareForCredWrite( id, NULL, &type, &target, &username, &blob, &size ); + ok( status == SEC_E_OK, "got %08x\n", status ); + ok( type == CRED_TYPE_DOMAIN_PASSWORD, "got %u\n", type ); + ok( !lstrcmpW( target, targetW ), "got %s\n", wine_dbgstr_w(target) ); + ok( !lstrcmpW( username, targetW ), "got %s\n", wine_dbgstr_w(username) ); + ok( !memcmp( blob, passwordW, sizeof(passwordW) - sizeof(WCHAR) ), "wrong data\n" ); + ok( size == sizeof(passwordW) - sizeof(WCHAR), "got %u\n", size ); + pSspiLocalFree( (void *)target ); + pSspiLocalFree( (void *)username ); + pSspiLocalFree( blob ); + + type = size = 0; + status = pSspiPrepareForCredWrite( id, target2W, &type, &target, &username, &blob, &size ); + ok( status == SEC_E_OK, "got %08x\n", status ); + ok( type == CRED_TYPE_DOMAIN_PASSWORD, "got %u\n", type ); + ok( !lstrcmpW( target, target2W ), "got %s\n", wine_dbgstr_w(target) ); + ok( !lstrcmpW( username, targetW ), "got %s\n", wine_dbgstr_w(username) ); + ok( !memcmp( blob, passwordW, sizeof(passwordW) - sizeof(WCHAR) ), "wrong data\n" ); + ok( size == sizeof(passwordW) - sizeof(WCHAR), "got %u\n", size ); + pSspiLocalFree( (void *)target ); + pSspiLocalFree( (void *)username ); + pSspiLocalFree( blob ); + + pSspiFreeAuthIdentity( id ); +} + static void test_kerberos(void) { SecPkgInfoA *info; @@ -379,6 +436,7 @@ START_TEST(secur32) pSspiEncodeStringsAsAuthIdentity = (void *)GetProcAddress(secdll, "SspiEncodeStringsAsAuthIdentity"); pSspiFreeAuthIdentity = (void *)GetProcAddress(secdll, "SspiFreeAuthIdentity"); pSspiLocalFree = (void *)GetProcAddress(secdll, "SspiLocalFree"); + pSspiPrepareForCredWrite = (void *)GetProcAddress(secdll, "SspiPrepareForCredWrite"); pSspiZeroAuthIdentity = (void *)GetProcAddress(secdll, "SspiZeroAuthIdentity"); pGetComputerObjectNameA = (PVOID)GetProcAddress(secdll, "GetComputerObjectNameA"); pGetComputerObjectNameW = (PVOID)GetProcAddress(secdll, "GetComputerObjectNameW"); @@ -386,7 +444,7 @@ START_TEST(secur32) pGetUserNameExW = (PVOID)GetProcAddress(secdll, "GetUserNameExW"); pInitSecurityInterfaceA = (PVOID)GetProcAddress(secdll, "InitSecurityInterfaceA"); pInitSecurityInterfaceW = (PVOID)GetProcAddress(secdll, "InitSecurityInterfaceW"); - + if (pGetComputerObjectNameA) testGetComputerObjectNameA(); else @@ -409,6 +467,7 @@ START_TEST(secur32) test_InitSecurityInterface(); test_SspiEncodeStringsAsAuthIdentity(); + test_SspiPrepareForCredWrite(); FreeLibrary(secdll); } diff --git a/dlls/sspicli/main.c b/dlls/sspicli/main.c index 34f0a64..dfcc1cc 100644 --- a/dlls/sspicli/main.c +++ b/dlls/sspicli/main.c @@ -22,6 +22,7 @@ #include "winbase.h" #include "rpc.h" #include "sspi.h" +#include "wincred.h" #include "wine/debug.h" #include "wine/unicode.h" @@ -140,3 +141,59 @@ void SEC_ENTRY SspiLocalFree( void *ptr ) TRACE( "%p\n", ptr ); HeapFree( GetProcessHeap(), 0, ptr ); } + +/*********************************************************************** + * SspiPrepareForCredWrite (SECUR32.0) + */ +SECURITY_STATUS SEC_ENTRY SspiPrepareForCredWrite( PSEC_WINNT_AUTH_IDENTITY_OPAQUE opaque_id, + PCWSTR target, PULONG type, PCWSTR *targetname, PCWSTR *username, PUCHAR *blob, PULONG size ) +{ + SEC_WINNT_AUTH_IDENTITY_W *id = (SEC_WINNT_AUTH_IDENTITY_W *)opaque_id; + WCHAR *str, *str2; + UCHAR *password; + ULONG len; + + FIXME( "%p %s %p %p %p %p %p\n", opaque_id, debugstr_w(target), type, targetname, username, + blob, size ); + + if (id->DomainLength) + { + len = (id->DomainLength + id->UserLength + 2) * sizeof(WCHAR); + if (!(str = HeapAlloc(GetProcessHeap(), 0 , len ))) return SEC_E_INSUFFICIENT_MEMORY; + memcpy( str, id->Domain, id->DomainLength * sizeof(WCHAR) ); + str[id->DomainLength] = '\\'; + memcpy( str + id->DomainLength + 1, id->User, id->UserLength * sizeof(WCHAR) ); + str[id->DomainLength + 1 + id->UserLength] = 0; + } + else + { + len = (id->UserLength + 1) * sizeof(WCHAR); + if (!(str = HeapAlloc(GetProcessHeap(), 0 , len ))) return SEC_E_INSUFFICIENT_MEMORY; + memcpy( str, id->User, id->UserLength * sizeof(WCHAR) ); + str[id->UserLength] = 0; + } + + str2 = target ? strdupW( target ) : strdupW( str ); + if (!str2) + { + HeapFree( GetProcessHeap(), 0, str ); + return SEC_E_INSUFFICIENT_MEMORY; + } + + len = id->PasswordLength * sizeof(WCHAR); + if (!(password = HeapAlloc(GetProcessHeap(), 0 , len ))) + { + HeapFree( GetProcessHeap(), 0, str ); + HeapFree( GetProcessHeap(), 0, str2 ); + return SEC_E_INSUFFICIENT_MEMORY; + } + memcpy( password, id->Password, len ); + + *type = CRED_TYPE_DOMAIN_PASSWORD; + *username = str; + *targetname = str2; + *blob = password; + *size = len; + + return SEC_E_OK; +} diff --git a/dlls/sspicli/sspicli.spec b/dlls/sspicli/sspicli.spec index 8876bc0..e18d359 100644 --- a/dlls/sspicli/sspicli.spec +++ b/dlls/sspicli/sspicli.spec @@ -95,7 +95,7 @@ @ stdcall SspiLocalFree(ptr) @ stub SspiMarshalAuthIdentity @ stub SspiPrepareForCredRead -@ stub SspiPrepareForCredWrite +@ stdcall SspiPrepareForCredWrite(ptr wstr ptr ptr ptr ptr ptr) @ stub SspiUnmarshalAuthIdentity @ stub SspiUnmarshalAuthIdentityInternal @ stub SspiValidateAuthIdentity diff --git a/include/sspi.h b/include/sspi.h index 9e44a6d..0e248fb 100644 --- a/include/sspi.h +++ b/include/sspi.h @@ -98,6 +98,9 @@ SECURITY_STATUS SEC_ENTRY SspiEncodeAuthIdentityAsStrings( SECURITY_STATUS SEC_ENTRY SspiEncodeStringsAsAuthIdentity( PCWSTR, PCWSTR, PCWSTR, PSEC_WINNT_AUTH_IDENTITY_OPAQUE *); +SECURITY_STATUS SEC_ENTRY SspiPrepareForCredWrite( + PSEC_WINNT_AUTH_IDENTITY_OPAQUE, PCWSTR, PULONG, PCWSTR*, PCWSTR*, PUCHAR*, PULONG); + ULONG SEC_ENTRY SspiPromptForCredentialsA(PCSTR, void *, ULONG, PCSTR, PSEC_WINNT_AUTH_IDENTITY_OPAQUE, PSEC_WINNT_AUTH_IDENTITY_OPAQUE *, int *, ULONG);

1 0

Henri Verbeet : wined3d: Just use glBufferSubData() in buffer_direct_upload().
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 602a23a60aaab164acb8a2767094d545172ab4cf URL: http://source.winehq.org/git/wine.git/?a=commit;h=602a23a60aaab164acb8a2767… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jan 27 11:36:31 2017 +0100 wined3d: Just use glBufferSubData() in buffer_direct_upload(). It might seem like a good idea to try to use glMapBufferRange() in buffer_direct_upload(), but it really isn't. The interesting cases for glMapBufferRange() are GL_MAP_INVALIDATE_BUFFER_BIT or GL_MAP_UNSYNCHRONIZED_BIT, but the only way for those to be set here would be for the buffer to be mapped with WINED3D_MAP_DISCARD or WINED3D_MAP_NOOVERWRITE before the buffer object was created. In that case the GPU is obviously not using the buffer object, so glMapBufferRange() doesn't help. On the other hand, if the buffer is currently in use by the application and neither WINED3D_MAP_DISCARD nor WINED3D_MAP_NOOVERWRITE is set, glMapBufferRange() would need to wait for the GPU to finish using the buffer object, while glBufferSubData() may decide not to. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/buffer.c | 67 +++++---------------------------------------------- 1 file changed, 6 insertions(+), 61 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index 7d0552b..1f7292e 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -33,9 +33,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d); #define WINED3D_BUFFER_USE_BO 0x02 /* Use a buffer object for this buffer. */ #define WINED3D_BUFFER_DOUBLEBUFFER 0x04 /* Keep both a buffer object and a system memory copy for this buffer. */ #define WINED3D_BUFFER_DISCARD 0x08 /* A DISCARD lock has occurred since the last preload. */ -#define WINED3D_BUFFER_SYNC 0x10 /* There has been at least one synchronized map since the last preload. */ -#define WINED3D_BUFFER_MAP 0x20 /* There has been at least one map since the last preload. */ -#define WINED3D_BUFFER_APPLESYNC 0x40 /* Using sync as in GL_APPLE_flush_buffer_range. */ +#define WINED3D_BUFFER_APPLESYNC 0x10 /* Using sync as in GL_APPLE_flush_buffer_range. */ #define VB_MAXDECLCHANGES 100 /* After that number of decl changes we stop converting */ #define VB_RESETDECLCHANGE 1000 /* Reset the decl changecount after that number of draws */ @@ -807,44 +805,12 @@ drop_query: } /* The caller provides a GL context */ -static void buffer_direct_upload(struct wined3d_buffer *This, struct wined3d_context *context, DWORD flags) +static void buffer_direct_upload(struct wined3d_buffer *This, struct wined3d_context *context) { const struct wined3d_gl_info *gl_info = context->gl_info; unsigned int start, len; - BYTE *map; buffer_bind(This, context); - if (gl_info->supported[ARB_MAP_BUFFER_RANGE]) - { - GLbitfield mapflags; - mapflags = GL_MAP_WRITE_BIT | GL_MAP_FLUSH_EXPLICIT_BIT; - if (flags & WINED3D_BUFFER_DISCARD) - mapflags |= GL_MAP_INVALIDATE_BUFFER_BIT; - else if (flags & WINED3D_BUFFER_MAP && !(flags & WINED3D_BUFFER_SYNC)) - mapflags |= GL_MAP_UNSYNCHRONIZED_BIT; - map = GL_EXTCALL(glMapBufferRange(This->buffer_type_hint, 0, - This->resource.size, mapflags)); - checkGLcall("glMapBufferRange"); - } - else - { - if (This->flags & WINED3D_BUFFER_APPLESYNC) - { - DWORD syncflags = 0; - if (flags & WINED3D_BUFFER_DISCARD) - syncflags |= WINED3D_MAP_DISCARD; - else if (flags & WINED3D_BUFFER_MAP && !(flags & WINED3D_BUFFER_SYNC)) - syncflags |= WINED3D_MAP_NOOVERWRITE; - buffer_sync_apple(This, syncflags, gl_info); - } - map = GL_EXTCALL(glMapBuffer(This->buffer_type_hint, GL_WRITE_ONLY)); - checkGLcall("glMapBuffer"); - } - if (!map) - { - ERR("Failed to map OpenGL buffer.\n"); - return; - } while (This->modified_areas) { @@ -852,21 +818,9 @@ static void buffer_direct_upload(struct wined3d_buffer *This, struct wined3d_con start = This->maps[This->modified_areas].offset; len = This->maps[This->modified_areas].size; - memcpy(map + start, (BYTE *)This->resource.heap_memory + start, len); - - if (gl_info->supported[ARB_MAP_BUFFER_RANGE]) - { - GL_EXTCALL(glFlushMappedBufferRange(This->buffer_type_hint, start, len)); - checkGLcall("glFlushMappedBufferRange"); - } - else if (This->flags & WINED3D_BUFFER_APPLESYNC) - { - GL_EXTCALL(glFlushMappedBufferRangeAPPLE(This->buffer_type_hint, start, len)); - checkGLcall("glFlushMappedBufferRangeAPPLE"); - } + GL_EXTCALL(glBufferSubData(This->buffer_type_hint, start, len, (BYTE *)This->resource.heap_memory + start)); + checkGLcall("glBufferSubData"); } - GL_EXTCALL(glUnmapBuffer(This->buffer_type_hint)); - checkGLcall("glUnmapBuffer"); wined3d_buffer_validate_location(This, WINED3D_LOCATION_BUFFER); } @@ -931,14 +885,13 @@ static void buffer_conversion_upload(struct wined3d_buffer *buffer, struct wined void buffer_mark_used(struct wined3d_buffer *buffer) { - buffer->flags &= ~(WINED3D_BUFFER_MAP | WINED3D_BUFFER_SYNC | WINED3D_BUFFER_DISCARD); + buffer->flags &= ~WINED3D_BUFFER_DISCARD; } /* Context activation is done by the caller. */ void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context *context, const struct wined3d_state *state) { - DWORD flags = buffer->flags & (WINED3D_BUFFER_MAP | WINED3D_BUFFER_SYNC | WINED3D_BUFFER_DISCARD); const struct wined3d_gl_info *gl_info = context->gl_info; BOOL decl_changed = FALSE; @@ -1009,11 +962,6 @@ void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context * /* The declaration changed, reload the whole buffer. */ WARN("Reloading buffer because of a vertex declaration change.\n"); buffer_invalidate_bo_range(buffer, 0, 0); - - /* Avoid unfenced updates, we might overwrite more areas of the buffer than the application - * cleared for unsynchronized updates. - */ - flags = 0; } else { @@ -1054,7 +1002,7 @@ void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context * if (!(buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER)) return; - buffer_direct_upload(buffer, context, flags); + buffer_direct_upload(buffer, context); } else { @@ -1173,11 +1121,8 @@ static HRESULT wined3d_buffer_map(struct wined3d_buffer *buffer, UINT offset, UI } } - buffer->flags |= WINED3D_BUFFER_MAP; if (flags & WINED3D_MAP_DISCARD) buffer->flags |= WINED3D_BUFFER_DISCARD; - else if (!(flags & WINED3D_MAP_NOOVERWRITE)) - buffer->flags |= WINED3D_BUFFER_SYNC; } base = buffer->map_ptr ? buffer->map_ptr : buffer->resource.heap_memory;

1 0

Henri Verbeet : wined3d: Rename WINED3D_BUFFER_DOUBLEBUFFER to WINED3D_BUFFER_PIN_SYSMEM.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 6a70c55615731a99e1a834f6fcc62766a9d228d3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6a70c55615731a99e1a834f6f… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jan 27 11:36:33 2017 +0100 wined3d: Rename WINED3D_BUFFER_DOUBLEBUFFER to WINED3D_BUFFER_PIN_SYSMEM. For consistency with the texture code. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/buffer.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index a597f18..2d0303f 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -31,7 +31,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d); #define WINED3D_BUFFER_HASDESC 0x01 /* A vertex description has been found. */ #define WINED3D_BUFFER_USE_BO 0x02 /* Use a buffer object for this buffer. */ -#define WINED3D_BUFFER_DOUBLEBUFFER 0x04 /* Keep both a buffer object and a system memory copy for this buffer. */ +#define WINED3D_BUFFER_PIN_SYSMEM 0x04 /* Keep a system memory copy for this buffer. */ #define WINED3D_BUFFER_DISCARD 0x08 /* A DISCARD lock has occurred since the last preload. */ #define WINED3D_BUFFER_APPLESYNC 0x10 /* Using sync as in GL_APPLE_flush_buffer_range. */ @@ -249,7 +249,7 @@ static BOOL buffer_create_buffer_object(struct wined3d_buffer *buffer, struct wi buffer->buffer_object_usage = gl_usage; - if (buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER) + if (buffer->flags & WINED3D_BUFFER_PIN_SYSMEM) { buffer_invalidate_bo_range(buffer, 0, 0); } @@ -606,7 +606,7 @@ BOOL wined3d_buffer_load_location(struct wined3d_buffer *buffer, GL_EXTCALL(glGetBufferSubData(buffer->buffer_type_hint, 0, buffer->resource.size, buffer->resource.heap_memory)); checkGLcall("buffer download"); - buffer->flags |= WINED3D_BUFFER_DOUBLEBUFFER; + buffer->flags |= WINED3D_BUFFER_PIN_SYSMEM; break; case WINED3D_LOCATION_BUFFER: @@ -669,8 +669,8 @@ static void buffer_unload(struct wined3d_resource *resource) /* Download the buffer, but don't permanently enable double buffering. */ wined3d_buffer_load_location(buffer, context, WINED3D_LOCATION_SYSMEM); - if (!(flags & WINED3D_BUFFER_DOUBLEBUFFER)) - buffer->flags &= ~WINED3D_BUFFER_DOUBLEBUFFER; + if (!(flags & WINED3D_BUFFER_PIN_SYSMEM)) + buffer->flags &= ~WINED3D_BUFFER_PIN_SYSMEM; wined3d_buffer_invalidate_location(buffer, WINED3D_LOCATION_BUFFER); buffer_destroy_buffer_object(buffer, context); @@ -989,7 +989,7 @@ void wined3d_buffer_load(struct wined3d_buffer *buffer, struct wined3d_context * TRACE("No conversion needed.\n"); /* Nothing to do because heap memory exists if the buffer is double buffer or has no BO at all. */ - if (!(buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER)) + if (!(buffer->flags & WINED3D_BUFFER_PIN_SYSMEM)) return; wined3d_buffer_upload_ranges(buffer, context, buffer->resource.heap_memory, @@ -1042,7 +1042,7 @@ static HRESULT wined3d_buffer_map(struct wined3d_buffer *buffer, UINT offset, UI dirty_size = 0; } - if (buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER) + if (buffer->flags & WINED3D_BUFFER_PIN_SYSMEM) { if (!(buffer->locations & WINED3D_LOCATION_SYSMEM)) { @@ -1149,7 +1149,7 @@ static void wined3d_buffer_unmap(struct wined3d_buffer *buffer) return; } - if (!(buffer->flags & WINED3D_BUFFER_DOUBLEBUFFER) && buffer->buffer_object) + if (!(buffer->flags & WINED3D_BUFFER_PIN_SYSMEM) && buffer->buffer_object) { struct wined3d_device *device = buffer->resource.device; const struct wined3d_gl_info *gl_info; @@ -1420,7 +1420,7 @@ static HRESULT buffer_init(struct wined3d_buffer *buffer, struct wined3d_device * maps and retain data in DISCARD maps. Keep a system memory copy of * the buffer to provide the same behavior to the application. */ TRACE("Using doublebuffer mode.\n"); - buffer->flags |= WINED3D_BUFFER_DOUBLEBUFFER; + buffer->flags |= WINED3D_BUFFER_PIN_SYSMEM; } /* Observations show that draw_primitive_immediate_mode() is faster on

1 0

Henri Verbeet : wined3d: Introduce wined3d_buffer_evict_sysmem().
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 7a18b2473f923f22f2038bb273b5bcc52735c1a6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7a18b2473f923f22f2038bb27… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jan 27 11:36:34 2017 +0100 wined3d: Introduce wined3d_buffer_evict_sysmem(). For consistency with the texture code. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/buffer.c | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index 2d0303f..5303d09 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -40,6 +40,19 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d); #define VB_MAXFULLCONVERSIONS 5 /* Number of full conversions before we stop converting */ #define VB_RESETFULLCONVS 20 /* Reset full conversion counts after that number of draws */ +static void wined3d_buffer_evict_sysmem(struct wined3d_buffer *buffer) +{ + if (buffer->flags & WINED3D_BUFFER_PIN_SYSMEM) + { + TRACE("Not evicting system memory for buffer %p.\n", buffer); + return; + } + + TRACE("Evicting system memory for buffer %p.\n", buffer); + wined3d_buffer_invalidate_location(buffer, WINED3D_LOCATION_SYSMEM); + wined3d_resource_free_sysmem(&buffer->resource); +} + static void buffer_invalidate_bo_range(struct wined3d_buffer *buffer, unsigned int offset, unsigned int size) { if (!offset && (!size || size == buffer->resource.size)) @@ -250,15 +263,11 @@ static BOOL buffer_create_buffer_object(struct wined3d_buffer *buffer, struct wi buffer->buffer_object_usage = gl_usage; if (buffer->flags & WINED3D_BUFFER_PIN_SYSMEM) - { buffer_invalidate_bo_range(buffer, 0, 0); - } else - { - wined3d_resource_free_sysmem(&buffer->resource); wined3d_buffer_validate_location(buffer, WINED3D_LOCATION_BUFFER); - wined3d_buffer_invalidate_location(buffer, WINED3D_LOCATION_SYSMEM); - } + + wined3d_buffer_evict_sysmem(buffer); return TRUE;

1 0

Józef Kucia : d3d10core/tests: Add test for shader buffer views.
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: 59ea167fd5e3bde086de202c34323d69db15e77b URL: http://source.winehq.org/git/wine.git/?a=commit;h=59ea167fd5e3bde086de202c3… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Thu Jan 26 13:07:34 2017 +0100 d3d10core/tests: Add test for shader buffer views. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10core/tests/device.c | 181 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 181 insertions(+) diff --git a/dlls/d3d10core/tests/device.c b/dlls/d3d10core/tests/device.c index b148a9c..86d3df3 100644 --- a/dlls/d3d10core/tests/device.c +++ b/dlls/d3d10core/tests/device.c @@ -10620,6 +10620,186 @@ static void test_render_target_device_mismatch(void) release_test_context(&test_context); } +static void test_buffer_srv(void) +{ + struct buffer + { + UINT byte_count; + const void *data; + }; + + struct d3d10core_test_context test_context; + D3D10_SHADER_RESOURCE_VIEW_DESC srv_desc; + D3D10_SUBRESOURCE_DATA resource_data; + const struct buffer *current_buffer; + ID3D10ShaderResourceView *srv; + D3D10_BUFFER_DESC buffer_desc; + struct resource_readback rb; + ID3D10Buffer *cb, *buffer; + ID3D10PixelShader *ps; + ID3D10Device *device; + unsigned int i, x, y; + struct vec4 cb_size; + DWORD color; + HRESULT hr; + + static const DWORD ps_float4_code[] = + { +#if 0 + Buffer<float4> b; + + float2 size; + + float4 main(float4 position : SV_POSITION) : SV_Target + { + float2 p; + int2 coords; + p.x = position.x / 640.0f; + p.y = position.y / 480.0f; + coords = int2(p.x * size.x, p.y * size.y); + return b.Load(coords.y * size.x + coords.x); + } +#endif + 0x43425844, 0xf10ea650, 0x311f5c38, 0x3a888b7f, 0x58230334, 0x00000001, 0x000001a0, 0x00000003, + 0x0000002c, 0x00000060, 0x00000094, 0x4e475349, 0x0000002c, 0x00000001, 0x00000008, 0x00000020, + 0x00000000, 0x00000001, 0x00000003, 0x00000000, 0x0000030f, 0x505f5653, 0x5449534f, 0x004e4f49, + 0x4e47534f, 0x0000002c, 0x00000001, 0x00000008, 0x00000020, 0x00000000, 0x00000000, 0x00000003, + 0x00000000, 0x0000000f, 0x545f5653, 0x65677261, 0xabab0074, 0x52444853, 0x00000104, 0x00000040, + 0x00000041, 0x04000059, 0x00208e46, 0x00000000, 0x00000001, 0x04000858, 0x00107000, 0x00000000, + 0x00005555, 0x04002064, 0x00101032, 0x00000000, 0x00000001, 0x03000065, 0x001020f2, 0x00000000, + 0x02000068, 0x00000001, 0x08000038, 0x00100032, 0x00000000, 0x00101516, 0x00000000, 0x00208516, + 0x00000000, 0x00000000, 0x0a000038, 0x00100032, 0x00000000, 0x00100046, 0x00000000, 0x00004002, + 0x3b088889, 0x3acccccd, 0x00000000, 0x00000000, 0x05000043, 0x00100032, 0x00000000, 0x00100046, + 0x00000000, 0x0a000032, 0x00100012, 0x00000000, 0x0010000a, 0x00000000, 0x0020800a, 0x00000000, + 0x00000000, 0x0010001a, 0x00000000, 0x0500001b, 0x00100012, 0x00000000, 0x0010000a, 0x00000000, + 0x0700002d, 0x001020f2, 0x00000000, 0x00100006, 0x00000000, 0x00107e46, 0x00000000, 0x0100003e, + }; + static const DWORD rgba16[] = + { + 0xff0000ff, 0xff00ffff, 0xff00ff00, 0xffffff00, + 0xffff0000, 0xffff00ff, 0xff000000, 0xff7f7f7f, + 0xffffffff, 0xffffffff, 0xffffffff, 0xff000000, + 0xffffffff, 0xff000000, 0xff000000, 0xff000000, + }; + static const DWORD rgba4[] = + { + 0xffffffff, 0xff0000ff, + 0xff000000, 0xff00ff00, + }; + static const struct buffer rgba16_buffer = {sizeof(rgba16), &rgba16}; + static const struct buffer rgba4_buffer = {sizeof(rgba4), &rgba4}; + static const DWORD rgba4_colors[] = + { + 0xffffffff, 0xffffffff, 0xff0000ff, 0xff0000ff, + 0xffffffff, 0xffffffff, 0xff0000ff, 0xff0000ff, + 0xff000000, 0xff000000, 0xff00ff00, 0xff00ff00, + 0xff000000, 0xff000000, 0xff00ff00, 0xff00ff00, + }; + static const DWORD zero_colors[16] = {0}; + static const float red[] = {1.0f, 0.0f, 0.0f, 0.5f}; + + static const struct test + { + const struct buffer *buffer; + DXGI_FORMAT srv_format; + UINT srv_first_element; + UINT srv_element_count; + struct vec2 size; + const DWORD *expected_colors; + } + tests[] = + { + {&rgba16_buffer, DXGI_FORMAT_R8G8B8A8_UNORM, 0, 16, {4.0f, 4.0f}, rgba16}, + {&rgba4_buffer, DXGI_FORMAT_R8G8B8A8_UNORM, 0, 4, {2.0f, 2.0f}, rgba4_colors}, + {NULL, 0, 0, 0, {2.0f, 2.0f}, zero_colors}, + }; + + if (!init_test_context(&test_context)) + return; + + device = test_context.device; + + cb = create_buffer(device, D3D10_BIND_CONSTANT_BUFFER, sizeof(cb_size), NULL); + + hr = ID3D10Device_CreatePixelShader(device, ps_float4_code, sizeof(ps_float4_code), &ps); + ok(SUCCEEDED(hr), "Failed to create pixel shader, hr %#x.\n", hr); + + ID3D10Device_PSSetShader(device, ps); + ID3D10Device_PSSetConstantBuffers(device, 0, 1, &cb); + + srv = NULL; + buffer = NULL; + current_buffer = NULL; + for (i = 0; i < sizeof(tests) / sizeof(*tests); ++i) + { + const struct test *test = &tests[i]; + + if (current_buffer != test->buffer) + { + if (buffer) + ID3D10Buffer_Release(buffer); + if (srv) + ID3D10ShaderResourceView_Release(srv); + + current_buffer = test->buffer; + if (current_buffer) + { + buffer_desc.ByteWidth = current_buffer->byte_count; + buffer_desc.Usage = D3D10_USAGE_DEFAULT; + buffer_desc.BindFlags = D3D10_BIND_SHADER_RESOURCE; + buffer_desc.CPUAccessFlags = 0; + buffer_desc.MiscFlags = 0; + resource_data.SysMemPitch = 0; + resource_data.SysMemSlicePitch = 0; + resource_data.pSysMem = current_buffer->data; + hr = ID3D10Device_CreateBuffer(device, &buffer_desc, &resource_data, &buffer); + ok(SUCCEEDED(hr), "Test %u: Failed to create buffer, hr %#x.\n", i, hr); + + srv_desc.Format = test->srv_format; + srv_desc.ViewDimension = D3D10_SRV_DIMENSION_BUFFER; + U(srv_desc).Buffer.ElementOffset = test->srv_first_element; + U(srv_desc).Buffer.ElementWidth = test->srv_element_count; + hr = ID3D10Device_CreateShaderResourceView(device, (ID3D10Resource *)buffer, &srv_desc, &srv); + ok(SUCCEEDED(hr), "Test %u: Failed to create shader resource view, hr %#x.\n", i, hr); + } + else + { + buffer = NULL; + srv = NULL; + } + + ID3D10Device_PSSetShaderResources(device, 0, 1, &srv); + } + + cb_size.x = test->size.x; + cb_size.y = test->size.y; + ID3D10Device_UpdateSubresource(device, (ID3D10Resource *)cb, 0, NULL, &cb_size, 0, 0); + + ID3D10Device_ClearRenderTargetView(device, test_context.backbuffer_rtv, red); + draw_quad(&test_context); + + get_texture_readback(test_context.backbuffer, 0, &rb); + for (y = 0; y < 4; ++y) + { + for (x = 0; x < 4; ++x) + { + color = get_readback_color(&rb, 80 + x * 160, 60 + y * 120); + ok(compare_color(color, test->expected_colors[y * 4 + x], 1), + "Test %u: Got unexpected color 0x%08x at (%u, %u).\n", i, color, x, y); + } + } + release_resource_readback(&rb); + } + if (srv) + ID3D10ShaderResourceView_Release(srv); + if (buffer) + ID3D10Buffer_Release(buffer); + + ID3D10Buffer_Release(cb); + ID3D10PixelShader_Release(ps); + release_test_context(&test_context); +} + START_TEST(device) { test_feature_level(); @@ -10680,4 +10860,5 @@ START_TEST(device) test_sm4_ret_instruction(); test_primitive_restart(); test_render_target_device_mismatch(); + test_buffer_srv(); }

1 0

Nikolay Sivov : comctl32/syslink: Don't use exported StrCmpNIW().
by Alexandre Julliard Jan. 27, 2017

Jan. 27, 2017

Module: wine Branch: master Commit: a0e73a1fb32a0a20ed5ed97df6e837046c1ce697 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a0e73a1fb32a0a20ed5ed97df… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Jan 27 16:49:04 2017 +0300 comctl32/syslink: Don't use exported StrCmpNIW(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/syslink.c | 39 ++++++++------------------------------- 1 file changed, 8 insertions(+), 31 deletions(-) diff --git a/dlls/comctl32/syslink.c b/dlls/comctl32/syslink.c index 08e5230..339c728 100644 --- a/dlls/comctl32/syslink.c +++ b/dlls/comctl32/syslink.c @@ -42,8 +42,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(syslink); -INT WINAPI StrCmpNIW(LPCWSTR,LPCWSTR,INT); - typedef struct { int nChars; @@ -99,11 +97,6 @@ typedef struct BOOL IgnoreReturn; /* (infoPtr->Style & LWS_IGNORERETURN) on creation */ } SYSLINK_INFO; -static const WCHAR SL_LINKOPEN[] = { '<','a', 0 }; -static const WCHAR SL_HREF[] = { 'h','r','e','f','=','\"',0 }; -static const WCHAR SL_ID[] = { 'i','d','=','\"',0 }; -static const WCHAR SL_LINKCLOSE[] = { '<','/','a','>',0 }; - /* Control configuration constants */ #define SL_LEFTMARGIN (0) @@ -176,32 +169,16 @@ static VOID SYSLINK_ClearDoc (SYSLINK_INFO *infoPtr) } /*********************************************************************** - * SYSLINK_StrCmpNIW - * Wrapper for StrCmpNIW to ensure 'len' is not too big. - */ -static INT SYSLINK_StrCmpNIW (LPCWSTR str, LPCWSTR comp, INT len) -{ - INT i; - - for(i = 0; i < len; i++) - { - if(!str[i]) - { - len = i + 1; - break; - } - } - - return StrCmpNIW(str, comp, len); -} - -/*********************************************************************** * SYSLINK_ParseText * Parses the window text string and creates a document. Returns the * number of document items created. */ static UINT SYSLINK_ParseText (SYSLINK_INFO *infoPtr, LPCWSTR Text) { + static const WCHAR SL_LINKOPEN[] = { '<','a' }; + static const WCHAR SL_HREF[] = { 'h','r','e','f','=','\"' }; + static const WCHAR SL_ID[] = { 'i','d','=','\"' }; + static const WCHAR SL_LINKCLOSE[] = { '<','/','a','>' }; LPCWSTR current, textstart = NULL, linktext = NULL, firsttag = NULL; int taglen = 0, textlen = 0, linklen = 0, docitems = 0; PDOC_ITEM Last = NULL; @@ -215,7 +192,7 @@ static UINT SYSLINK_ParseText (SYSLINK_INFO *infoPtr, LPCWSTR Text) { if(*current == '<') { - if(!SYSLINK_StrCmpNIW(current, SL_LINKOPEN, 2) && (CurrentType == slText)) + if(!strncmpiW(current, SL_LINKOPEN, sizeof(SL_LINKOPEN)/sizeof(SL_LINKOPEN[0])) && (CurrentType == slText)) { BOOL ValidParam = FALSE, ValidLink = FALSE; @@ -243,14 +220,14 @@ static UINT SYSLINK_ParseText (SYSLINK_INFO *infoPtr, LPCWSTR Text) CheckParameter: /* compare the current position with all known parameters */ - if(!SYSLINK_StrCmpNIW(tmp, SL_HREF, 6)) + if(!strncmpiW(tmp, SL_HREF, sizeof(SL_HREF)/sizeof(SL_HREF[0]))) { taglen += 6; ValidParam = TRUE; CurrentParameter = &lpUrl; CurrentParameterLen = &lenUrl; } - else if(!SYSLINK_StrCmpNIW(tmp, SL_ID, 4)) + else if(!strncmpiW(tmp, SL_ID, sizeof(SL_ID)/sizeof(SL_ID[0]))) { taglen += 4; ValidParam = TRUE; @@ -324,7 +301,7 @@ CheckParameter: } } } - else if(!SYSLINK_StrCmpNIW(current, SL_LINKCLOSE, 4) && (CurrentType == slLink) && firsttag) + else if(!strncmpiW(current, SL_LINKCLOSE, sizeof(SL_LINKCLOSE)/sizeof(SL_LINKCLOSE[0])) && (CurrentType == slLink) && firsttag) { /* there's a <a...> tag opened, first add the previous text, if present */ if(textstart != NULL && textlen > 0 && firsttag > textstart)

1 0