May 2019 - Wine-Commits - list.winehq.org

Alexandre Julliard : include/msvcrt: Add snprintf() compatibility macro.
by Alexandre Julliard May 29, 2019

May 29, 2019

Module: wine Branch: master Commit: 44be6a8294ea6b0a4349cbbbd33463aa511df9f2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=44be6a8294ea6b0a4349cbbb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed May 29 09:10:01 2019 +0200 include/msvcrt: Add snprintf() compatibility macro. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/msvcrt/stdio.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/msvcrt/stdio.h b/include/msvcrt/stdio.h index 6506dc7..014e68a 100644 --- a/include/msvcrt/stdio.h +++ b/include/msvcrt/stdio.h @@ -298,6 +298,7 @@ static inline int unlink(const char* path) { return _unlink(path); } #define _UNLINK_DEFINED #endif static inline int vsnprintf(char *buffer, size_t size, const char *format, __ms_va_list args) { return _vsnprintf(buffer,size,format,args); } +#define snprintf _snprintf static inline wint_t fgetwchar(void) { return _fgetwchar(); } static inline wint_t fputwchar(wint_t wc) { return _fputwchar(wc); }

1 0

Jacek Caban : mshtml/tests: Include leading path slash in location test array.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: adeb76bd15a4acdd741fc7ba66239e0ba230257b URL: https://source.winehq.org/git/wine.git/?a=commit;h=adeb76bd15a4acdd741fc7ba… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 17:33:44 2019 +0200 mshtml/tests: Include leading path slash in location test array. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/htmllocation.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/mshtml/tests/htmllocation.c b/dlls/mshtml/tests/htmllocation.c index 4781de9..3d81eb6 100644 --- a/dlls/mshtml/tests/htmllocation.c +++ b/dlls/mshtml/tests/htmllocation.c @@ -47,7 +47,7 @@ static const struct location_test location_tests[] = { "www.winehq.org:80", "www.winehq.org", "80", - "", + "/", "?search", "#hash" }, @@ -59,7 +59,7 @@ static const struct location_test location_tests[] = { "www.winehq.org:80", "www.winehq.org", "80", - "file", + "/file", "?search", "#hash" }, @@ -71,7 +71,7 @@ static const struct location_test location_tests[] = { "ftp.winehq.org:21", "ftp.winehq.org", "21", - "", + "/", NULL, NULL }, @@ -83,7 +83,7 @@ static const struct location_test location_tests[] = { "ftp.winehq.org:21", "ftp.winehq.org", "21", - "file", + "/file", NULL, NULL }, @@ -240,7 +240,7 @@ static void test_pathname(IHTMLLocation *loc, const struct location_test *test) hres = IHTMLLocation_get_pathname(loc, &str); ok(hres == S_OK, "%s: get_pathname failed: 0x%08x\n", test->name, hres); if(hres == S_OK) - ok(str_eq_wa(str, test->pathname), + ok(str_eq_wa(str, *test->pathname == '/' ? test->pathname + 1 : test->pathname), "%s: expected retrieved pathname to be L\"%s\", was: %s\n", test->name, test->pathname, wine_dbgstr_w(str)); SysFreeString(str);

1 0

Jacek Caban : mshtml/tests: Fix tests on some recent IE versions.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 6941232d8be5b0f6e46f4a01b70ff60db9895773 URL: https://source.winehq.org/git/wine.git/?a=commit;h=6941232d8be5b0f6e46f4a01… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 17:33:35 2019 +0200 mshtml/tests: Fix tests on some recent IE versions. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/script.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/tests/script.c b/dlls/mshtml/tests/script.c index c0ddc81..7d6afd2 100644 --- a/dlls/mshtml/tests/script.c +++ b/dlls/mshtml/tests/script.c @@ -135,6 +135,7 @@ DEFINE_EXPECT(QS_VariantConversion); DEFINE_EXPECT(QS_IActiveScriptSite); DEFINE_EXPECT(QS_GetCaller); DEFINE_EXPECT(ChangeType); +DEFINE_EXPECT(GetTypeInfo); #define TESTSCRIPT_CLSID "{178fc163-f585-4e24-9c13-4bb7faf80746}" #define TESTACTIVEX_CLSID "{178fc163-f585-4e24-9c13-4bb7faf80646}" @@ -399,7 +400,7 @@ static HRESULT WINAPI DispatchEx_GetTypeInfoCount(IDispatchEx *iface, UINT *pcti static HRESULT WINAPI DispatchEx_GetTypeInfo(IDispatchEx *iface, UINT iTInfo, LCID lcid, ITypeInfo **ppTInfo) { - ok(0, "unexpected call\n"); + CHECK_EXPECT2(GetTypeInfo); return E_NOTIMPL; } @@ -3453,9 +3454,12 @@ static void run_js_tests(void) { run_js_script("jstest.html"); run_js_script("exectest.html"); - run_js_script("vbtest.html"); run_js_script("events.html"); + SET_EXPECT(GetTypeInfo); + run_js_script("vbtest.html"); + CLEAR_CALLED(GetTypeInfo); + if(!is_ie9plus) { win_skip("Skipping some script tests on IE older than 9.\n"); return;

1 0

Jacek Caban : server: Remove IRP from device queue before terminating async.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: dc5421f9506da691dbc631fb319e8d10a0e22ce9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=dc5421f9506da691dbc631fb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 14:09:59 2019 +0200 server: Remove IRP from device queue before terminating async. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- server/device.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/server/device.c b/server/device.c index e123583..1cebc3b 100644 --- a/server/device.c +++ b/server/device.c @@ -378,6 +378,9 @@ static void set_irp_result( struct irp_call *irp, unsigned int status, iosb->out_size = min( iosb->out_size, out_size ); if (iosb->out_size && !(iosb->out_data = memdup( out_data, iosb->out_size ))) iosb->out_size = 0; + + /* remove it from the device queue */ + list_remove( &irp->dev_entry ); irp->file = NULL; if (irp->async) { @@ -388,8 +391,6 @@ static void set_irp_result( struct irp_call *irp, unsigned int status, } wake_up( &irp->obj, 0 ); - /* remove it from the device queue */ - list_remove( &irp->dev_entry ); release_object( irp ); /* no longer on the device queue */ release_object( file ); }

1 0

Aric Stewart : winemac.drv: Use NI_COMPOSITIONSTR/ CPS_CANCEL when composition string is empty.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 6d05fae6f7ebab83a2f1c6621d9619bbe91833fd URL: https://source.winehq.org/git/wine.git/?a=commit;h=6d05fae6f7ebab83a2f1c662… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri May 24 09:07:55 2019 -0500 winemac.drv: Use NI_COMPOSITIONSTR/CPS_CANCEL when composition string is empty. This sends the expected WM_IME_ENDCOMPOSITION message. Signed-off-by: Aric Stewart <aric(a)codeweavers.com> Signed-off-by: Ken Thomases <ken(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/ime.c | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/dlls/winemac.drv/ime.c b/dlls/winemac.drv/ime.c index 1f5cb6b..a071408 100644 --- a/dlls/winemac.drv/ime.c +++ b/dlls/winemac.drv/ime.c @@ -901,6 +901,7 @@ static BOOL IME_SetCompositionString(void* hIMC, DWORD dwIndex, LPCVOID lpComp, DWORD flags = 0; WCHAR wParam = 0; LPIMEPRIVATE myPrivate; + BOOL sendMessage = TRUE; TRACE("(%p, %d, %p, %d):\n", hIMC, dwIndex, lpComp, dwCompLen); @@ -943,28 +944,29 @@ static BOOL IME_SetCompositionString(void* hIMC, DWORD dwIndex, LPCVOID lpComp, wParam = ((const WCHAR*)lpComp)[0]; flags |= GCS_COMPCLAUSE | GCS_COMPATTR | GCS_DELTASTART; + + if (cursor_valid) + { + LPCOMPOSITIONSTRING compstr; + compstr = ImmLockIMCC(lpIMC->hCompStr); + compstr->dwCursorPos = cursor_pos; + ImmUnlockIMCC(lpIMC->hCompStr); + flags |= GCS_CURSORPOS; + } } else { - newCompStr = updateCompStr(lpIMC->hCompStr, NULL, 0, &flags); - ImmDestroyIMCC(lpIMC->hCompStr); - lpIMC->hCompStr = newCompStr; - } - - if (cursor_valid) - { - LPCOMPOSITIONSTRING compstr; - compstr = ImmLockIMCC(lpIMC->hCompStr); - compstr->dwCursorPos = cursor_pos; - ImmUnlockIMCC(lpIMC->hCompStr); - flags |= GCS_CURSORPOS; + NotifyIME(hIMC, NI_COMPOSITIONSTR, CPS_CANCEL, 0); + sendMessage = FALSE; } } - GenerateIMEMessage(hIMC, WM_IME_COMPOSITION, wParam, flags); - ImmUnlockIMCC(lpIMC->hPrivate); - UnlockRealIMC(hIMC); + if (sendMessage) { + GenerateIMEMessage(hIMC, WM_IME_COMPOSITION, wParam, flags); + ImmUnlockIMCC(lpIMC->hPrivate); + UnlockRealIMC(hIMC); + } return TRUE; }

1 0

Jacek Caban : mshtml: Rewrite IHTMLLocation::get_pathname on top of IUri.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: e33ce7c53854c1fe179218265642e9ede4152d39 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e33ce7c53854c1fe17921826… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 17:33:58 2019 +0200 mshtml: Rewrite IHTMLLocation::get_pathname on top of IUri. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=37279 Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmllocation.c | 23 +++++++++++++---------- dlls/mshtml/tests/elements.js | 17 +++++++++++++++++ dlls/mshtml/tests/htmllocation.c | 3 +++ dlls/mshtml/tests/script.c | 6 ++++++ 4 files changed, 39 insertions(+), 10 deletions(-) diff --git a/dlls/mshtml/htmllocation.c b/dlls/mshtml/htmllocation.c index 4b8a07a..06cf7e1 100644 --- a/dlls/mshtml/htmllocation.c +++ b/dlls/mshtml/htmllocation.c @@ -459,7 +459,8 @@ static HRESULT WINAPI HTMLLocation_put_pathname(IHTMLLocation *iface, BSTR v) static HRESULT WINAPI HTMLLocation_get_pathname(IHTMLLocation *iface, BSTR *p) { HTMLLocation *This = impl_from_IHTMLLocation(iface); - URL_COMPONENTSW url = {sizeof(URL_COMPONENTSW)}; + BSTR path; + IUri *uri; HRESULT hres; TRACE("(%p)->(%p)\n", This, p); @@ -467,19 +468,21 @@ static HRESULT WINAPI HTMLLocation_get_pathname(IHTMLLocation *iface, BSTR *p) if(!p) return E_POINTER; - url.dwUrlPathLength = 1; - url.dwExtraInfoLength = 1; - hres = get_url_components(This, &url); + if(!(uri = get_uri(This))) { + FIXME("No current URI\n"); + return E_NOTIMPL; + } + + hres = IUri_GetPath(uri, &path); if(FAILED(hres)) return hres; - if(url.dwUrlPathLength && url.lpszUrlPath[0] == '/') - *p = SysAllocStringLen(url.lpszUrlPath + 1, url.dwUrlPathLength - 1); - else - *p = SysAllocStringLen(url.lpszUrlPath, url.dwUrlPathLength); + if(hres == S_FALSE) { + SysFreeString(path); + path = NULL; + } - if(!*p) - return E_OUTOFMEMORY; + *p = path; return S_OK; } diff --git a/dlls/mshtml/tests/elements.js b/dlls/mshtml/tests/elements.js index be37d90..60e4a7e 100644 --- a/dlls/mshtml/tests/elements.js +++ b/dlls/mshtml/tests/elements.js @@ -126,6 +126,22 @@ function test_iframe() { }); } +function test_iframe_location() { + document.body.innerHTML = '<iframe src="emptyfile"></iframe>' + var iframe = document.body.firstChild; + + iframe.onload = function() { + ok(iframe.contentWindow.location.pathname === "/emptyfile", + "path = " + iframe.contentWindow.location.pathname); + iframe.onload = function () { + ok(iframe.contentWindow.location.pathname === "/empty/file", + "path = " + iframe.contentWindow.location.pathname); + next_test(); + } + iframe.src = "empty/file"; + } +} + function test_anchor() { var iframe = document.body.firstChild; var anchor = document.createElement("a"); @@ -420,6 +436,7 @@ var tests = [ test_createElementNS, test_head, test_iframe, + test_iframe_location, test_anchor, test_query_selector, test_compare_position, diff --git a/dlls/mshtml/tests/htmllocation.c b/dlls/mshtml/tests/htmllocation.c index 3d81eb6..120ab7e 100644 --- a/dlls/mshtml/tests/htmllocation.c +++ b/dlls/mshtml/tests/htmllocation.c @@ -240,6 +240,9 @@ static void test_pathname(IHTMLLocation *loc, const struct location_test *test) hres = IHTMLLocation_get_pathname(loc, &str); ok(hres == S_OK, "%s: get_pathname failed: 0x%08x\n", test->name, hres); if(hres == S_OK) + /* FIXME: We seem to be testing location in a buggy state. Path names do not + * include leading slash, while other tests confirm that it should be there */ + todo_wine_if(*test->pathname == '/') ok(str_eq_wa(str, *test->pathname == '/' ? test->pathname + 1 : test->pathname), "%s: expected retrieved pathname to be L\"%s\", was: %s\n", test->name, test->pathname, wine_dbgstr_w(str)); diff --git a/dlls/mshtml/tests/script.c b/dlls/mshtml/tests/script.c index 7d6afd2..e546649 100644 --- a/dlls/mshtml/tests/script.c +++ b/dlls/mshtml/tests/script.c @@ -3087,6 +3087,8 @@ static HRESULT WINAPI ProtocolEx_StartEx(IInternetProtocolEx *iface, IUri *uri, HRSRC src; HRESULT hres; + static const WCHAR empty_prefixW[] = {'/','e','m','p','t','y'}; + hres = IInternetBindInfo_GetBindInfo(pOIBindInfo, &bindf, &This->bind_info); ok(hres == S_OK, "GetBindInfo failed: %08x\n", hres); @@ -3121,6 +3123,10 @@ static HRESULT WINAPI ProtocolEx_StartEx(IInternetProtocolEx *iface, IUri *uri, register_stream(This, query+1); SysFreeString(query); block = TRUE; + }else if(SysStringLen(path) >= ARRAY_SIZE(empty_prefixW) && !memcmp(path, empty_prefixW, sizeof(empty_prefixW))) { + static char empty_data[] = " "; + This->data = empty_data; + This->size = strlen(This->data); }else { src = FindResourceW(NULL, *path == '/' ? path+1 : path, (const WCHAR*)RT_HTML); ok(src != NULL, "Could not find resource for path %s\n", wine_dbgstr_w(path));

1 0

Nikolay Sivov : mf: Implement presentation clock methods for sample grabber.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 2407a0f53b0e4805406fae0bd578c06fef4d9f02 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2407a0f53b0e4805406fae0b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue May 28 15:08:28 2019 +0300 mf: Implement presentation clock methods for sample grabber. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mf/samplegrabber.c | 51 +++++++++++++++++++++++++++++++++++++++++++++---- dlls/mf/tests/mf.c | 18 +++++++++++------ 2 files changed, 59 insertions(+), 10 deletions(-) diff --git a/dlls/mf/samplegrabber.c b/dlls/mf/samplegrabber.c index 86dcb13..755d80f 100644 --- a/dlls/mf/samplegrabber.c +++ b/dlls/mf/samplegrabber.c @@ -50,6 +50,7 @@ struct sample_grabber BOOL is_shut_down; IMFStreamSink *stream; IMFMediaEventQueue *event_queue; + IMFPresentationClock *clock; CRITICAL_SECTION cs; }; @@ -430,6 +431,8 @@ static ULONG WINAPI sample_grabber_sink_Release(IMFMediaSink *iface) IMFMediaEventQueue_Shutdown(grabber->event_queue); IMFMediaEventQueue_Release(grabber->event_queue); } + if (grabber->clock) + IMFPresentationClock_Release(grabber->clock); DeleteCriticalSection(&grabber->cs); heap_free(grabber); } @@ -539,16 +542,56 @@ static HRESULT WINAPI sample_grabber_sink_GetStreamSinkById(IMFMediaSink *iface, static HRESULT WINAPI sample_grabber_sink_SetPresentationClock(IMFMediaSink *iface, IMFPresentationClock *clock) { - FIXME("%p, %p.\n", iface, clock); + struct sample_grabber *grabber = impl_from_IMFMediaSink(iface); + HRESULT hr; - return E_NOTIMPL; + TRACE("%p, %p.\n", iface, clock); + + EnterCriticalSection(&grabber->cs); + + if (SUCCEEDED(hr = IMFSampleGrabberSinkCallback_OnSetPresentationClock(grabber->callback, clock))) + { + if (grabber->clock) + { + IMFPresentationClock_RemoveClockStateSink(grabber->clock, &grabber->IMFClockStateSink_iface); + IMFPresentationClock_Release(grabber->clock); + } + grabber->clock = clock; + if (grabber->clock) + { + IMFPresentationClock_AddRef(grabber->clock); + IMFPresentationClock_AddClockStateSink(grabber->clock, &grabber->IMFClockStateSink_iface); + } + } + + LeaveCriticalSection(&grabber->cs); + + return hr; } static HRESULT WINAPI sample_grabber_sink_GetPresentationClock(IMFMediaSink *iface, IMFPresentationClock **clock) { - FIXME("%p, %p.\n", iface, clock); + struct sample_grabber *grabber = impl_from_IMFMediaSink(iface); + HRESULT hr = S_OK; - return E_NOTIMPL; + TRACE("%p, %p.\n", iface, clock); + + if (!clock) + return E_POINTER; + + EnterCriticalSection(&grabber->cs); + + if (grabber->clock) + { + *clock = grabber->clock; + IMFPresentationClock_AddRef(*clock); + } + else + hr = MF_E_NO_CLOCK; + + LeaveCriticalSection(&grabber->cs); + + return hr; } static HRESULT WINAPI sample_grabber_sink_Shutdown(IMFMediaSink *iface) diff --git a/dlls/mf/tests/mf.c b/dlls/mf/tests/mf.c index b381daf..2285eb8 100644 --- a/dlls/mf/tests/mf.c +++ b/dlls/mf/tests/mf.c @@ -1605,7 +1605,7 @@ static HRESULT WINAPI grabber_callback_OnClockSetRate(IMFSampleGrabberSinkCallba static HRESULT WINAPI grabber_callback_OnSetPresentationClock(IMFSampleGrabberSinkCallback *iface, IMFPresentationClock *clock) { - return E_NOTIMPL; + return S_OK; } static HRESULT WINAPI grabber_callback_OnProcessSample(IMFSampleGrabberSinkCallback *iface, REFGUID major_type, @@ -1641,9 +1641,9 @@ static void test_sample_grabber(void) IMFMediaType *media_type, *media_type2, *media_type3; IMFMediaTypeHandler *handler, *handler2; IMFPresentationTimeSource *time_source; + IMFPresentationClock *clock, *clock2; IMFStreamSink *stream, *stream2; IMFClockStateSink *clocksink; - IMFPresentationClock *clock; IMFMediaEventGenerator *eg; IMFMediaSink *sink, *sink2; DWORD flags, count, id; @@ -1742,8 +1742,17 @@ static void test_sample_grabber(void) hr = MFCreatePresentationClock(&clock); ok(hr == S_OK, "Failed to create clock object, hr %#x.\n", hr); + hr = IMFMediaSink_GetPresentationClock(sink, NULL); + ok(hr == E_POINTER, "Unexpected hr %#x.\n", hr); + + hr = IMFMediaSink_GetPresentationClock(sink, &clock2); + ok(hr == MF_E_NO_CLOCK, "Unexpected hr %#x.\n", hr); + + hr = IMFMediaSink_SetPresentationClock(sink, NULL); + ok(hr == S_OK, "Failed to set presentation clock, hr %#x.\n", hr); + hr = IMFMediaSink_SetPresentationClock(sink, clock); - ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr); + ok(hr == S_OK, "Failed to set presentation clock, hr %#x.\n", hr); hr = MFCreateSystemTimeSource(&time_source); ok(hr == S_OK, "Failed to create time source, hr %#x.\n", hr); @@ -1752,9 +1761,6 @@ static void test_sample_grabber(void) ok(hr == S_OK, "Failed to set time source, hr %#x.\n", hr); IMFPresentationTimeSource_Release(time_source); - hr = IMFMediaSink_SetPresentationClock(sink, clock); - ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr); - IMFPresentationClock_Release(clock); hr = IMFActivate_ShutdownObject(activate);

1 0

Jacek Caban : ntoskrnl.exe: Cancel IRPs terminated by server.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 9157129fc86c8759e75fc6f68b49b28693c6c9ca URL: https://source.winehq.org/git/wine.git/?a=commit;h=9157129fc86c8759e75fc6f6… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 14:10:48 2019 +0200 ntoskrnl.exe: Cancel IRPs terminated by server. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/ntoskrnl.c | 38 +++++++++++++++++++++++++++++++++++--- dlls/ntoskrnl.exe/tests/ntoskrnl.c | 3 --- 2 files changed, 35 insertions(+), 6 deletions(-) diff --git a/dlls/ntoskrnl.exe/ntoskrnl.c b/dlls/ntoskrnl.exe/ntoskrnl.c index 04a3f31..b5d89f6 100644 --- a/dlls/ntoskrnl.exe/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/ntoskrnl.c @@ -523,15 +523,31 @@ static void *create_file_object( HANDLE handle ) return file; } +DECLARE_CRITICAL_SECTION(irp_completion_cs); + +static void WINAPI cancel_completed_irp( DEVICE_OBJECT *device, IRP *irp ) +{ + TRACE( "(%p %p)\n", device, irp ); + + IoReleaseCancelSpinLock(irp->CancelIrql); + + irp->IoStatus.u.Status = STATUS_CANCELLED; + irp->IoStatus.Information = 0; + IoCompleteRequest(irp, IO_NO_INCREMENT); +} + /* transfer result of IRP back to wineserver */ static NTSTATUS WINAPI dispatch_irp_completion( DEVICE_OBJECT *device, IRP *irp, void *context ) { HANDLE irp_handle = context; void *out_buff = irp->UserBuffer; + NTSTATUS status; if (irp->Flags & IRP_WRITE_OPERATION) out_buff = NULL; /* do not transfer back input buffer */ + EnterCriticalSection( &irp_completion_cs ); + SERVER_START_REQ( set_irp_result ) { req->handle = wine_server_obj_handle( irp_handle ); @@ -541,16 +557,29 @@ static NTSTATUS WINAPI dispatch_irp_completion( DEVICE_OBJECT *device, IRP *irp, req->size = irp->IoStatus.Information; if (out_buff) wine_server_add_data( req, out_buff, irp->IoStatus.Information ); } - wine_server_call( req ); + status = wine_server_call( req ); } SERVER_END_REQ; + if (status == STATUS_MORE_PROCESSING_REQUIRED) + { + /* IRP is complete, but server may have already ordered cancel call. In such case, + * it will return STATUS_MORE_PROCESSING_REQUIRED, leaving the IRP alive until + * cancel frees it. */ + if (irp->Cancel) + status = STATUS_SUCCESS; + else + IoSetCancelRoutine( irp, cancel_completed_irp ); + } + if (irp->UserBuffer != irp->AssociatedIrp.SystemBuffer) { HeapFree( GetProcessHeap(), 0, irp->UserBuffer ); irp->UserBuffer = NULL; } - return STATUS_SUCCESS; + + LeaveCriticalSection( &irp_completion_cs ); + return status; } struct dispatch_context @@ -842,8 +871,11 @@ static NTSTATUS dispatch_cancel( struct dispatch_context *context ) { IRP *irp = wine_server_get_ptr( context->params.cancel.irp ); - FIXME( "%p\n", irp ); + TRACE( "%p\n", irp ); + EnterCriticalSection( &irp_completion_cs ); + IoCancelIrp( irp ); + LeaveCriticalSection( &irp_completion_cs ); return STATUS_SUCCESS; } diff --git a/dlls/ntoskrnl.exe/tests/ntoskrnl.c b/dlls/ntoskrnl.exe/tests/ntoskrnl.c index d505643..43418f3 100644 --- a/dlls/ntoskrnl.exe/tests/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/tests/ntoskrnl.c @@ -234,7 +234,6 @@ static void test_overlapped(void) cancel_cnt = 0xdeadbeef; res = DeviceIoControl(file, IOCTL_WINETEST_GET_CANCEL_COUNT, NULL, 0, &cancel_cnt, sizeof(cancel_cnt), NULL, &overlapped); ok(res, "DeviceIoControl failed: %u\n", GetLastError()); - todo_wine ok(cancel_cnt == 2, "cancel_cnt = %u\n", cancel_cnt); /* test cancelling selected overlapped event */ @@ -254,7 +253,6 @@ static void test_overlapped(void) cancel_cnt = 0xdeadbeef; res = DeviceIoControl(file, IOCTL_WINETEST_GET_CANCEL_COUNT, NULL, 0, &cancel_cnt, sizeof(cancel_cnt), NULL, &overlapped); ok(res, "DeviceIoControl failed: %u\n", GetLastError()); - todo_wine ok(cancel_cnt == 1, "cancel_cnt = %u\n", cancel_cnt); pCancelIoEx(file, &overlapped2); @@ -262,7 +260,6 @@ static void test_overlapped(void) cancel_cnt = 0xdeadbeef; res = DeviceIoControl(file, IOCTL_WINETEST_GET_CANCEL_COUNT, NULL, 0, &cancel_cnt, sizeof(cancel_cnt), NULL, &overlapped); ok(res, "DeviceIoControl failed: %u\n", GetLastError()); - todo_wine ok(cancel_cnt == 2, "cancel_cnt = %u\n", cancel_cnt); }

1 0

Jacek Caban : server: Store IRP client pointer on server side.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 101da2b5127ae698e79bca4ac0bfa780c785722b URL: https://source.winehq.org/git/wine.git/?a=commit;h=101da2b5127ae698e79bca4a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 28 14:09:47 2019 +0200 server: Store IRP client pointer on server side. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/ntoskrnl.c | 10 +++++++--- include/wine/server_protocol.h | 3 ++- server/device.c | 6 ++++++ server/protocol.def | 1 + server/request.h | 3 ++- server/trace.c | 1 + 6 files changed, 19 insertions(+), 5 deletions(-) diff --git a/dlls/ntoskrnl.exe/ntoskrnl.c b/dlls/ntoskrnl.exe/ntoskrnl.c index 10cf2e2..1892809 100644 --- a/dlls/ntoskrnl.exe/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/ntoskrnl.c @@ -557,6 +557,7 @@ struct dispatch_context { irp_params_t params; HANDLE handle; + IRP *irp; ULONG in_size; void *in_buff; }; @@ -570,6 +571,7 @@ static void dispatch_irp( DEVICE_OBJECT *device, IRP *irp, struct dispatch_conte KeQueryTickCount( &count ); /* update the global KeTickCount */ + context->irp = irp; device->CurrentIrp = irp; IoCallDriver( device, irp ); device->CurrentIrp = NULL; @@ -931,9 +933,10 @@ NTSTATUS CDECL wine_ntoskrnl_main_loop( HANDLE stop_event ) SERVER_START_REQ( get_next_device_request ) { - req->manager = wine_server_obj_handle( manager ); - req->prev = wine_server_obj_handle( context.handle ); - req->status = status; + req->manager = wine_server_obj_handle( manager ); + req->prev = wine_server_obj_handle( context.handle ); + req->user_ptr = wine_server_client_ptr( context.irp ); + req->status = status; wine_server_set_reply( req, context.in_buff, context.in_size ); if (!(status = wine_server_call( req ))) { @@ -949,6 +952,7 @@ NTSTATUS CDECL wine_ntoskrnl_main_loop( HANDLE stop_event ) if (status == STATUS_BUFFER_OVERFLOW) context.in_size = reply->in_size; } + context.irp = NULL; } SERVER_END_REQ; diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index a45d092..eab5755 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -5252,6 +5252,7 @@ struct get_next_device_request_request obj_handle_t manager; obj_handle_t prev; unsigned int status; + client_ptr_t user_ptr; }; struct get_next_device_request_reply { @@ -6694,6 +6695,6 @@ union generic_reply struct resume_process_reply resume_process_reply; }; -#define SERVER_PROTOCOL_VERSION 583 +#define SERVER_PROTOCOL_VERSION 584 #endif /* __WINE_WINE_SERVER_PROTOCOL_H */ diff --git a/server/device.c b/server/device.c index d6ab1c5..e123583 100644 --- a/server/device.c +++ b/server/device.c @@ -52,6 +52,7 @@ struct irp_call struct async *async; /* pending async op */ irp_params_t params; /* irp parameters */ struct iosb *iosb; /* I/O status block */ + client_ptr_t user_ptr; /* client side pointer */ }; static void irp_call_dump( struct object *obj, int verbose ); @@ -351,6 +352,7 @@ static struct irp_call *create_irp( struct device_file *file, const irp_params_t irp->async = NULL; irp->params = *params; irp->iosb = NULL; + irp->user_ptr = 0; if (async) irp->iosb = async_get_iosb( async ); if (!irp->iosb && !(irp->iosb = create_iosb( NULL, 0, 0 ))) @@ -886,13 +888,17 @@ DECL_HANDLER(get_next_device_request) if (req->prev) close_handle( current->process, req->prev ); /* avoid an extra round-trip for close */ + /* process result of previous call */ if (manager->current_call) { irp = manager->current_call; + irp->user_ptr = req->user_ptr; + if (req->status) set_irp_result( irp, req->status, NULL, 0, 0 ); else if (irp->async) set_async_pending( irp->async, irp->file && is_fd_overlapped( irp->file->fd ) ); + free_irp_params( irp ); release_object( irp ); manager->current_call = NULL; diff --git a/server/protocol.def b/server/protocol.def index 21008d7..6b6e868 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -3633,6 +3633,7 @@ struct handle_info obj_handle_t manager; /* handle to the device manager */ obj_handle_t prev; /* handle to the previous irp */ unsigned int status; /* status of the previous irp */ + client_ptr_t user_ptr; /* user pointer of the previous irp */ @REPLY irp_params_t params; /* irp parameters */ obj_handle_t next; /* handle to the next irp */ diff --git a/server/request.h b/server/request.h index 07918df..1858b41 100644 --- a/server/request.h +++ b/server/request.h @@ -2287,7 +2287,8 @@ C_ASSERT( sizeof(struct delete_device_request) == 24 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_request, manager) == 12 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_request, prev) == 16 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_request, status) == 20 ); -C_ASSERT( sizeof(struct get_next_device_request_request) == 24 ); +C_ASSERT( FIELD_OFFSET(struct get_next_device_request_request, user_ptr) == 24 ); +C_ASSERT( sizeof(struct get_next_device_request_request) == 32 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_reply, params) == 8 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_reply, next) == 40 ); C_ASSERT( FIELD_OFFSET(struct get_next_device_request_reply, client_tid) == 44 ); diff --git a/server/trace.c b/server/trace.c index 28bf3dd..82ea93f 100644 --- a/server/trace.c +++ b/server/trace.c @@ -4282,6 +4282,7 @@ static void dump_get_next_device_request_request( const struct get_next_device_r fprintf( stderr, " manager=%04x", req->manager ); fprintf( stderr, ", prev=%04x", req->prev ); fprintf( stderr, ", status=%08x", req->status ); + dump_uint64( ", user_ptr=", &req->user_ptr ); } static void dump_get_next_device_request_reply( const struct get_next_device_request_reply *req )

1 0

Andrey Gusev : ext-ms-win-ntuser-windowclass-l1-1-0: Update spec file.
by Alexandre Julliard May 28, 2019

May 28, 2019

Module: wine Branch: master Commit: 82a0181a59a97bb8f8de7c0d6023607c0100c314 URL: https://source.winehq.org/git/wine.git/?a=commit;h=82a0181a59a97bb8f8de7c0d… Author: Andrey Gusev <andrey.goosev(a)gmail.com> Date: Tue May 28 18:22:22 2019 +0300 ext-ms-win-ntuser-windowclass-l1-1-0: Update spec file. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47280 Signed-off-by: Andrey Gusev <andrey.goosev(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../ext-ms-win-ntuser-windowclass-l1-1-0.spec | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/ext-ms-win-ntuser-windowclass-l1-1-0/ext-ms-win-ntuser-windowclass-l1-1-0.spec b/dlls/ext-ms-win-ntuser-windowclass-l1-1-0/ext-ms-win-ntuser-windowclass-l1-1-0.spec index 8b2ab2b..ad8dafe 100644 --- a/dlls/ext-ms-win-ntuser-windowclass-l1-1-0/ext-ms-win-ntuser-windowclass-l1-1-0.spec +++ b/dlls/ext-ms-win-ntuser-windowclass-l1-1-0/ext-ms-win-ntuser-windowclass-l1-1-0.spec @@ -5,11 +5,13 @@ @ stdcall GetClassNameA(long ptr long) user32.GetClassNameA @ stdcall GetClassNameW(long ptr long) user32.GetClassNameW @ stdcall GetWindowLongA(long long) user32.GetWindowLongA +@ stdcall -arch=win64 GetWindowLongPtrW(long long) user32.GetWindowLongPtrW @ stdcall GetWindowLongW(long long) user32.GetWindowLongW @ stdcall RegisterClassA(ptr) user32.RegisterClassA @ stdcall RegisterClassExW(ptr) user32.RegisterClassExW @ stdcall RegisterClassW(ptr) user32.RegisterClassW @ stdcall SetWindowLongA(long long long) user32.SetWindowLongA +@ stdcall -arch=win64 SetWindowLongPtrW(long long long) user32.SetWindowLongPtrW @ stdcall SetWindowLongW(long long long) user32.SetWindowLongW @ stdcall UnregisterClassA(str long) user32.UnregisterClassA @ stdcall UnregisterClassW(wstr long) user32.UnregisterClassW

1 0