wine-commits December 2023

wine-commits@winehq.org

1 participants
418 discussions

Alexandre Julliard : ntdll: Export KeAddSystemServiceTable() instead of the KeServiceDescriptorTable variable.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: 0fea0db866d843c14070a45b3f5e5fb26d4c5cb9 URL: https://gitlab.winehq.org/wine/wine/-/commit/0fea0db866d843c14070a45b3f5e5f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 6 12:42:34 2023 +0100 ntdll: Export KeAddSystemServiceTable() instead of the KeServiceDescriptorTable variable. --- dlls/ntdll/unix/loader.c | 15 +++++++++++++++ dlls/win32u/syscall.c | 15 +++------------ include/wine/unixlib.h | 3 ++- 3 files changed, 20 insertions(+), 13 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index ee71584b953..803d8079213 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -625,6 +625,21 @@ void start_server( BOOL debug ) } +/*********************************************************************** + * KeAddSystemServiceTable + */ +BOOLEAN KeAddSystemServiceTable( ULONG_PTR *funcs, ULONG_PTR *counters, ULONG limit, + BYTE *arguments, ULONG index ) +{ + if (index >= ARRAY_SIZE(KeServiceDescriptorTable)) return FALSE; + KeServiceDescriptorTable[index].ServiceTable = funcs; + KeServiceDescriptorTable[index].CounterTable = counters; + KeServiceDescriptorTable[index].ServiceLimit = limit; + KeServiceDescriptorTable[index].ArgumentTable = arguments; + return TRUE; +} + + /************************************************************************* * map_so_dll * diff --git a/dlls/win32u/syscall.c b/dlls/win32u/syscall.c index 2320ac18744..c03118821d9 100644 --- a/dlls/win32u/syscall.c +++ b/dlls/win32u/syscall.c @@ -36,9 +36,9 @@ ULONG_PTR zero_bits = 0; -static void * const syscalls[] = +static ULONG_PTR syscalls[] = { -#define SYSCALL_ENTRY(id,name,args) name, +#define SYSCALL_ENTRY(id,name,args) (ULONG_PTR)name, #ifdef _WIN64 ALL_SYSCALLS64 #else @@ -58,14 +58,6 @@ static BYTE arguments[ARRAY_SIZE(syscalls)] = #undef SYSCALL_ENTRY }; -static const SYSTEM_SERVICE_TABLE syscall_table = -{ - (ULONG_PTR *)syscalls, - 0, - ARRAY_SIZE(syscalls), - arguments -}; - static NTSTATUS init( void *args ) { #ifdef _WIN64 @@ -77,8 +69,7 @@ static NTSTATUS init( void *args ) zero_bits = (ULONG_PTR)info.HighestUserAddress | 0x7fffffff; } #endif - - KeServiceDescriptorTable[1] = syscall_table; + KeAddSystemServiceTable( syscalls, NULL, ARRAY_SIZE(syscalls), arguments, 1 ); return STATUS_SUCCESS; } diff --git a/include/wine/unixlib.h b/include/wine/unixlib.h index 2a615179e64..04978dcd680 100644 --- a/include/wine/unixlib.h +++ b/include/wine/unixlib.h @@ -31,7 +31,6 @@ typedef NTSTATUS (*unixlib_entry_t)( void *args ); extern DECLSPEC_EXPORT const unixlib_entry_t __wine_unix_call_funcs[]; extern DECLSPEC_EXPORT const unixlib_entry_t __wine_unix_call_wow64_funcs[]; -extern DECLSPEC_EXPORT SYSTEM_SERVICE_TABLE KeServiceDescriptorTable[4]; /* some useful helpers from ntdll */ NTSYSAPI const char *ntdll_get_build_dir(void); @@ -84,6 +83,8 @@ NTSYSAPI void ntdll_set_exception_jmp_buf( __wine_jmp_buf *jmp ); } \ } while (0); +NTSYSAPI BOOLEAN KeAddSystemServiceTable( ULONG_PTR *funcs, ULONG_PTR *counters, ULONG limit, + BYTE *arguments, ULONG index ); NTSYSAPI NTSTATUS KeUserModeCallback( ULONG id, const void *args, ULONG len, void **ret_ptr, ULONG *ret_len ); /* wide char string functions */

1 0

Alexandre Julliard : ntdll: Use posix_spawn() to start the server.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: 48182726e93be7a573c989c822b52eb56d5fbad9 URL: https://gitlab.winehq.org/wine/wine/-/commit/48182726e93be7a573c989c822b52e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 6 11:38:45 2023 +0100 ntdll: Use posix_spawn() to start the server. Based on a patch by Brendan Shanks. --- dlls/ntdll/unix/loader.c | 40 +++++++++++++++++++--------------------- 1 file changed, 19 insertions(+), 21 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 7b7005d282d..ee71584b953 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -30,6 +30,7 @@ #include <stdarg.h> #include <stdio.h> #include <signal.h> +#include <spawn.h> #include <string.h> #include <stdlib.h> #include <sys/types.h> @@ -69,7 +70,6 @@ # include <mach/mach_error.h> # include <mach-o/getsect.h> # include <crt_externs.h> -# include <spawn.h> # ifndef _POSIX_SPAWN_DISABLE_ASLR # define _POSIX_SPAWN_DISABLE_ASLR 0x0100 # endif @@ -77,6 +77,7 @@ #ifdef __ANDROID__ # include <jni.h> #endif +extern char **environ; #include "ntstatus.h" #define WIN32_NO_STATUS @@ -309,11 +310,14 @@ static char *build_path( const char *dir, const char *name ) /* build a path to a binary and exec it */ -static void build_path_and_exec( const char *dir, const char *name, char **argv ) +static int build_path_and_exec( pid_t *pid, const char *dir, const char *name, char **argv ) { + int ret; + argv[0] = build_path( dir, name ); - execv( argv[0], argv ); + ret = posix_spawn( pid, argv[0], NULL, NULL, argv, environ ); free( argv[0] ); + return ret; } @@ -567,7 +571,7 @@ NTSTATUS exec_wineloader( char **argv, int socketfd, const pe_image_info_t *pe_i * * Exec a new wine server. */ -static void exec_wineserver( char **argv ) +static int exec_wineserver( pid_t *pid, char **argv ) { char *path; @@ -576,22 +580,20 @@ static void exec_wineserver( char **argv ) if (!is_win64) /* look for 64-bit server */ { char *loader = realpath_dirname( build_path( build_dir, "loader/wine64" )); - if (loader) build_path_and_exec( loader, "../server/wineserver", argv ); + if (loader && !build_path_and_exec( pid, loader, "../server/wineserver", argv )) return 0; } - build_path_and_exec( build_dir, "server/wineserver", argv ); - return; + return build_path_and_exec( pid, build_dir, "server/wineserver", argv ); } - build_path_and_exec( bin_dir, "wineserver", argv ); - - if ((path = getenv( "WINESERVER" ))) build_path_and_exec( "", path, argv ); + if (!build_path_and_exec( pid, bin_dir, "wineserver", argv )) return 0; + if ((path = getenv( "WINESERVER" )) && !build_path_and_exec( pid, "", path, argv )) return 0; if ((path = getenv( "PATH" ))) { for (path = strtok( strdup( path ), ":" ); path; path = strtok( NULL, ":" )) - build_path_and_exec( path, "wineserver", argv ); + if (!build_path_and_exec( pid, path, "wineserver", argv )) return 0; } - build_path_and_exec( BINDIR, "wineserver", argv ); + return build_path_and_exec( pid, BINDIR, "wineserver", argv ); } @@ -609,15 +611,11 @@ void start_server( BOOL debug ) if (!started) { int status; - int pid = fork(); - if (pid == -1) fatal_error( "fork: %s", strerror(errno) ); - if (!pid) - { - argv[1] = debug ? debug_flag : NULL; - argv[2] = NULL; - exec_wineserver( argv ); - fatal_error( "could not exec wineserver\n" ); - } + pid_t pid; + + argv[1] = debug ? debug_flag : NULL; + argv[2] = NULL; + if (exec_wineserver( &pid, argv )) fatal_error( "could not exec wineserver\n" ); waitpid( pid, &status, 0 ); status = WIFEXITED(status) ? WEXITSTATUS(status) : 1; if (status == 2) return; /* server lock held by someone else, will retry later */

1 0

Alexandre Julliard : ntdll: Add a helper to build a path and exec it.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: 7c777af498cbc93a95292577f5e613fd0d8b34ce URL: https://gitlab.winehq.org/wine/wine/-/commit/7c777af498cbc93a95292577f5e613… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 6 10:48:06 2023 +0100 ntdll: Add a helper to build a path and exec it. --- dlls/ntdll/unix/loader.c | 42 +++++++++++++++++++++--------------------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 090f9906e9f..7b7005d282d 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -297,14 +297,26 @@ static char *build_path( const char *dir, const char *name ) size_t len = strlen( dir ); char *ret = malloc( len + strlen( name ) + 2 ); - memcpy( ret, dir, len ); - if (len && ret[len - 1] != '/') ret[len++] = '/'; - if (name[0] == '/') name++; + if (len) + { + memcpy( ret, dir, len ); + if (ret[len - 1] != '/') ret[len++] = '/'; + if (name[0] == '/') name++; + } strcpy( ret + len, name ); return ret; } +/* build a path to a binary and exec it */ +static void build_path_and_exec( const char *dir, const char *name, char **argv ) +{ + argv[0] = build_path( dir, name ); + execv( argv[0], argv ); + free( argv[0] ); +} + + static const char *get_pe_dir( WORD machine ) { if (!machine) machine = current_machine; @@ -564,34 +576,22 @@ static void exec_wineserver( char **argv ) if (!is_win64) /* look for 64-bit server */ { char *loader = realpath_dirname( build_path( build_dir, "loader/wine64" )); - if (loader) - { - argv[0] = build_path( loader, "../server/wineserver" ); - execv( argv[0], argv ); - } + if (loader) build_path_and_exec( loader, "../server/wineserver", argv ); } - argv[0] = build_path( build_dir, "server/wineserver" ); - execv( argv[0], argv ); + build_path_and_exec( build_dir, "server/wineserver", argv ); return; } - argv[0] = build_path( bin_dir, "wineserver" ); - execv( argv[0], argv ); + build_path_and_exec( bin_dir, "wineserver", argv ); - argv[0] = getenv( "WINESERVER" ); - if (argv[0]) execv( argv[0], argv ); + if ((path = getenv( "WINESERVER" ))) build_path_and_exec( "", path, argv ); if ((path = getenv( "PATH" ))) { for (path = strtok( strdup( path ), ":" ); path; path = strtok( NULL, ":" )) - { - argv[0] = build_path( path, "wineserver" ); - execvp( argv[0], argv ); - } + build_path_and_exec( path, "wineserver", argv ); } - - argv[0] = build_path( BINDIR, "wineserver" ); - execv( argv[0], argv ); + build_path_and_exec( BINDIR, "wineserver", argv ); }

1 0

Nikolay Sivov : ntdll: Use public type for SystemSupportedProcessorArchitectures returned data.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: 26f11bdfe5365d8fa9354b530638b42de6aa2c42 URL: https://gitlab.winehq.org/wine/wine/-/commit/26f11bdfe5365d8fa9354b530638b4… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Dec 5 17:41:33 2023 +0100 ntdll: Use public type for SystemSupportedProcessorArchitectures returned data. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/ntdll/process.c | 20 +++++++++--------- dlls/ntdll/tests/wow64.c | 48 +++++++++++++++++++++----------------------- dlls/ntdll/unix/system.c | 23 +++++++++++++++------ dlls/wow64/system.c | 2 +- include/winnt.h | 11 ++++++++++ programs/regsvr32/regsvr32.c | 9 +++++---- programs/wineboot/wineboot.c | 17 ++++++++-------- 7 files changed, 76 insertions(+), 54 deletions(-)

1 0

Nikolay Sivov : include: Add missing enum to wdm.h.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: dbeaaf2b0fa4bab687b434a23f258088c07a7fd3 URL: https://gitlab.winehq.org/wine/wine/-/commit/dbeaaf2b0fa4bab687b434a23f2580… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Dec 5 21:27:13 2023 +0100 include: Add missing enum to wdm.h. --- include/ddk/wdm.h | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/include/ddk/wdm.h b/include/ddk/wdm.h index 7dcec028a50..54b9e6aa199 100644 --- a/include/ddk/wdm.h +++ b/include/ddk/wdm.h @@ -1657,8 +1657,12 @@ static inline void IoCopyCurrentIrpStackLocationToNext(IRP *irp) next->Control = 0; } -#define KernelMode 0 -#define UserMode 1 +typedef enum _MODE +{ + KernelMode, + UserMode, + MaximumMode +} MODE; /* directory object access rights */ #define DIRECTORY_QUERY 0x0001

1 0

Brendan Shanks : winemac: Use RTL_CONSTANT_STRING.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: e4ec6f87e0be4c4ea9908bc22b59c829a2e4ad67 URL: https://gitlab.winehq.org/wine/wine/-/commit/e4ec6f87e0be4c4ea9908bc22b59c8… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Tue Dec 5 14:32:24 2023 -0800 winemac: Use RTL_CONSTANT_STRING. --- dlls/winemac.drv/clipboard.c | 3 +-- dlls/winemac.drv/display.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/winemac.drv/clipboard.c b/dlls/winemac.drv/clipboard.c index ef0c3ae9c25..5a1065cf7c4 100644 --- a/dlls/winemac.drv/clipboard.c +++ b/dlls/winemac.drv/clipboard.c @@ -1665,8 +1665,7 @@ void macdrv_UpdateClipboard(void) if (!NtUserIsWindow(clipboard_manager)) { - UNICODE_STRING str; - RtlInitUnicodeString(&str, clipboard_classname); + UNICODE_STRING str = RTL_CONSTANT_STRING(clipboard_classname); clipboard_manager = NtUserFindWindowEx(NULL, NULL, &str, NULL, 0); if (!clipboard_manager) { diff --git a/dlls/winemac.drv/display.c b/dlls/winemac.drv/display.c index 14c09649211..9bc8c79896f 100644 --- a/dlls/winemac.drv/display.c +++ b/dlls/winemac.drv/display.c @@ -149,7 +149,7 @@ static BOOL write_display_settings(HKEY parent_hkey, CGDirectDisplayID displayID char display_key_name[19]; HKEY display_hkey; CGDisplayModeRef display_mode; - UNICODE_STRING str; + UNICODE_STRING str = RTL_CONSTANT_STRING(pixelencodingW); DWORD val; CFStringRef pixel_encoding; size_t len; @@ -189,7 +189,6 @@ static BOOL write_display_settings(HKEY parent_hkey, CGDirectDisplayID displayID CFStringGetCharacters(pixel_encoding, CFRangeMake(0, len), (UniChar*)buf); buf[len] = 0; CFRelease(pixel_encoding); - RtlInitUnicodeString(&str, pixelencodingW); if (NtSetValueKey(display_hkey, &str, 0, REG_SZ, (const BYTE*)buf, (len + 1) * sizeof(WCHAR))) goto fail;

1 0

Gabriel Ivăncescu : mshtml: Get rid of outer_window member in HTMLDocumentNode.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: e5c4acc459f135bd6c59d8d7a5de8c601e1227cf URL: https://gitlab.winehq.org/wine/wine/-/commit/e5c4acc459f135bd6c59d8d7a5de8c… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Tue Dec 5 19:42:19 2023 +0200 mshtml: Get rid of outer_window member in HTMLDocumentNode. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmldoc.c | 3 +-- dlls/mshtml/htmlframe.c | 6 +++--- dlls/mshtml/mshtml_private.h | 1 - dlls/mshtml/mutation.c | 2 +- dlls/mshtml/nsevents.c | 11 ++++++----- dlls/mshtml/olecmd.c | 4 ++-- dlls/mshtml/pluginhost.c | 4 ++-- dlls/mshtml/secmgr.c | 8 ++++---- 8 files changed, 19 insertions(+), 20 deletions(-) diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c index bc3782d54cc..a20472d22b8 100644 --- a/dlls/mshtml/htmldoc.c +++ b/dlls/mshtml/htmldoc.c @@ -4895,7 +4895,7 @@ static void HTMLDocumentNode_on_advise(IUnknown *iface, cp_static_data_t *cp) { HTMLDocumentNode *This = CONTAINING_RECORD((IHTMLDocument2*)iface, HTMLDocumentNode, IHTMLDocument2_iface); - if(This->outer_window) + if(This->window && This->window->base.outer_window) update_doc_cp_events(This, cp); } @@ -6208,7 +6208,6 @@ static HTMLDocumentNode *alloc_doc_node(HTMLDocumentObj *doc_obj, HTMLInnerWindo doc->IDocumentRange_iface.lpVtbl = &DocumentRangeVtbl; doc->doc_obj = doc_obj; - doc->outer_window = window ? window->base.outer_window : NULL; doc->window = window; if(window) diff --git a/dlls/mshtml/htmlframe.c b/dlls/mshtml/htmlframe.c index 1a78dc3cc07..28bf42e2569 100644 --- a/dlls/mshtml/htmlframe.c +++ b/dlls/mshtml/htmlframe.c @@ -50,7 +50,7 @@ static HRESULT set_frame_doc(HTMLFrameBase *frame, nsIDOMDocument *nsdoc) window = mozwindow_to_window(mozwindow); if(!window && frame->element.node.doc->browser) { hres = create_outer_window(frame->element.node.doc->browser, mozwindow, - frame->element.node.doc->outer_window, &window); + frame->element.node.doc->window->base.outer_window, &window); /* Don't hold ref to the created window; the parent keeps ref to it */ if(SUCCEEDED(hres)) @@ -132,7 +132,7 @@ static HRESULT WINAPI HTMLFrameBase_put_src(IHTMLFrameBase *iface, BSTR v) TRACE("(%p)->(%s)\n", This, debugstr_w(v)); - if(!This->content_window || !This->element.node.doc || !This->element.node.doc->outer_window) { + if(!This->content_window || !This->element.node.doc || !This->element.node.doc->window || !This->element.node.doc->window->base.outer_window) { nsAString nsstr; nsresult nsres; @@ -150,7 +150,7 @@ static HRESULT WINAPI HTMLFrameBase_put_src(IHTMLFrameBase *iface, BSTR v) return S_OK; } - return navigate_url(This->content_window, v, This->element.node.doc->outer_window->uri, BINDING_NAVIGATED); + return navigate_url(This->content_window, v, This->element.node.doc->window->base.outer_window->uri, BINDING_NAVIGATED); } static HRESULT WINAPI HTMLFrameBase_get_src(IHTMLFrameBase *iface, BSTR *p) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 40dcb0ebfab..db37375c089 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -940,7 +940,6 @@ struct HTMLDocumentNode { nsIDocumentObserver nsIDocumentObserver_iface; ConnectionPointContainer cp_container; - HTMLOuterWindow *outer_window; HTMLInnerWindow *window; HTMLDocumentObj *doc_obj; diff --git a/dlls/mshtml/mutation.c b/dlls/mshtml/mutation.c index a1438debd6f..845c189f98b 100644 --- a/dlls/mshtml/mutation.c +++ b/dlls/mshtml/mutation.c @@ -314,7 +314,7 @@ static nsresult run_end_load(HTMLDocumentNode *This, nsISupports *arg1, nsISuppo if(This->window == window) { window->dom_interactive_time = get_time_stamp(); - set_ready_state(This->outer_window, READYSTATE_INTERACTIVE); + set_ready_state(This->window->base.outer_window, READYSTATE_INTERACTIVE); } IHTMLWindow2_Release(&window->base.IHTMLWindow2_iface); return NS_OK; diff --git a/dlls/mshtml/nsevents.c b/dlls/mshtml/nsevents.c index 8c6631626d1..482d3196770 100644 --- a/dlls/mshtml/nsevents.c +++ b/dlls/mshtml/nsevents.c @@ -336,7 +336,7 @@ static nsresult handle_load(HTMLDocumentNode *doc, nsIDOMEvent *event) TRACE("(%p)\n", doc); - if(!doc->outer_window) + if(!doc->window || !doc->window->base.outer_window) return NS_ERROR_FAILURE; if(doc->doc_obj && doc->doc_obj->doc_node == doc) { doc_obj = doc->doc_obj; @@ -348,7 +348,8 @@ static nsresult handle_load(HTMLDocumentNode *doc, nsIDOMEvent *event) handle_docobj_load(doc_obj); doc->window->dom_complete_time = get_time_stamp(); - set_ready_state(doc->outer_window, READYSTATE_COMPLETE); + if(doc->window->base.outer_window) + set_ready_state(doc->window->base.outer_window, READYSTATE_COMPLETE); if(doc_obj) { if(doc_obj->view_sink) @@ -358,9 +359,9 @@ static nsresult handle_load(HTMLDocumentNode *doc, nsIDOMEvent *event) update_title(doc_obj); - if(doc_obj->doc_object_service && !(doc->outer_window->load_flags & BINDING_REFRESH)) + if(doc_obj->doc_object_service && doc->window->base.outer_window && !(doc->window->base.outer_window->load_flags & BINDING_REFRESH)) IDocObjectService_FireDocumentComplete(doc_obj->doc_object_service, - &doc->outer_window->base.IHTMLWindow2_iface, 0); + &doc->window->base.outer_window->base.IHTMLWindow2_iface, 0); IUnknown_Release(doc_obj->outer_unk); } @@ -516,7 +517,7 @@ static nsIDOMEventTarget *get_default_document_target(HTMLDocumentNode *doc) nsISupports *target_iface; nsresult nsres; - target_iface = doc->window ? (nsISupports*)doc->outer_window->nswindow : (nsISupports*)doc->dom_document; + target_iface = doc->window && doc->window->base.outer_window ? (nsISupports*)doc->window->base.outer_window->nswindow : (nsISupports*)doc->dom_document; nsres = nsISupports_QueryInterface(target_iface, &IID_nsIDOMEventTarget, (void**)&target); return NS_SUCCEEDED(nsres) ? target : NULL; } diff --git a/dlls/mshtml/olecmd.c b/dlls/mshtml/olecmd.c index 76ed84341cf..650c44c9d2b 100644 --- a/dlls/mshtml/olecmd.c +++ b/dlls/mshtml/olecmd.c @@ -490,10 +490,10 @@ static HRESULT exec_refresh(HTMLDocumentNode *doc, DWORD nCmdexecopt, VARIANT *p } } - if(!doc->outer_window) + if(!doc->window || !doc->window->base.outer_window) return E_UNEXPECTED; - return reload_page(doc->outer_window); + return reload_page(doc->window->base.outer_window); } static HRESULT exec_stop(HTMLDocumentNode *doc, DWORD nCmdexecopt, VARIANT *pvaIn, VARIANT *pvaOut) diff --git a/dlls/mshtml/pluginhost.c b/dlls/mshtml/pluginhost.c index f910600de18..5f9e3588239 100644 --- a/dlls/mshtml/pluginhost.c +++ b/dlls/mshtml/pluginhost.c @@ -2227,12 +2227,12 @@ static HRESULT WINAPI PHServiceProvider_QueryService(IServiceProvider *iface, RE TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); - if(!This->doc || !This->doc->outer_window) { + if(!This->doc || !This->doc->window || !This->doc->window->base.outer_window) { *ppv = NULL; return E_NOINTERFACE; } - return IServiceProvider_QueryService(&This->doc->outer_window->base.IServiceProvider_iface, + return IServiceProvider_QueryService(&This->doc->window->base.outer_window->base.IServiceProvider_iface, guidService, riid, ppv); } diff --git a/dlls/mshtml/secmgr.c b/dlls/mshtml/secmgr.c index 24c7ef2c817..ea53166f354 100644 --- a/dlls/mshtml/secmgr.c +++ b/dlls/mshtml/secmgr.c @@ -76,10 +76,10 @@ static HRESULT WINAPI InternetHostSecurityManager_ProcessUrlAction(IInternetHost TRACE("(%p)->(%ld %p %ld %p %ld %lx %lx)\n", This, dwAction, pPolicy, cbPolicy, pContext, cbContext, dwFlags, dwReserved); - if(!This->outer_window) + if(!This->window || !This->window->base.outer_window) return E_UNEXPECTED; - url = This->outer_window->url ? This->outer_window->url : L"about:blank"; + url = This->window->base.outer_window->url ? This->window->base.outer_window->url : L"about:blank"; return IInternetSecurityManager_ProcessUrlAction(get_security_manager(), url, dwAction, pPolicy, cbPolicy, pContext, cbContext, dwFlags, dwReserved); @@ -181,10 +181,10 @@ static HRESULT WINAPI InternetHostSecurityManager_QueryCustomPolicy(IInternetHos TRACE("(%p)->(%s %p %p %p %ld %lx)\n", This, debugstr_guid(guidKey), ppPolicy, pcbPolicy, pContext, cbContext, dwReserved); - if(!This->outer_window) + if(!This->window || !This->window->base.outer_window) return E_UNEXPECTED; - url = This->outer_window->url ? This->outer_window->url : L"about:blank"; + url = This->window->base.outer_window->url ? This->window->base.outer_window->url : L"about:blank"; hres = IInternetSecurityManager_QueryCustomPolicy(get_security_manager(), url, guidKey, ppPolicy, pcbPolicy, pContext, cbContext, dwReserved);

1 0

Gabriel Ivăncescu : mshtml: Use inner window's outer_window when checking if it's interactive for edit mode.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: ff617c838fb5ef26fc0a6ed13f74953157c90219 URL: https://gitlab.winehq.org/wine/wine/-/commit/ff617c838fb5ef26fc0a6ed13f7495… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Tue Dec 5 19:42:19 2023 +0200 mshtml: Use inner window's outer_window when checking if it's interactive for edit mode. We can't rely on dom_interactive_time either because setup_edit_mode can reload the page (and sets it uninitialized). Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/editor.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/editor.c b/dlls/mshtml/editor.c index e4a16956e46..7d1c55dc56f 100644 --- a/dlls/mshtml/editor.c +++ b/dlls/mshtml/editor.c @@ -146,7 +146,7 @@ static DWORD query_ns_edit_status(HTMLDocumentNode *doc, const char *nscmd) nsICommandParams *nsparam; cpp_bool b = FALSE; - if(doc->browser->usermode != EDITMODE || doc->outer_window->readystate < READYSTATE_INTERACTIVE) + if(doc->browser->usermode != EDITMODE || !doc->window->base.outer_window || doc->window->base.outer_window->readystate < READYSTATE_INTERACTIVE) return OLECMDF_SUPPORTED; if(nscmd) { @@ -180,7 +180,7 @@ static DWORD query_align_status(HTMLDocumentNode *doc, const WCHAR *align) cpp_bool b; nsresult nsres; - if(doc->browser->usermode != EDITMODE || doc->outer_window->readystate < READYSTATE_INTERACTIVE) + if(doc->browser->usermode != EDITMODE || !doc->window->base.outer_window || doc->window->base.outer_window->readystate < READYSTATE_INTERACTIVE) return OLECMDF_SUPPORTED; if(!doc->html_document) { @@ -669,7 +669,7 @@ static HRESULT query_justify(HTMLDocumentNode *doc, OLECMD *cmd) case IDM_JUSTIFYLEFT: TRACE("(%p) IDM_JUSTIFYLEFT\n", doc); /* FIXME: We should set OLECMDF_LATCHED only if it's set explicitly. */ - if(doc->browser->usermode != EDITMODE || doc->outer_window->readystate < READYSTATE_INTERACTIVE) + if(doc->browser->usermode != EDITMODE || !doc->window->base.outer_window || doc->window->base.outer_window->readystate < READYSTATE_INTERACTIVE) cmd->cmdf = OLECMDF_SUPPORTED; else cmd->cmdf = OLECMDF_SUPPORTED | OLECMDF_ENABLED;

1 0

Gabriel Ivăncescu : mshtml: Skip navigation handling on anchor/area elements for detached documents.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: db12c470c8dc9c0dd8f137124d93fb48776eb9f4 URL: https://gitlab.winehq.org/wine/wine/-/commit/db12c470c8dc9c0dd8f137124d93fb… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Tue Dec 5 19:42:19 2023 +0200 mshtml: Skip navigation handling on anchor/area elements for detached documents. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmlanchor.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/htmlanchor.c b/dlls/mshtml/htmlanchor.c index 5398f87ce55..b3bdd3584d7 100644 --- a/dlls/mshtml/htmlanchor.c +++ b/dlls/mshtml/htmlanchor.c @@ -48,12 +48,15 @@ static HRESULT navigate_href_new_window(HTMLElement *element, nsAString *href_st IUri *uri; HRESULT hres; + if(!element->node.doc->window->base.outer_window) + return S_OK; + nsAString_GetData(href_str, &href); - hres = create_relative_uri(element->node.doc->outer_window, href, &uri); + hres = create_relative_uri(element->node.doc->window->base.outer_window, href, &uri); if(FAILED(hres)) return hres; - hres = navigate_new_window(element->node.doc->outer_window, uri, target, NULL, NULL); + hres = navigate_new_window(element->node.doc->window->base.outer_window, uri, target, NULL, NULL); IUri_Release(uri); return hres; } @@ -110,7 +113,10 @@ static HRESULT navigate_href(HTMLElement *element, nsAString *href_str, nsAStrin const PRUnichar *href; HRESULT hres; - window = get_target_window(element->node.doc->outer_window, target_str, &use_new_window); + if(!element->node.doc->window->base.outer_window) + return S_OK; + + window = get_target_window(element->node.doc->window->base.outer_window, target_str, &use_new_window); if(!window) { if(use_new_window) { const PRUnichar *target;

1 0

Gabriel Ivăncescu : mshtml: Return failure when setting IHTMLDocument2::location for detached documents.
by Alexandre Julliard 06 Dec '23

06 Dec '23

Module: wine Branch: master Commit: dffb5e0538ece9a1c08d15e63fe0f64254a9037a URL: https://gitlab.winehq.org/wine/wine/-/commit/dffb5e0538ece9a1c08d15e63fe0f6… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Tue Dec 5 19:42:19 2023 +0200 mshtml: Return failure when setting IHTMLDocument2::location for detached documents. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/mshtml/htmldoc.c | 6 ++++-- dlls/mshtml/tests/navigation.js | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c index a8edc3d0251..bc3782d54cc 100644 --- a/dlls/mshtml/htmldoc.c +++ b/dlls/mshtml/htmldoc.c @@ -6089,10 +6089,12 @@ static HRESULT HTMLDocumentNode_location_hook(DispatchEx *dispex, WORD flags, DI { HTMLDocumentNode *This = impl_from_DispatchEx(dispex); - if(!(flags & DISPATCH_PROPERTYPUT) || !This->outer_window) + if(!(flags & DISPATCH_PROPERTYPUT) || !This->window) return S_FALSE; + if(!This->window->base.outer_window) + return E_FAIL; - return IDispatchEx_InvokeEx(&This->outer_window->base.IDispatchEx_iface, DISPID_IHTMLWINDOW2_LOCATION, + return IDispatchEx_InvokeEx(&This->window->base.outer_window->base.IDispatchEx_iface, DISPID_IHTMLWINDOW2_LOCATION, 0, flags, dp, res, ei, caller); } diff --git a/dlls/mshtml/tests/navigation.js b/dlls/mshtml/tests/navigation.js index 59ca6ce948b..2b32903abdb 100644 --- a/dlls/mshtml/tests/navigation.js +++ b/dlls/mshtml/tests/navigation.js @@ -124,6 +124,7 @@ function detached_iframe_doc() { expect_exception(function() { origDoc.documentElement; }); expect_exception(function() { origDoc.domain; }); expect_exception(function() { origDoc.frames; }); + expect_exception(function() { origDoc.location = "blank.html"; }); expect_exception(function() { origDoc.readyState; }, true); expect_exception(function() { origDoc.URL; }); expect_exception(function() { origDoc.URL = "blank.html"; });

1 0

← Newer
1
...
23
24
25
26
27
28
29
...
42
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 December 2023