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

Akihiro Sagawa : winmm/tests: Fix uninitialized memory access (Valgrind).
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 857b4dcda2e689f21e4703310de86b59b65e4d47 URL: http://source.winehq.org/git/wine.git/?a=commit;h=857b4dcda2e689f21e4703310… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Wed May 6 21:33:15 2015 +0900 winmm/tests: Fix uninitialized memory access (Valgrind). --- dlls/winmm/tests/wave.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/winmm/tests/wave.c b/dlls/winmm/tests/wave.c index e0232e7..a2fe8bb 100644 --- a/dlls/winmm/tests/wave.c +++ b/dlls/winmm/tests/wave.c @@ -1621,10 +1621,12 @@ static void create_wav_file(char *temp_file) ok(h != NULL, "Can't open temp_file\n"); riff_chunk.fccType = mmioFOURCC('W','A','V','E'); + riff_chunk.cksize = 0; rc = mmioCreateChunk(h, &riff_chunk, MMIO_CREATERIFF); ok(rc == MMSYSERR_NOERROR, "mmioCreateChunk failed, got %u\n", rc); chunk.ckid = mmioFOURCC('f','m','t',' '); + chunk.cksize = 0; rc = mmioCreateChunk(h, &chunk, 0); ok(rc == MMSYSERR_NOERROR, "mmioCreateChunk failed, got %u\n", rc); written = mmioWrite(h, (char*)&format, sizeof(format));

1 0

Nikolay Sivov : oleaut32: Cleanup connection point methods a bit.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: f07c9784db148e158dc118163ad5f5bde6f9ad4e URL: http://source.winehq.org/git/wine.git/?a=commit;h=f07c9784db148e158dc118163… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu May 7 10:56:46 2015 +0300 oleaut32: Cleanup connection point methods a bit. --- dlls/oleaut32/connpt.c | 47 ++++++++++++++--------------------------------- 1 file changed, 14 insertions(+), 33 deletions(-) diff --git a/dlls/oleaut32/connpt.c b/dlls/oleaut32/connpt.c index 5e15e08..0405d38 100644 --- a/dlls/oleaut32/connpt.c +++ b/dlls/oleaut32/connpt.c @@ -118,7 +118,6 @@ static void ConnectionPointImpl_Destroy(ConnectionPointImpl *Obj) return; } -static ULONG WINAPI ConnectionPointImpl_AddRef(IConnectionPoint* iface); /************************************************************************ * ConnectionPointImpl_QueryInterface (IUnknown) * @@ -135,7 +134,7 @@ static HRESULT WINAPI ConnectionPointImpl_QueryInterface( /* * Perform a sanity check on the parameters. */ - if ( (This==0) || (ppvObject==0) ) + if (!ppvObject) return E_INVALIDARG; /* @@ -143,28 +142,20 @@ static HRESULT WINAPI ConnectionPointImpl_QueryInterface( */ *ppvObject = 0; - /* - * Compare the riid with the interface IDs implemented by this object. - */ - if (IsEqualIID(&IID_IUnknown, riid)) - *ppvObject = This; - else if (IsEqualIID(&IID_IConnectionPoint, riid)) - *ppvObject = This; + + if (IsEqualIID(&IID_IConnectionPoint, riid) || IsEqualIID(&IID_IUnknown, riid)) + *ppvObject = iface; /* * Check that we obtained an interface. */ if ((*ppvObject)==0) { - FIXME("() : asking for un supported interface %s\n",debugstr_guid(riid)); + FIXME("() : asking for unsupported interface %s\n", debugstr_guid(riid)); return E_NOINTERFACE; } - /* - * Query Interface always increases the reference count by one when it is - * successful - */ - ConnectionPointImpl_AddRef(&This->IConnectionPoint_iface); + IUnknown_AddRef((IUnknown*)*ppvObject); return S_OK; } @@ -320,9 +311,9 @@ static HRESULT WINAPI ConnectionPointImpl_EnumConnections( /* Bump the ref count of this object up by one. It gets Released in IEnumConnections_Release */ - IUnknown_AddRef((IUnknown*)This); + IConnectionPoint_AddRef(iface); - EnumObj = EnumConnectionsImpl_Construct((IUnknown*)This, This->nSinks, pCD); + EnumObj = EnumConnectionsImpl_Construct((IUnknown*)iface, This->nSinks, pCD); hr = IEnumConnections_QueryInterface(&EnumObj->IEnumConnections_iface, &IID_IEnumConnections, (void**)ppEnum); IEnumConnections_Release(&EnumObj->IEnumConnections_iface); @@ -345,7 +336,6 @@ static const IConnectionPointVtbl ConnectionPointImpl_VTable = static const IEnumConnectionsVtbl EnumConnectionsImpl_VTable; -static ULONG WINAPI EnumConnectionsImpl_AddRef(IEnumConnections* iface); /************************************************************************ * EnumConnectionsImpl_Construct @@ -402,7 +392,7 @@ static HRESULT WINAPI EnumConnectionsImpl_QueryInterface( /* * Perform a sanity check on the parameters. */ - if ( (This==0) || (ppvObject==0) ) + if (!ppvObject) return E_INVALIDARG; /* @@ -410,28 +400,19 @@ static HRESULT WINAPI EnumConnectionsImpl_QueryInterface( */ *ppvObject = 0; - /* - * Compare the riid with the interface IDs implemented by this object. - */ - if (IsEqualIID(&IID_IUnknown, riid)) - *ppvObject = This; - else if (IsEqualIID(&IID_IEnumConnections, riid)) - *ppvObject = This; + if (IsEqualIID(&IID_IEnumConnections, riid) || IsEqualIID(&IID_IUnknown, riid)) + *ppvObject = iface; /* * Check that we obtained an interface. */ if ((*ppvObject)==0) { - FIXME("() : asking for un supported interface %s\n",debugstr_guid(riid)); + FIXME("() : asking for unsupported interface %s\n", debugstr_guid(riid)); return E_NOINTERFACE; } - /* - * Query Interface always increases the reference count by one when it is - * successful - */ - EnumConnectionsImpl_AddRef((IEnumConnections*)This); + IUnknown_AddRef((IUnknown*)*ppvObject); return S_OK; } @@ -558,7 +539,7 @@ static HRESULT WINAPI EnumConnectionsImpl_Clone(IEnumConnections* iface, newObj = EnumConnectionsImpl_Construct(This->pUnk, This->nConns, This->pCD); newObj->nCur = This->nCur; - *ppEnum = (LPENUMCONNECTIONS)newObj; + *ppEnum = &newObj->IEnumConnections_iface; IUnknown_AddRef(This->pUnk); return S_OK; }

1 0

Nikolay Sivov : oleaut32: Simplify connection point creation.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 72eaeb14a6021aacf88d7189bdb2a4177e8c5ba9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=72eaeb14a6021aacf88d7189b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu May 7 10:39:11 2015 +0300 oleaut32: Simplify connection point creation. --- dlls/oleaut32/connpt.c | 46 ++++++++++++++++------------------------------ 1 file changed, 16 insertions(+), 30 deletions(-) diff --git a/dlls/oleaut32/connpt.c b/dlls/oleaut32/connpt.c index 1d7b05c..5e15e08 100644 --- a/dlls/oleaut32/connpt.c +++ b/dlls/oleaut32/connpt.c @@ -66,9 +66,6 @@ typedef struct ConnectionPointImpl { DWORD nSinks; } ConnectionPointImpl; -static const IConnectionPointVtbl ConnectionPointImpl_VTable; - - /************************************************************************ * Implementation of IEnumConnections */ @@ -105,26 +102,6 @@ static inline EnumConnectionsImpl *impl_from_IEnumConnections(IEnumConnections * } /************************************************************************ - * ConnectionPointImpl_Construct - */ -static ConnectionPointImpl *ConnectionPointImpl_Construct(IUnknown *pUnk, - REFIID riid) -{ - ConnectionPointImpl *Obj; - - Obj = HeapAlloc(GetProcessHeap(), 0, sizeof(*Obj)); - Obj->IConnectionPoint_iface.lpVtbl = &ConnectionPointImpl_VTable; - Obj->Obj = pUnk; - Obj->ref = 1; - Obj->iid = *riid; - Obj->maxSinks = MAXSINKS; - Obj->sinks = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, - sizeof(IUnknown*) * MAXSINKS); - Obj->nSinks = 0; - return Obj; -} - -/************************************************************************ * ConnectionPointImpl_Destroy */ static void ConnectionPointImpl_Destroy(ConnectionPointImpl *Obj) @@ -615,13 +592,22 @@ HRESULT CreateConnectionPoint(IUnknown *pUnk, REFIID riid, IConnectionPoint **pCP) { ConnectionPointImpl *Obj; - HRESULT hr; - Obj = ConnectionPointImpl_Construct(pUnk, riid); - if(!Obj) return E_OUTOFMEMORY; + TRACE("(%p %s %p)\n", pUnk, debugstr_guid(riid), pCP); - hr = IConnectionPoint_QueryInterface(&Obj->IConnectionPoint_iface, - &IID_IConnectionPoint, (void**)pCP); - IConnectionPoint_Release(&Obj->IConnectionPoint_iface); - return hr; + *pCP = NULL; + Obj = HeapAlloc(GetProcessHeap(), 0, sizeof(*Obj)); + if (!Obj) + return E_OUTOFMEMORY; + + Obj->IConnectionPoint_iface.lpVtbl = &ConnectionPointImpl_VTable; + Obj->Obj = pUnk; + Obj->ref = 1; + Obj->iid = *riid; + Obj->maxSinks = MAXSINKS; + Obj->sinks = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IUnknown*) * MAXSINKS); + Obj->nSinks = 0; + + *pCP = &Obj->IConnectionPoint_iface; + return S_OK; }

1 0

Alexandre Julliard : user32/tests: Update some todos for changed test failure behavior.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 7f956031bb4f493e7fde9ff30517a30eb13eb756 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f956031bb4f493e7fde9ff30… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu May 7 22:28:41 2015 +0900 user32/tests: Update some todos for changed test failure behavior. --- dlls/user32/tests/winstation.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/winstation.c b/dlls/user32/tests/winstation.c index c09ebb1..30c56c3 100644 --- a/dlls/user32/tests/winstation.c +++ b/dlls/user32/tests/winstation.c @@ -880,7 +880,10 @@ static void test_foregroundwindow(void) if (input_desk_id == thread_desk_id) { ok(ret, "SetForegroundWindow failed!\n"); - ok(hwnd == hwnd_test , "unexpected foreground window %p\n", hwnd); + if (hwnd) + ok(hwnd == hwnd_test , "unexpected foreground window %p\n", hwnd); + else + todo_wine ok(hwnd == hwnd_test , "unexpected foreground window %p\n", hwnd); } else { @@ -893,12 +896,18 @@ static void test_foregroundwindow(void) if (input_desk_id == thread_desk_id) { ok(!ret, "SetForegroundWindow should fail!\n"); - ok(hwnd == partners[input_desk_id] , "unexpected foreground window %p\n", hwnd); + if (hwnd) + ok(hwnd == partners[input_desk_id] , "unexpected foreground window %p\n", hwnd); + else + todo_wine ok(hwnd == partners[input_desk_id] , "unexpected foreground window %p\n", hwnd); } else { todo_wine ok(!ret, "SetForegroundWindow should fail!\n"); - todo_wine ok(hwnd == 0, "unexpected foreground window %p\n", hwnd); + if (!hwnd) + ok(hwnd == 0, "unexpected foreground window %p\n", hwnd); + else + todo_wine ok(hwnd == 0, "unexpected foreground window %p\n", hwnd); } } }

1 0

Marcus Meissner : winedbg: Close filehandle on one exit branch (Coverity).
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 92286a521f3272c3aec834b485be89d3e0ebae19 URL: http://source.winehq.org/git/wine.git/?a=commit;h=92286a521f3272c3aec834b48… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Thu May 7 10:39:20 2015 +0200 winedbg: Close filehandle on one exit branch (Coverity). --- programs/winedbg/source.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/programs/winedbg/source.c b/programs/winedbg/source.c index e564815..c21336a 100644 --- a/programs/winedbg/source.c +++ b/programs/winedbg/source.c @@ -87,7 +87,10 @@ static void* source_map_file(const char* name, HANDLE* hMap, unsigned* size) hFile = CreateFileA(name, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); if (hFile == INVALID_HANDLE_VALUE) return (void*)-1; - if (size != NULL && (*size = GetFileSize(hFile, NULL)) == -1) return (void*)-1; + if (size != NULL && (*size = GetFileSize(hFile, NULL)) == INVALID_FILE_SIZE) { + CloseHandle(hFile); + return (void*)-1; + } *hMap = CreateFileMappingW(hFile, NULL, PAGE_READONLY, 0, 0, NULL); CloseHandle(hFile); if (!*hMap) return (void*)-1;

1 0

Marcus Meissner : winedbg: Initialize type.module (Coverity).
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: aa59e6744dce853678f3fb4f5ee149320ca7765e URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa59e6744dce853678f3fb4f5… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Thu May 7 10:37:10 2015 +0200 winedbg: Initialize type.module (Coverity). --- programs/winedbg/types.c | 1 + 1 file changed, 1 insertion(+) diff --git a/programs/winedbg/types.c b/programs/winedbg/types.c index de790c7..31e7d38 100644 --- a/programs/winedbg/types.c +++ b/programs/winedbg/types.c @@ -697,6 +697,7 @@ BOOL types_print_type(const struct dbg_type* type, BOOL details) } else { + subtype.module = 0; dbg_printf("<ret_type=self>"); } dbg_printf(" (*%s)(", name);

1 0

Alexandre Julliard : ntdll: Fix handling of async cancellation for directory changes.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 193667ecd7e8543d48d6c2f16cfd0abca818c3c8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=193667ecd7e8543d48d6c2f16… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu May 7 16:02:26 2015 +0900 ntdll: Fix handling of async cancellation for directory changes. --- dlls/ntdll/file.c | 113 ++++++++++++++++++++++++---------------------- dlls/ntdll/tests/change.c | 10 ++-- server/change.c | 2 +- 3 files changed, 65 insertions(+), 60 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index 4c6b1b7..43e52c6 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -1805,73 +1805,78 @@ static NTSTATUS read_changes_apc( void *user, IO_STATUS_BLOCK *iosb, NTSTATUS status, void **apc, void **arg ) { struct read_changes_fileio *fileio = user; - NTSTATUS ret; - int size; + int size = 0; - SERVER_START_REQ( read_change ) - { - req->handle = wine_server_obj_handle( fileio->io.handle ); - wine_server_set_reply( req, fileio->data, fileio->data_size ); - ret = wine_server_call( req ); - size = wine_server_reply_size( reply ); - } - SERVER_END_REQ; - - if (ret == STATUS_SUCCESS && fileio->buffer) + if (status == STATUS_ALERTED) { - FILE_NOTIFY_INFORMATION *pfni = fileio->buffer; - int i, left = fileio->buffer_size; - DWORD *last_entry_offset = NULL; - struct filesystem_event *event = (struct filesystem_event*)fileio->data; - - while (size && left >= sizeof(*pfni)) + SERVER_START_REQ( read_change ) { - /* convert to an NT style path */ - for (i=0; i<event->len; i++) - if (event->name[i] == '/') event->name[i] = '\\'; - - pfni->Action = event->action; - pfni->FileNameLength = ntdll_umbstowcs( 0, event->name, event->len, pfni->FileName, - (left - offsetof(FILE_NOTIFY_INFORMATION, FileName)) / sizeof(WCHAR)); - last_entry_offset = &pfni->NextEntryOffset; - - if (pfni->FileNameLength == -1 || pfni->FileNameLength == -2) break; - - i = offsetof(FILE_NOTIFY_INFORMATION, FileName[pfni->FileNameLength]); - pfni->FileNameLength *= sizeof(WCHAR); - pfni->NextEntryOffset = i; - pfni = (FILE_NOTIFY_INFORMATION*)((char*)pfni + i); - left -= i; - - i = (offsetof(struct filesystem_event, name[event->len]) - + sizeof(int)-1) / sizeof(int) * sizeof(int); - event = (struct filesystem_event*)((char*)event + i); - size -= i; + req->handle = wine_server_obj_handle( fileio->io.handle ); + wine_server_set_reply( req, fileio->data, fileio->data_size ); + status = wine_server_call( req ); + size = wine_server_reply_size( reply ); } + SERVER_END_REQ; - if (size) + if (status == STATUS_SUCCESS && fileio->buffer) { - ret = STATUS_NOTIFY_ENUM_DIR; - size = 0; + FILE_NOTIFY_INFORMATION *pfni = fileio->buffer; + int i, left = fileio->buffer_size; + DWORD *last_entry_offset = NULL; + struct filesystem_event *event = (struct filesystem_event*)fileio->data; + + while (size && left >= sizeof(*pfni)) + { + /* convert to an NT style path */ + for (i = 0; i < event->len; i++) + if (event->name[i] == '/') event->name[i] = '\\'; + + pfni->Action = event->action; + pfni->FileNameLength = ntdll_umbstowcs( 0, event->name, event->len, pfni->FileName, + (left - offsetof(FILE_NOTIFY_INFORMATION, FileName)) / sizeof(WCHAR)); + last_entry_offset = &pfni->NextEntryOffset; + + if (pfni->FileNameLength == -1 || pfni->FileNameLength == -2) break; + + i = offsetof(FILE_NOTIFY_INFORMATION, FileName[pfni->FileNameLength]); + pfni->FileNameLength *= sizeof(WCHAR); + pfni->NextEntryOffset = i; + pfni = (FILE_NOTIFY_INFORMATION*)((char*)pfni + i); + left -= i; + + i = (offsetof(struct filesystem_event, name[event->len]) + + sizeof(int)-1) / sizeof(int) * sizeof(int); + event = (struct filesystem_event*)((char*)event + i); + size -= i; + } + + if (size) + { + status = STATUS_NOTIFY_ENUM_DIR; + size = 0; + } + else + { + if (last_entry_offset) *last_entry_offset = 0; + size = fileio->buffer_size - left; + } } else { - *last_entry_offset = 0; - size = fileio->buffer_size - left; + status = STATUS_NOTIFY_ENUM_DIR; + size = 0; } } - else + + if (status != STATUS_PENDING) { - ret = STATUS_NOTIFY_ENUM_DIR; - size = 0; + iosb->u.Status = status; + iosb->Information = size; + *apc = fileio->io.apc; + *arg = fileio->io.apc_arg; + release_fileio( &fileio->io ); } - - iosb->u.Status = ret; - iosb->Information = size; - *apc = fileio->io.apc; - *arg = fileio->io.apc_arg; - release_fileio( &fileio->io ); - return ret; + return status; } #define FILE_NOTIFY_ALL ( \ diff --git a/dlls/ntdll/tests/change.c b/dlls/ntdll/tests/change.c index 84e281c..0e76fb5 100644 --- a/dlls/ntdll/tests/change.c +++ b/dlls/ntdll/tests/change.c @@ -99,19 +99,19 @@ static void test_ntncdf(void) r = pNtNotifyChangeDirectoryFile(hdir,hEvent,NULL,NULL,&iosb,buffer,sizeof buffer,filter,0); ok(r==STATUS_PENDING, "should return status pending\n"); - r = WaitForSingleObject( hEvent, 0 ); + r = WaitForSingleObject( hEvent, 100 ); ok( r == STATUS_TIMEOUT, "should timeout\n" ); - r = WaitForSingleObject( hdir, 0 ); + r = WaitForSingleObject( hdir, 100 ); ok( r == STATUS_TIMEOUT, "should timeout\n" ); r = CreateDirectoryW( subdir, NULL ); ok( r == TRUE, "failed to create directory\n"); - r = WaitForSingleObject( hdir, 0 ); + r = WaitForSingleObject( hdir, 100 ); ok( r == STATUS_TIMEOUT, "should timeout\n" ); - r = WaitForSingleObject( hEvent, 0 ); + r = WaitForSingleObject( hEvent, 100 ); ok( r == WAIT_OBJECT_0, "event should be ready\n" ); ok( U(iosb).Status == STATUS_SUCCESS, "information wrong\n"); @@ -299,7 +299,7 @@ static void test_ntncdf_async(void) CloseHandle(hdir); ok(U(iosb).Status == STATUS_SUCCESS, "status wrong\n"); - todo_wine ok(U(iosb2).Status == STATUS_CANCELLED, "status wrong\n"); + ok(U(iosb2).Status == STATUS_CANCELLED, "status wrong %x\n",U(iosb2).Status); ok(iosb.Information == 0, "info wrong\n"); ok(iosb2.Information == 0, "info wrong\n"); diff --git a/server/change.c b/server/change.c index 08cee9b..11d9b93 100644 --- a/server/change.c +++ b/server/change.c @@ -272,7 +272,7 @@ void sigio_callback(void) LIST_FOR_EACH_ENTRY( dir, &change_list, struct dir, entry ) { if (interlocked_xchg( &dir->notified, 0 )) - fd_async_wake_up( dir->fd, ASYNC_TYPE_WAIT, STATUS_NOTIFY_ENUM_DIR ); + fd_async_wake_up( dir->fd, ASYNC_TYPE_WAIT, STATUS_ALERTED ); } }

1 0

Alexandre Julliard : ntdll: Don't run user APCs in NtCancelIoFile.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: 38b7c8839c9a9b6c02a2bf76250cb6f339dfb1e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=38b7c8839c9a9b6c02a2bf762… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu May 7 15:13:34 2015 +0900 ntdll: Don't run user APCs in NtCancelIoFile. --- dlls/ntdll/file.c | 22 ---------------------- dlls/ntdll/tests/file.c | 10 ++++------ dlls/ntdll/tests/pipe.c | 1 - 3 files changed, 4 insertions(+), 29 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index e43ef41..4c6b1b7 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -3457,8 +3457,6 @@ NTSTATUS WINAPI NtDeleteFile( POBJECT_ATTRIBUTES ObjectAttributes ) */ NTSTATUS WINAPI NtCancelIoFileEx( HANDLE hFile, PIO_STATUS_BLOCK iosb, PIO_STATUS_BLOCK io_status ) { - LARGE_INTEGER timeout; - TRACE("%p %p %p\n", hFile, iosb, io_status ); SERVER_START_REQ( cancel_async ) @@ -3469,16 +3467,7 @@ NTSTATUS WINAPI NtCancelIoFileEx( HANDLE hFile, PIO_STATUS_BLOCK iosb, PIO_STATU io_status->u.Status = wine_server_call( req ); } SERVER_END_REQ; - if (io_status->u.Status) - return io_status->u.Status; - /* Let some APC be run, so that we can run the remaining APCs on hFile - * either the cancelation of the pending one, but also the execution - * of the queued APC, but not yet run. This is needed to ensure proper - * clean-up of allocated data. - */ - timeout.QuadPart = 0; - NtDelayExecution( TRUE, &timeout ); return io_status->u.Status; } @@ -3489,8 +3478,6 @@ NTSTATUS WINAPI NtCancelIoFileEx( HANDLE hFile, PIO_STATUS_BLOCK iosb, PIO_STATU */ NTSTATUS WINAPI NtCancelIoFile( HANDLE hFile, PIO_STATUS_BLOCK io_status ) { - LARGE_INTEGER timeout; - TRACE("%p %p\n", hFile, io_status ); SERVER_START_REQ( cancel_async ) @@ -3501,16 +3488,7 @@ NTSTATUS WINAPI NtCancelIoFile( HANDLE hFile, PIO_STATUS_BLOCK io_status ) io_status->u.Status = wine_server_call( req ); } SERVER_END_REQ; - if (io_status->u.Status) - return io_status->u.Status; - /* Let some APC be run, so that we can run the remaining APCs on hFile - * either the cancelation of the pending one, but also the execution - * of the queued APC, but not yet run. This is needed to ensure proper - * clean-up of allocated data. - */ - timeout.QuadPart = 0; - NtDelayExecution( TRUE, &timeout ); return io_status->u.Status; } diff --git a/dlls/ntdll/tests/file.c b/dlls/ntdll/tests/file.c index c02c926..19a886a 100644 --- a/dlls/ntdll/tests/file.c +++ b/dlls/ntdll/tests/file.c @@ -668,7 +668,7 @@ static void read_file_test(void) ok( U(iosb).Status == STATUS_CANCELLED, "wrong status %x\n", U(iosb).Status ); ok( iosb.Information == 0, "wrong info %lu\n", iosb.Information ); ok( is_signaled( event ), "event is signaled\n" ); - todo_wine ok( !apc_count, "apc was called\n" ); + ok( !apc_count, "apc was called\n" ); SleepEx( 1, TRUE ); /* alertable sleep */ ok( apc_count == 1, "apc was not called\n" ); @@ -694,7 +694,7 @@ static void read_file_test(void) ok( U(iosb).Status == STATUS_CANCELLED, "wrong status %x\n", U(iosb).Status ); ok( iosb.Information == 0, "wrong info %lu\n", iosb.Information ); ok( is_signaled( event ), "event is signaled\n" ); - todo_wine ok( !apc_count, "apc was called\n" ); + ok( !apc_count, "apc was called\n" ); SleepEx( 1, TRUE ); /* alertable sleep */ ok( apc_count == 1, "apc was not called\n" ); CloseHandle( handle ); @@ -720,7 +720,7 @@ static void read_file_test(void) ok( U(iosb).Status == STATUS_CANCELLED, "wrong status %x\n", U(iosb).Status ); ok( iosb.Information == 0, "wrong info %lu\n", iosb.Information ); ok( is_signaled( event ), "event is signaled\n" ); - todo_wine ok( !apc_count, "apc was called\n" ); + ok( !apc_count, "apc was called\n" ); SleepEx( 1, TRUE ); /* alertable sleep */ ok( apc_count == 1, "apc was not called\n" ); @@ -746,7 +746,7 @@ static void read_file_test(void) ok( U(iosb).Status == STATUS_CANCELLED, "wrong status %x\n", U(iosb).Status ); ok( iosb.Information == 0, "wrong info %lu\n", iosb.Information ); ok( is_signaled( event ), "event is signaled\n" ); - todo_wine ok( !apc_count, "apc was called\n" ); + ok( !apc_count, "apc was called\n" ); SleepEx( 1, TRUE ); /* alertable sleep */ ok( apc_count == 2, "apc was not called\n" ); @@ -2521,7 +2521,6 @@ static void test_read_write(void) ret = ReadFile(hfile, buf, 0, &bytes, &ovl); /* ReadFile return value depends on Windows version and testing it is not practical */ if (!ret) -todo_wine ok(GetLastError() == ERROR_IO_PENDING, "expected ERROR_IO_PENDING, got %d\n", GetLastError()); ret = GetLastError(); ok(bytes == 0, "bytes %u\n", bytes); @@ -2552,7 +2551,6 @@ todo_wine ret = ReadFile(hfile, NULL, 0, &bytes, &ovl); /* ReadFile return value depends on Windows version and testing it is not practical */ if (!ret) -todo_wine ok(GetLastError() == ERROR_IO_PENDING, "expected ERROR_IO_PENDING, got %d\n", GetLastError()); ret = GetLastError(); ok(bytes == 0, "bytes %u\n", bytes); diff --git a/dlls/ntdll/tests/pipe.c b/dlls/ntdll/tests/pipe.c index 9fadb0f..20d82f4 100644 --- a/dlls/ntdll/tests/pipe.c +++ b/dlls/ntdll/tests/pipe.c @@ -474,7 +474,6 @@ static void test_cancelio(void) ok(U(iosb).Status == STATUS_CANCELLED, "Wrong iostatus %x\n", U(iosb).Status); ok(WaitForSingleObject(hEvent, 0) == 0, "hEvent not signaled\n"); - todo_wine ok(!ioapc_called, "IOAPC ran too early\n"); SleepEx(0, TRUE); /* alertable wait state */

1 0

Sebastian Lackner : ntdll/tests: Add/ remove todo_wine to avoid test failures.
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: e40e5661ee9a78d83195eb79b347cdc552dc694a URL: http://source.winehq.org/git/wine.git/?a=commit;h=e40e5661ee9a78d83195eb79b… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Thu May 7 01:15:12 2015 +0200 ntdll/tests: Add/remove todo_wine to avoid test failures. --- dlls/ntdll/tests/pipe.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/dlls/ntdll/tests/pipe.c b/dlls/ntdll/tests/pipe.c index 4bf2ad2..9fadb0f 100644 --- a/dlls/ntdll/tests/pipe.c +++ b/dlls/ntdll/tests/pipe.c @@ -469,13 +469,12 @@ static void test_cancelio(void) ok(res == STATUS_PENDING, "NtFsControlFile returned %x\n", res); res = pNtCancelIoFile(hPipe, &cancel_sb); - todo_wine ok(!res, "NtCancelIoFile returned %x\n", res); + ok(!res, "NtCancelIoFile returned %x\n", res); - todo_wine { - ok(U(iosb).Status == STATUS_CANCELLED, "Wrong iostatus %x\n", U(iosb).Status); - ok(WaitForSingleObject(hEvent, 0) == 0, "hEvent not signaled\n"); - } + ok(U(iosb).Status == STATUS_CANCELLED, "Wrong iostatus %x\n", U(iosb).Status); + ok(WaitForSingleObject(hEvent, 0) == 0, "hEvent not signaled\n"); + todo_wine ok(!ioapc_called, "IOAPC ran too early\n"); SleepEx(0, TRUE); /* alertable wait state */

1 0

Marcus Meissner : oleaut32: Avoid a library handle leak (Coverity).
by Alexandre Julliard May 7, 2015

May 7, 2015

Module: wine Branch: master Commit: f4f8679ae98189b756601679d7be872b22207a60 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f4f8679ae98189b756601679d… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Wed May 6 17:31:27 2015 +0200 oleaut32: Avoid a library handle leak (Coverity). --- dlls/oleaut32/olepropframe.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/oleaut32/olepropframe.c b/dlls/oleaut32/olepropframe.c index b89d3a6..73af745 100644 --- a/dlls/oleaut32/olepropframe.c +++ b/dlls/oleaut32/olepropframe.c @@ -210,6 +210,7 @@ HRESULT WINAPI OleCreatePropertyFrameIndirect(LPOCPFIPARAMS lpParams) if(property_sheet_dialog_data) { if(property_sheet_dialog_data[1] == 0xffff) { ERR("Expected DLGTEMPLATE structure\n"); + FreeLibrary(hcomctl); return E_OUTOFMEMORY; }

1 0