wine-commits April 2024

wine-commits@winehq.org

1 participants
590 discussions

Dmitry Timoshkov : widl: In old style typelibs all types are public.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: b8921b414e867b1df01cd2ee2365ed8a3ab524f0 URL: https://gitlab.winehq.org/wine/wine/-/commit/b8921b414e867b1df01cd2ee2365ed… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Apr 3 19:13:17 2024 +0300 widl: In old style typelibs all types are public. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> --- tools/widl/write_sltg.c | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/tools/widl/write_sltg.c b/tools/widl/write_sltg.c index ac4db609919..7380d94016c 100644 --- a/tools/widl/write_sltg.c +++ b/tools/widl/write_sltg.c @@ -411,11 +411,6 @@ static void append_data(struct sltg_data *block, const void *data, int size) block->size = new_size; } -static void add_typedef_typeinfo(struct sltg_typelib *typelib, type_t *type) -{ - error("add_typedef_typeinfo: %s not implemented\n", type->name); -} - static void add_module_typeinfo(struct sltg_typelib *typelib, type_t *type) { error("add_module_typeinfo: %s not implemented\n", type->name); @@ -996,12 +991,8 @@ static void add_statement(struct sltg_typelib *typelib, const statement_t *stmt) LIST_FOR_EACH_ENTRY(ref, stmt->u.type_list, typeref_t, entry) { - /* if the type is public then add the typedef, otherwise attempt - * to add the aliased type */ - if (is_attr(ref->type->attrs, ATTR_PUBLIC)) - add_typedef_typeinfo(typelib, ref->type); - else - add_type_typeinfo(typelib, type_alias_get_aliasee_type(ref->type)); + /* in old style typelibs all types are public */ + add_type_typeinfo(typelib, ref->type); } break; }

1 0

Brendan Shanks : winecoreaudio: Correctly handle devices whose UID contains non-ASCII characters.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: a630b125ef4429c1c52c1272eba8b7939e7e6857 URL: https://gitlab.winehq.org/wine/wine/-/commit/a630b125ef4429c1c52c1272eba8b7… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Fri Apr 5 22:05:52 2024 -0700 winecoreaudio: Correctly handle devices whose UID contains non-ASCII characters. --- dlls/winecoreaudio.drv/coreaudio.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/dlls/winecoreaudio.drv/coreaudio.c b/dlls/winecoreaudio.drv/coreaudio.c index d78744455a8..a79a0d16839 100644 --- a/dlls/winecoreaudio.drv/coreaudio.c +++ b/dlls/winecoreaudio.drv/coreaudio.c @@ -325,7 +325,12 @@ static NTSTATUS unix_get_endpoint_ids(void *args) for(i = 0; i < params->num; i++){ const SIZE_T name_len = CFStringGetLength(info[i].name) + 1; - const SIZE_T device_len = CFStringGetLength(info[i].uid) + 1; + CFIndex device_len; + + CFStringGetBytes(info[i].uid, CFRangeMake(0, CFStringGetLength(info[i].uid)), kCFStringEncodingUTF8, + 0, false, NULL, 0, &device_len); + device_len++; /* for null terminator */ + needed += name_len * sizeof(WCHAR) + ((device_len + 1) & ~1); if(needed <= params->size){ @@ -336,7 +341,8 @@ static NTSTATUS unix_get_endpoint_ids(void *args) offset += name_len * sizeof(WCHAR); endpoint->device = offset; - CFStringGetCString(info[i].uid, (char *)params->endpoints + offset, params->size - offset, kCFStringEncodingUTF8); + CFStringGetBytes(info[i].uid, CFRangeMake(0, CFStringGetLength(info[i].uid)), kCFStringEncodingUTF8, + 0, false, (UInt8 *)params->endpoints + offset, params->size - offset, NULL); ((char *)params->endpoints)[offset + device_len - 1] = '\0'; offset += (device_len + 1) & ~1;

1 0

Paul Gofman : ntdll: Preserve untouched parts of xstate in set_thread_wow64_context().
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 14a51fa4460d958efc22c2fe108d90c37b6465c1 URL: https://gitlab.winehq.org/wine/wine/-/commit/14a51fa4460d958efc22c2fe108d90… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Mon Apr 8 17:37:00 2024 -0600 ntdll: Preserve untouched parts of xstate in set_thread_wow64_context(). --- dlls/ntdll/unix/signal_x86_64.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c index d8d16917942..4fc2727595d 100644 --- a/dlls/ntdll/unix/signal_x86_64.c +++ b/dlls/ntdll/unix/signal_x86_64.c @@ -1281,9 +1281,11 @@ NTSTATUS set_thread_wow64_context( HANDLE handle, const void *ctx, ULONG size ) { CONTEXT_EX *context_ex = (CONTEXT_EX *)(context + 1); XSAVE_AREA_HEADER *xs = (XSAVE_AREA_HEADER *)((char *)context_ex + context_ex->XState.Offset); + UINT64 mask = frame->xstate.Mask; if (xstate_compaction_enabled) frame->xstate.CompactionMask |= xstate_extended_features(); copy_xstate( &frame->xstate, xs, xs->Mask ); + if (xs->CompactionMask) frame->xstate.Mask |= mask & ~xs->CompactionMask; frame->restore_flags |= CONTEXT_XSTATE; } return STATUS_SUCCESS;

1 0

Paul Gofman : wow64: Support generic xstate in call_user_exception_dispatcher().
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 34e49897a39812837bc4c1cdcf4ca274185483c0 URL: https://gitlab.winehq.org/wine/wine/-/commit/34e49897a39812837bc4c1cdcf4ca2… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Mon Apr 8 16:52:19 2024 -0600 wow64: Support generic xstate in call_user_exception_dispatcher(). --- dlls/wow64/syscall.c | 26 ++++++-------------------- 1 file changed, 6 insertions(+), 20 deletions(-) diff --git a/dlls/wow64/syscall.c b/dlls/wow64/syscall.c index 40eb5327934..9ad1194b21e 100644 --- a/dlls/wow64/syscall.c +++ b/dlls/wow64/syscall.c @@ -197,17 +197,12 @@ static void __attribute__((used)) call_user_exception_dispatcher( EXCEPTION_RECO ULONG context_ptr; /* 004 */ EXCEPTION_RECORD32 rec; /* 008 */ I386_CONTEXT context; /* 058 */ - CONTEXT_EX32 context_ex; /* 324 */ - BYTE xstate[sizeof(XSTATE)+64]; /* 33c */ - DWORD align; /* 4bc */ } *stack; I386_CONTEXT ctx = { CONTEXT_I386_ALL }; CONTEXT_EX *context_ex, *src_ex = NULL; - ULONG flags; + ULONG flags, context_length; C_ASSERT( offsetof(struct exc_stack_layout32, context) == 0x58 ); - C_ASSERT( offsetof(struct exc_stack_layout32, xstate) == 0x33c ); - C_ASSERT( sizeof(struct exc_stack_layout32) == 0x4c0 ); pBTCpuGetContext( GetCurrentThread(), GetCurrentProcess(), NULL, &ctx ); @@ -231,29 +226,20 @@ static void __attribute__((used)) call_user_exception_dispatcher( EXCEPTION_RECO flags = ctx.ContextFlags; if (src_ex) flags |= CONTEXT_I386_XSTATE; - stack = (struct exc_stack_layout32 *)ULongToPtr( ctx.Esp & ~3 ) - 1; + RtlGetExtendedContextLength( flags, &context_length ); + + stack = (struct exc_stack_layout32 *)ULongToPtr( (ctx.Esp - offsetof(struct exc_stack_layout32, context) - context_length) & ~3 ); stack->rec_ptr = PtrToUlong( &stack->rec ); stack->context_ptr = PtrToUlong( &stack->context ); stack->rec = *rec; stack->context = ctx; RtlInitializeExtendedContext( &stack->context, flags, &context_ex ); + if (src_ex) RtlCopyExtendedContext( context_ex, WOW64_CONTEXT_XSTATE, src_ex ); + /* adjust Eip for breakpoints in software emulation (hardware exceptions already adjust Rip) */ if (rec->ExceptionCode == EXCEPTION_BREAKPOINT && (wow64info->CpuFlags & WOW64_CPUFLAGS_SOFTWARE)) stack->context.Eip--; - if (src_ex) - { - XSTATE *src_xs = (XSTATE *)((char *)src_ex + src_ex->XState.Offset); - XSTATE *dst_xs = (XSTATE *)((char *)context_ex + context_ex->XState.Offset); - - dst_xs->Mask = src_xs->Mask & ~(ULONG64)3; - dst_xs->CompactionMask = src_xs->CompactionMask; - if ((dst_xs->Mask & 4) && - src_ex->XState.Length >= sizeof(XSTATE) && - context_ex->XState.Length >= sizeof(XSTATE)) - memcpy( &dst_xs->YmmContext, &src_xs->YmmContext, sizeof(dst_xs->YmmContext) ); - } - ctx.Esp = PtrToUlong( stack ); ctx.Eip = pLdrSystemDllInitBlock->pKiUserExceptionDispatcher; ctx.EFlags &= ~(0x100|0x400|0x40000);

1 0

Paul Gofman : winhttp: Do not cache connection if suggested by request headers.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: ff97d3976ec0ee5444afddecfca5198cf2e53a1c URL: https://gitlab.winehq.org/wine/wine/-/commit/ff97d3976ec0ee5444afddecfca519… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Mon Apr 8 22:45:08 2024 -0600 winhttp: Do not cache connection if suggested by request headers. --- dlls/winhttp/request.c | 20 +++++++++----------- dlls/winhttp/tests/winhttp.c | 2 +- 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index 1b6cb1e2a45..207f1ec5c61 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -1922,7 +1922,7 @@ static DWORD refill_buffer( struct request *request, BOOL notify ) static void finished_reading( struct request *request ) { - BOOL close = FALSE, notify; + BOOL close = FALSE, close_request_headers; WCHAR connection[20]; DWORD size = sizeof(connection); @@ -1937,18 +1937,16 @@ static void finished_reading( struct request *request ) } else if (!wcscmp( request->version, L"HTTP/1.0" )) close = TRUE; - if (close) + size = sizeof(connection); + close_request_headers = + (!query_headers( request, WINHTTP_QUERY_CONNECTION | WINHTTP_QUERY_FLAG_REQUEST_HEADERS, NULL, connection, &size, NULL ) + || !query_headers( request, WINHTTP_QUERY_PROXY_CONNECTION | WINHTTP_QUERY_FLAG_REQUEST_HEADERS, NULL, connection, &size, NULL )) + && !wcsicmp( connection, L"close" ); + if (close || close_request_headers) { - size = sizeof(connection); - notify = (!query_headers( request, WINHTTP_QUERY_CONNECTION | WINHTTP_QUERY_FLAG_REQUEST_HEADERS, - NULL, connection, &size, NULL ) - || !query_headers( request, WINHTTP_QUERY_PROXY_CONNECTION | WINHTTP_QUERY_FLAG_REQUEST_HEADERS, - NULL, connection, &size, NULL )) - && !wcsicmp( connection, L"close" ); - - if (notify) send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, 0, 0 ); + if (close_request_headers) send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, 0, 0 ); netconn_release( request->netconn ); - if (notify) send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, 0, 0 ); + if (close_request_headers) send_callback( &request->hdr, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, 0, 0 ); } else cache_connection( request->netconn ); diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c index 5d0ccd11f52..ee01fd5ec72 100644 --- a/dlls/winhttp/tests/winhttp.c +++ b/dlls/winhttp/tests/winhttp.c @@ -2591,7 +2591,7 @@ static DWORD CALLBACK server_thread(LPVOID param) ok(!!strstr(buffer, "GET /cached"), "request not found.\n"); send(c, okmsg_length0, sizeof okmsg_length0 - 1, 0); r = server_receive_request(c, buffer, sizeof(buffer)); - todo_wine ok(!r, "got %d, buffer[0] %d.\n", r, buffer[0]); + ok(!r, "got %d, buffer[0] %d.\n", r, buffer[0]); } if (strstr(buffer, "GET /notcached")) {

1 0

Paul Gofman : winhttp/tests: Add some tests for actual connection caching.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 11b8243f062b651b25b3b62f771e32b7fe0d364c URL: https://gitlab.winehq.org/wine/wine/-/commit/11b8243f062b651b25b3b62f771e32… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Mon Apr 8 22:35:05 2024 -0600 winhttp/tests: Add some tests for actual connection caching. --- dlls/winhttp/tests/winhttp.c | 114 ++++++++++++++++++++++++++++++++++++++----- 1 file changed, 102 insertions(+), 12 deletions(-) diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c index 1724612202a..5d0ccd11f52 100644 --- a/dlls/winhttp/tests/winhttp.c +++ b/dlls/winhttp/tests/winhttp.c @@ -2197,6 +2197,12 @@ static const char okmsg[] = "Server: winetest\r\n" "\r\n"; +static const char okmsg_length0[] = +"HTTP/1.1 200 OK\r\n" +"Server: winetest\r\n" +"Content-length: 0\r\n" +"\r\n"; + static const char notokmsg[] = "HTTP/1.1 400 Bad Request\r\n" "\r\n"; @@ -2324,6 +2330,24 @@ static void create_websocket_accept(const char *key, char *buf, unsigned int buf CryptBinaryToStringA( (BYTE *)sha1, sizeof(sha1), CRYPT_STRING_BASE64, buf, &len); } +static int server_receive_request(int c, char *buffer, size_t buffer_size) +{ + int i, r; + + memset(buffer, 0, buffer_size); + for(i = 0; i < buffer_size - 1; i++) + { + r = recv(c, &buffer[i], 1, 0); + if (r != 1) + break; + if (i < 4) continue; + if (buffer[i - 2] == '\n' && buffer[i] == '\n' && + buffer[i - 3] == '\r' && buffer[i - 1] == '\r') + break; + } + return r; +} + static DWORD CALLBACK server_thread(LPVOID param) { struct server_info *si = param; @@ -2357,18 +2381,7 @@ static DWORD CALLBACK server_thread(LPVOID param) do { if (c == -1) c = accept(s, NULL, NULL); - - memset(buffer, 0, sizeof buffer); - for(i = 0; i < sizeof buffer - 1; i++) - { - r = recv(c, &buffer[i], 1, 0); - if (r != 1) - break; - if (i < 4) continue; - if (buffer[i - 2] == '\n' && buffer[i] == '\n' && - buffer[i - 3] == '\r' && buffer[i - 1] == '\r') - break; - } + server_receive_request(c, buffer, sizeof(buffer)); if (strstr(buffer, "GET /basic")) { send(c, okmsg, sizeof okmsg - 1, 0); @@ -2569,6 +2582,23 @@ static DWORD CALLBACK server_thread(LPVOID param) } send(c, okmsg, sizeof(okmsg) - 1, 0); } + + if (strstr(buffer, "GET /cached")) + { + send(c, okmsg_length0, sizeof okmsg_length0 - 1, 0); + r = server_receive_request(c, buffer, sizeof(buffer)); + ok(r > 0, "got %d.\n", r); + ok(!!strstr(buffer, "GET /cached"), "request not found.\n"); + send(c, okmsg_length0, sizeof okmsg_length0 - 1, 0); + r = server_receive_request(c, buffer, sizeof(buffer)); + todo_wine ok(!r, "got %d, buffer[0] %d.\n", r, buffer[0]); + } + if (strstr(buffer, "GET /notcached")) + { + send(c, okmsg, sizeof okmsg - 1, 0); + r = server_receive_request(c, buffer, sizeof(buffer)); + ok(!r, "got %d, buffer[0] %d.\n", r, buffer[0] ); + } shutdown(c, 2); closesocket(c); c = -1; @@ -5884,6 +5914,65 @@ static void test_client_cert_authentication(void) WinHttpCloseHandle( ses ); } +static void test_connection_cache(int port) +{ + HINTERNET ses, con, req; + DWORD status, size; + char buffer[256]; + BOOL ret; + + ses = WinHttpOpen(L"winetest", WINHTTP_ACCESS_TYPE_NO_PROXY, NULL, NULL, 0); + ok(ses != NULL, "failed to open session %lu\n", GetLastError()); + + con = WinHttpConnect(ses, L"localhost", port, 0); + ok(con != NULL, "failed to open a connection %lu\n", GetLastError()); + + req = WinHttpOpenRequest(con, L"GET", L"/cached", NULL, NULL, NULL, 0); + ok(req != NULL, "failed to open a request %lu\n", GetLastError()); + ret = WinHttpSendRequest(req, NULL, 0, NULL, 0, 0, 0); + ok(ret, "failed to send request %lu\n", GetLastError()); + ret = WinHttpReceiveResponse(req, NULL); + ok(ret, "failed to receive response %lu\n", GetLastError()); + size = sizeof(status); + ret = WinHttpQueryHeaders(req, WINHTTP_QUERY_STATUS_CODE|WINHTTP_QUERY_FLAG_NUMBER, NULL, &status, &size, NULL); + ok(ret, "failed to query status code %lu\n", GetLastError()); + ok(status == HTTP_STATUS_OK, "request failed unexpectedly %lu\n", status); + ret = WinHttpReadData(req, buffer, sizeof buffer, &size); + ok(ret, "failed to read data %lu\n", GetLastError()); + ok(!size, "got size %lu.\n", size); + WinHttpCloseHandle(req); + + req = WinHttpOpenRequest(con, L"GET", L"/cached", NULL, NULL, NULL, 0); + ok(req != NULL, "failed to open a request %lu\n", GetLastError()); + ret = WinHttpSendRequest(req, L"Connection: close", ~0ul, NULL, 0, 0, 0); + ok(ret, "failed to send request %lu\n", GetLastError()); + ret = WinHttpReceiveResponse(req, NULL); + ok(ret, "failed to receive response %lu\n", GetLastError()); + size = sizeof(status); + ret = WinHttpQueryHeaders(req, WINHTTP_QUERY_STATUS_CODE|WINHTTP_QUERY_FLAG_NUMBER, NULL, &status, &size, NULL); + ok(ret, "failed to query status code %lu\n", GetLastError()); + ok(status == HTTP_STATUS_OK, "request failed unexpectedly %lu\n", status); + ret = WinHttpReadData(req, buffer, sizeof buffer, &size); + ok(ret, "failed to read data %lu\n", GetLastError()); + ok(!size, "got size %lu.\n", size); + WinHttpCloseHandle(req); + + req = WinHttpOpenRequest(con, L"GET", L"/notcached", NULL, NULL, NULL, 0); + ok(req != NULL, "failed to open a request %lu\n", GetLastError()); + ret = WinHttpSendRequest(req, L"Connection: close", ~0ul, NULL, 0, 0, 0); + ok(ret, "failed to send request %lu\n", GetLastError()); + ret = WinHttpReceiveResponse(req, NULL); + ok(ret, "failed to receive response %lu\n", GetLastError()); + size = sizeof(status); + ret = WinHttpQueryHeaders(req, WINHTTP_QUERY_STATUS_CODE|WINHTTP_QUERY_FLAG_NUMBER, NULL, &status, &size, NULL); + ok(ret, "failed to query status code %lu\n", GetLastError()); + ok(status == HTTP_STATUS_OK, "request failed unexpectedly %lu\n", status); + WinHttpCloseHandle(req); + + WinHttpCloseHandle(con); + WinHttpCloseHandle(ses); +} + START_TEST (winhttp) { struct server_info si; @@ -5951,6 +6040,7 @@ START_TEST (winhttp) test_websocket(si.port); test_redirect(si.port); test_WinHttpGetProxyForUrl(si.port); + test_connection_cache(si.port); /* send the basic request again to shutdown the server thread */ test_basic_request(si.port, NULL, L"/quit");

1 0

Paul Gofman : winhttp: Set actual default receive response timeout to 21sec.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 5f93179744610bf322ce407b5dbb46162f833aa5 URL: https://gitlab.winehq.org/wine/wine/-/commit/5f93179744610bf322ce407b5dbb46… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Mon Apr 8 20:21:09 2024 -0600 winhttp: Set actual default receive response timeout to 21sec. --- dlls/winhttp/request.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c index 9fe8e1c4782..1b6cb1e2a45 100644 --- a/dlls/winhttp/request.c +++ b/dlls/winhttp/request.c @@ -43,6 +43,14 @@ WINE_DEFAULT_DEBUG_CHANNEL(winhttp); #define DEFAULT_KEEP_ALIVE_TIMEOUT 30000 +#define ACTUAL_DEFAULT_RECEIVE_RESPONSE_TIMEOUT 21000 + +static int request_receive_response_timeout( struct request *req ) +{ + if (req->receive_response_timeout == -1) return ACTUAL_DEFAULT_RECEIVE_RESPONSE_TIMEOUT; + return req->receive_response_timeout; +} + static const WCHAR *attribute_table[] = { L"Mime-Version", /* WINHTTP_QUERY_MIME_VERSION = 0 */ @@ -1658,7 +1666,7 @@ static DWORD open_connection( struct request *request ) return ret; } netconn_set_timeout( netconn, TRUE, request->send_timeout ); - netconn_set_timeout( netconn, FALSE, request->receive_response_timeout ); + netconn_set_timeout( netconn, FALSE, request_receive_response_timeout( request )); request->netconn = netconn; @@ -1696,7 +1704,7 @@ static DWORD open_connection( struct request *request ) TRACE("using connection %p\n", netconn); netconn_set_timeout( netconn, TRUE, request->send_timeout ); - netconn_set_timeout( netconn, FALSE, request->receive_response_timeout ); + netconn_set_timeout( netconn, FALSE, request_receive_response_timeout( request )); request->netconn = netconn; } @@ -2329,7 +2337,7 @@ static DWORD send_request( struct request *request, const WCHAR *headers, DWORD if (!chunked && content_length <= optional_len) { - netconn_set_timeout( request->netconn, FALSE, request->receive_response_timeout ); + netconn_set_timeout( request->netconn, FALSE, request_receive_response_timeout( request )); request->read_reply_status = read_reply( request ); if (request->state == REQUEST_RESPONSE_STATE_READ_RESPONSE_QUEUED) request->state = REQUEST_RESPONSE_STATE_READ_RESPONSE_QUEUED_REPLY_RECEIVED; @@ -2932,7 +2940,7 @@ static DWORD receive_response( struct request *request ) } /* fallthrough */ case REQUEST_RESPONSE_STATE_READ_RESPONSE_QUEUED_REQUEST_SENT: - netconn_set_timeout( request->netconn, FALSE, request->receive_response_timeout ); + netconn_set_timeout( request->netconn, FALSE, request_receive_response_timeout( request )); request->read_reply_status = read_reply( request ); request->state = REQUEST_RESPONSE_STATE_REPLY_RECEIVED; break;

1 0

Hans Leidekker : msi: Install global assemblies before running deferred custom actions.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 7f0ca9c284d15d03f14a8fefefac834990d622cd URL: https://gitlab.winehq.org/wine/wine/-/commit/7f0ca9c284d15d03f14a8fefefac83… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 5 09:22:06 2024 +0200 msi: Install global assemblies before running deferred custom actions. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56493 --- dlls/msi/action.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 06f399223f4..e3c7c23b2bd 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -5139,16 +5139,6 @@ static UINT ACTION_InstallFinalize(MSIPACKAGE *package) MSIFILE *file; MSIFILEPATCH *patch; - /* first do the same as an InstallExecute */ - rc = execute_script(package, SCRIPT_INSTALL); - if (rc != ERROR_SUCCESS) - return rc; - - /* then handle commit actions */ - rc = execute_script(package, SCRIPT_COMMIT); - if (rc != ERROR_SUCCESS) - return rc; - /* install global assemblies */ LIST_FOR_EACH_ENTRY( file, &package->files, MSIFILE, entry ) { @@ -5187,6 +5177,16 @@ static UINT ACTION_InstallFinalize(MSIPACKAGE *package) } } + /* first do the same as an InstallExecute */ + rc = execute_script(package, SCRIPT_INSTALL); + if (rc != ERROR_SUCCESS) + return rc; + + /* then handle commit actions */ + rc = execute_script(package, SCRIPT_COMMIT); + if (rc != ERROR_SUCCESS) + return rc; + if (is_full_uninstall(package)) rc = ACTION_UnpublishProduct(package);

1 0

Hans Leidekker : msi/tests: Get rid of workarounds for old Windows versions.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: 738ec8285583248a84349939734ed600a47ba73d URL: https://gitlab.winehq.org/wine/wine/-/commit/738ec8285583248a84349939734ed6… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 5 16:25:09 2024 +0200 msi/tests: Get rid of workarounds for old Windows versions. --- dlls/msi/tests/action.c | 260 +-- dlls/msi/tests/automation.c | 46 +- dlls/msi/tests/custom.c | 4 +- dlls/msi/tests/install.c | 115 +- dlls/msi/tests/msi.c | 4144 ++++++++++++++++--------------------------- dlls/msi/tests/package.c | 135 +- dlls/msi/tests/patch.c | 115 +- dlls/msi/tests/record.c | 2 +- dlls/msi/tests/source.c | 1485 ++++++---------- include/msi.h | 2 +- 10 files changed, 2203 insertions(+), 4105 deletions(-)

1 0

Hans Leidekker : msi/tests: Try restarting tests elevated.
by Alexandre Julliard 09 Apr '24

09 Apr '24

Module: wine Branch: master Commit: a6eb51f0e595b2e5ddc521fb61b10d72849dd0f2 URL: https://gitlab.winehq.org/wine/wine/-/commit/a6eb51f0e595b2e5ddc521fb61b10d… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 5 09:29:18 2024 +0200 msi/tests: Try restarting tests elevated. Wine starts processes non-elevated since e92ba2de43d7afbe0704b11b29f7c30f44dfaeca which means that many tests are skipped now. --- dlls/msi/tests/action.c | 105 ++++++++++++++++++++++++++++------------- dlls/msi/tests/automation.c | 4 +- dlls/msi/tests/install.c | 111 ++++++++++++++++++-------------------------- dlls/msi/tests/msi.c | 26 ++++++----- dlls/msi/tests/package.c | 6 ++- dlls/msi/tests/patch.c | 8 ++-- dlls/msi/tests/source.c | 10 ++-- dlls/msi/tests/utils.h | 3 +- 8 files changed, 153 insertions(+), 120 deletions(-)

1 0

← Newer
1
...
45
46
47
48
49
50
51
...
59
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits April 2024