wine-commits November 2009

wine-commits@winehq.org

3 participants
910 discussions

Roderick Colenbrander : opengl32: Fix wglCreateContextAttribsARB test on nvidia.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: c89fb41f802e6870e5d440d89f3eac5ee9c74b6f URL: http://source.winehq.org/git/wine.git/?a=commit;h=c89fb41f802e6870e5d440d89… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Mon Nov 23 20:18:54 2009 +0100 opengl32: Fix wglCreateContextAttribsARB test on nvidia. --- dlls/opengl32/tests/opengl.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c index 4709685..b2d8e18 100644 --- a/dlls/opengl32/tests/opengl.c +++ b/dlls/opengl32/tests/opengl.c @@ -551,6 +551,8 @@ static void test_dc(HWND hwnd, HDC hdc) } } +/* Nvidia converts win32 error codes to (0xc007 << 16) | win32_error_code */ +#define NVIDIA_HRESULT_FROM_WIN32(x) (HRESULT_FROM_WIN32(x) | 0x40000000) static void test_opengl3(HDC hdc) { /* Try to create a context compatible with OpenGL 1.x; 1.0-2.1 is allowed */ @@ -571,7 +573,7 @@ static void test_opengl3(HDC hdc) ok(gl3Ctx == 0, "pwglCreateContextAttribsARB using an invalid HDC passed\n"); error = GetLastError(); todo_wine ok(error == ERROR_DC_NOT_FOUND || - broken(HRESULT_FROM_WIN32(ERROR_INVALID_DATA)), /* Nvidia Vista + Win7 */ + broken(error == NVIDIA_HRESULT_FROM_WIN32(ERROR_INVALID_DATA)), /* Nvidia Vista + Win7 */ "Expected ERROR_DC_NOT_FOUND, got error=%x\n", error); wglDeleteContext(gl3Ctx); } @@ -585,7 +587,7 @@ static void test_opengl3(HDC hdc) error = GetLastError(); /* The Nvidia implementation seems to return hresults instead of win32 error codes */ todo_wine ok(error == ERROR_INVALID_OPERATION || - error == HRESULT_FROM_WIN32(ERROR_INVALID_OPERATION), "Expected ERROR_INVALID_OPERATION, got error=%x\n", error); + error == NVIDIA_HRESULT_FROM_WIN32(ERROR_INVALID_OPERATION), "Expected ERROR_INVALID_OPERATION, got error=%x\n", error); wglDeleteContext(gl3Ctx); }

1 0

Paul Vriens : advapi32/tests: Fix a test failure on W2K3 without a service pack.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: da92afa3e1c93d99c69e71fbe8d9209f2a5a41d0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=da92afa3e1c93d99c69e71fbe… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Tue Nov 24 09:36:57 2009 +0100 advapi32/tests: Fix a test failure on W2K3 without a service pack. --- dlls/advapi32/tests/eventlog.c | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) diff --git a/dlls/advapi32/tests/eventlog.c b/dlls/advapi32/tests/eventlog.c index c6272ff..207fec5 100644 --- a/dlls/advapi32/tests/eventlog.c +++ b/dlls/advapi32/tests/eventlog.c @@ -741,6 +741,27 @@ static void test_readwrite(void) return; } + count = 0xdeadbeef; + GetNumberOfEventLogRecords(handle, &count); + if (count != 0) + { + /* Needed for W2K3 without a service pack */ + win_skip("We most likely opened the Application eventlog\n"); + CloseEventLog(handle); + Sleep(2000); + + handle = OpenEventLogA(NULL, eventlogname); + count = 0xdeadbeef; + GetNumberOfEventLogRecords(handle, &count); + if (count != 0) + { + win_skip("We didn't open our new eventlog\n"); + HeapFree(GetProcessHeap(), 0, user); + CloseEventLog(handle); + return; + } + } + SetLastError(0xdeadbeef); ret = ReportEvent(handle, 0x20, 0, 0, NULL, 0, 0, NULL, NULL); if (!ret && GetLastError() == ERROR_CRC)

1 0

Paul Vriens : advapi32/tests: Fix an intermittent crash on NT4.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 91648ff4beaf8bf6e90c9e9b2caaf19da16c641f URL: http://source.winehq.org/git/wine.git/?a=commit;h=91648ff4beaf8bf6e90c9e9b2… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Tue Nov 24 08:53:18 2009 +0100 advapi32/tests: Fix an intermittent crash on NT4. --- dlls/advapi32/tests/eventlog.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/dlls/advapi32/tests/eventlog.c b/dlls/advapi32/tests/eventlog.c index e71f823..c6272ff 100644 --- a/dlls/advapi32/tests/eventlog.c +++ b/dlls/advapi32/tests/eventlog.c @@ -733,6 +733,13 @@ static void test_readwrite(void) * but succeed on all others, hence it's not part of the struct. */ handle = OpenEventLogA(NULL, eventlogname); + if (!handle) + { + /* Intermittently seen on NT4 when tests are run immediately after boot */ + win_skip("Could not get a handle to the eventlog\n"); + HeapFree(GetProcessHeap(), 0, user); + return; + } SetLastError(0xdeadbeef); ret = ReportEvent(handle, 0x20, 0, 0, NULL, 0, 0, NULL, NULL);

1 0

Alexandre Julliard : wing: Reimplement support for WinGGetDIBPointer16.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 92cd5a5c6bae3cd018187f6b1df01eeb74e6e152 URL: http://source.winehq.org/git/wine.git/?a=commit;h=92cd5a5c6bae3cd018187f6b1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Nov 24 10:58:55 2009 +0100 wing: Reimplement support for WinGGetDIBPointer16. --- dlls/wing.dll16/wing.c | 83 ++++++++++++++++++++++++++++++++++++----------- 1 files changed, 63 insertions(+), 20 deletions(-) diff --git a/dlls/wing.dll16/wing.c b/dlls/wing.dll16/wing.c index 750c8da..fb4aa56 100644 --- a/dlls/wing.dll16/wing.c +++ b/dlls/wing.dll16/wing.c @@ -27,10 +27,64 @@ #include "wingdi.h" #include "wownt32.h" #include "wine/wingdi16.h" +#include "wine/list.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(wing); +struct dib_segptr_bits +{ + struct list entry; + HBITMAP bmp; + WORD sel; + WORD count; +}; + +static struct list dib_segptr_list = LIST_INIT( dib_segptr_list ); + +/* remove saved bits for bitmaps that no longer exist */ +static void cleanup_segptr_bits(void) +{ + unsigned int i; + struct dib_segptr_bits *bits, *next; + + LIST_FOR_EACH_ENTRY_SAFE( bits, next, &dib_segptr_list, struct dib_segptr_bits, entry ) + { + if (GetObjectType( bits->bmp ) == OBJ_BITMAP) continue; + for (i = 0; i < bits->count; i++) FreeSelector16( bits->sel + (i << __AHSHIFT) ); + list_remove( &bits->entry ); + HeapFree( GetProcessHeap(), 0, bits ); + } +} + +static SEGPTR alloc_segptr_bits( HBITMAP bmp, void *bits32 ) +{ + DIBSECTION dib; + unsigned int i, size; + struct dib_segptr_bits *bits; + + cleanup_segptr_bits(); + + if (!(bits = HeapAlloc( GetProcessHeap(), 0, sizeof(*bits) ))) return 0; + + GetObjectW( bmp, sizeof(dib), &dib ); + size = dib.dsBm.bmHeight * dib.dsBm.bmWidthBytes; + + /* calculate number of sel's needed for size with 64K steps */ + bits->bmp = bmp; + bits->count = (size + 0xffff) / 0x10000; + bits->sel = AllocSelectorArray16( bits->count ); + + for (i = 0; i < bits->count; i++) + { + SetSelectorBase(bits->sel + (i << __AHSHIFT), (DWORD)bits32 + i * 0x10000); + SetSelectorLimit16(bits->sel + (i << __AHSHIFT), size - 1); /* yep, limit is correct */ + size -= 0x10000; + } + list_add_head( &dib_segptr_list, &bits->entry ); + return MAKESEGPTR( bits->sel, 0 ); +} + /************************************************************************* * WING {WING} * @@ -127,25 +181,8 @@ HBITMAP16 WINAPI WinGCreateBitmap16(HDC16 hdc, BITMAPINFO *bmpi, SEGPTR *bits) hbitmap = CreateDIBSection( HDC_32(hdc), bmpi, BI_RGB, &bits32, 0, 0 ); if (hbitmap) { - DIBSECTION dib; - DWORD size; - WORD count, sel; - int i; - - GetObjectW( hbitmap, sizeof(dib), &dib ); - size = dib.dsBm.bmHeight * dib.dsBm.bmWidthBytes; - - /* calculate number of sel's needed for size with 64K steps */ - count = (size + 0xffff) / 0x10000; - sel = AllocSelectorArray16(count); - - for (i = 0; i < count; i++) - { - SetSelectorBase(sel + (i << __AHSHIFT), (DWORD)bits32 + i * 0x10000); - SetSelectorLimit16(sel + (i << __AHSHIFT), size - 1); /* yep, limit is correct */ - size -= 0x10000; - } - if (bits) *bits = MAKESEGPTR( sel, 0 ); + SEGPTR segptr = alloc_segptr_bits( hbitmap, bits32 ); + if (bits) *bits = segptr; } return HBITMAP_16(hbitmap); } @@ -155,7 +192,13 @@ HBITMAP16 WINAPI WinGCreateBitmap16(HDC16 hdc, BITMAPINFO *bmpi, SEGPTR *bits) */ SEGPTR WINAPI WinGGetDIBPointer16(HBITMAP16 hWinGBitmap, BITMAPINFO* bmpi) { - FIXME("%x, %p: not supported\n", hWinGBitmap, bmpi ); + struct dib_segptr_bits *bits; + + if (bmpi) FIXME( "%04x %p: setting BITMAPINFO not supported\n", hWinGBitmap, bmpi ); + + LIST_FOR_EACH_ENTRY( bits, &dib_segptr_list, struct dib_segptr_bits, entry ) + if (HBITMAP_16(bits->bmp) == hWinGBitmap) return MAKESEGPTR( bits->sel, 0 ); + return 0; }

1 0

Vincent Povirk : ole32: Allocate the StorageImpl in StorageImpl_Construct.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 9da3365e840466aa55a4cc248e7a7c0c5aa54ff5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9da3365e840466aa55a4cc248… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Mon Nov 23 17:49:32 2009 -0600 ole32: Allocate the StorageImpl in StorageImpl_Construct. This makes cleanup after errors much easier, as we can just use the regular destructor. --- dlls/ole32/storage32.c | 122 +++++++++++++++++++++--------------------------- 1 files changed, 54 insertions(+), 68 deletions(-) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index d4c6527..089206d 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -2233,14 +2233,15 @@ static const IStorageVtbl Storage32Impl_Vtbl = }; static HRESULT StorageImpl_Construct( - StorageImpl* This, HANDLE hFile, LPCOLESTR pwcsName, ILockBytes* pLkbyt, DWORD openFlags, BOOL fileBased, - BOOL create) + BOOL create, + StorageImpl** result) { + StorageImpl* This; HRESULT hr = S_OK; DirEntry currentEntry; BOOL readSuccessful; @@ -2249,6 +2250,10 @@ static HRESULT StorageImpl_Construct( if ( FAILED( validateSTGM(openFlags) )) return STG_E_INVALIDFLAG; + This = HeapAlloc(GetProcessHeap(), 0, sizeof(StorageImpl)); + if (!This) + return E_OUTOFMEMORY; + memset(This, 0, sizeof(StorageImpl)); list_init(&This->base.strmHead); @@ -2259,6 +2264,7 @@ static HRESULT StorageImpl_Construct( This->base.pssVtbl = &IPropertySetStorage_Vtbl; This->base.v_destructor = StorageImpl_Destroy; This->base.openFlags = (openFlags & ~STGM_CREATE); + This->base.ref = 1; This->create = create; /* @@ -2273,7 +2279,10 @@ static HRESULT StorageImpl_Construct( This->pwcsName = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(pwcsName)+1)*sizeof(WCHAR)); if (!This->pwcsName) - return STG_E_INSUFFICIENTMEMORY; + { + hr = STG_E_INSUFFICIENTMEMORY; + goto end; + } strcpyW(This->pwcsName, pwcsName); } @@ -2289,7 +2298,10 @@ static HRESULT StorageImpl_Construct( fileBased); if (This->bigBlockFile == 0) - return E_FAIL; + { + hr = E_FAIL; + goto end; + } if (create) { @@ -2341,9 +2353,7 @@ static HRESULT StorageImpl_Construct( if (FAILED(hr)) { - BIGBLOCKFILE_Destructor(This->bigBlockFile); - - return hr; + goto end; } } @@ -2362,12 +2372,18 @@ static HRESULT StorageImpl_Construct( */ if(!(This->rootBlockChain = BlockChainStream_Construct(This, &This->rootStartBlock, DIRENTRY_NULL))) - return STG_E_READFAULT; + { + hr = STG_E_READFAULT; + goto end; + } if(!(This->smallBlockDepotChain = BlockChainStream_Construct(This, &This->smallBlockDepotStart, DIRENTRY_NULL))) - return STG_E_READFAULT; + { + hr = STG_E_READFAULT; + goto end; + } /* * Write the root storage entry (memory only) @@ -2420,8 +2436,8 @@ static HRESULT StorageImpl_Construct( if (!readSuccessful) { - /* TODO CLEANUP */ - return STG_E_READFAULT; + hr = STG_E_READFAULT; + goto end; } /* @@ -2429,7 +2445,18 @@ static HRESULT StorageImpl_Construct( */ if(!(This->smallBlockRootChain = BlockChainStream_Construct(This, NULL, This->base.storageDirEntry))) - return STG_E_READFAULT; + { + hr = STG_E_READFAULT; + } + +end: + if (FAILED(hr)) + { + IStorage_Release((IStorage*)This); + *result = NULL; + } + else + *result = This; return hr; } @@ -2447,7 +2474,8 @@ static void StorageImpl_Destroy(StorageBaseImpl* iface) BlockChainStream_Destroy(This->rootBlockChain); BlockChainStream_Destroy(This->smallBlockDepotChain); - BIGBLOCKFILE_Destructor(This->bigBlockFile); + if (This->bigBlockFile) + BIGBLOCKFILE_Destructor(This->bigBlockFile); HeapFree(GetProcessHeap(), 0, This); } @@ -5663,36 +5691,25 @@ HRESULT WINAPI StgCreateDocfile( /* * Allocate and initialize the new IStorage32object. */ - newStorage = HeapAlloc(GetProcessHeap(), 0, sizeof(StorageImpl)); - - if (newStorage == 0) - { - hr = STG_E_INSUFFICIENTMEMORY; - goto end; - } - hr = StorageImpl_Construct( - newStorage, hFile, pwcsName, NULL, grfMode, TRUE, - TRUE); + TRUE, + &newStorage); if (FAILED(hr)) { - HeapFree(GetProcessHeap(), 0, newStorage); goto end; } /* * Get an "out" pointer for the caller. */ - hr = StorageBaseImpl_QueryInterface( - (IStorage*)newStorage, - &IID_IStorage, - (void**)ppstgOpen); + *ppstgOpen = (IStorage*)newStorage; + end: TRACE("<-- %p r = %08x\n", *ppstgOpen, hr); @@ -5947,27 +5964,17 @@ HRESULT WINAPI StgOpenStorage( /* * Allocate and initialize the new IStorage32object. */ - newStorage = HeapAlloc(GetProcessHeap(), 0, sizeof(StorageImpl)); - - if (newStorage == 0) - { - hr = STG_E_INSUFFICIENTMEMORY; - goto end; - } - - /* Initialize the storage */ hr = StorageImpl_Construct( - newStorage, hFile, pwcsName, NULL, grfMode, TRUE, - FALSE ); + FALSE, + &newStorage); if (FAILED(hr)) { - HeapFree(GetProcessHeap(), 0, newStorage); /* * According to the docs if the file is not a storage, return STG_E_FILEALREADYEXISTS */ @@ -5984,10 +5991,7 @@ HRESULT WINAPI StgOpenStorage( /* * Get an "out" pointer for the caller. */ - hr = StorageBaseImpl_QueryInterface( - (IStorage*)newStorage, - &IID_IStorage, - (void**)ppstgOpen); + *ppstgOpen = (IStorage*)newStorage; end: TRACE("<-- %08x, IStorage %p\n", hr, ppstgOpen ? *ppstgOpen : NULL); @@ -6012,33 +6016,24 @@ HRESULT WINAPI StgCreateDocfileOnILockBytes( /* * Allocate and initialize the new IStorage object. */ - newStorage = HeapAlloc(GetProcessHeap(), 0, sizeof(StorageImpl)); - - if (newStorage == 0) - return STG_E_INSUFFICIENTMEMORY; - hr = StorageImpl_Construct( - newStorage, 0, 0, plkbyt, grfMode, FALSE, - TRUE); + TRUE, + &newStorage); if (FAILED(hr)) { - HeapFree(GetProcessHeap(), 0, newStorage); return hr; } /* * Get an "out" pointer for the caller. */ - hr = StorageBaseImpl_QueryInterface( - (IStorage*)newStorage, - &IID_IStorage, - (void**)ppstgOpen); + *ppstgOpen = (IStorage*)newStorage; return hr; } @@ -6068,33 +6063,24 @@ HRESULT WINAPI StgOpenStorageOnILockBytes( /* * Allocate and initialize the new IStorage object. */ - newStorage = HeapAlloc(GetProcessHeap(), 0, sizeof(StorageImpl)); - - if (newStorage == 0) - return STG_E_INSUFFICIENTMEMORY; - hr = StorageImpl_Construct( - newStorage, 0, 0, plkbyt, grfMode, FALSE, - FALSE); + FALSE, + &newStorage); if (FAILED(hr)) { - HeapFree(GetProcessHeap(), 0, newStorage); return hr; } /* * Get an "out" pointer for the caller. */ - hr = StorageBaseImpl_QueryInterface( - (IStorage*)newStorage, - &IID_IStorage, - (void**)ppstgOpen); + *ppstgOpen = (IStorage*)newStorage; return hr; }

1 0

Vincent Povirk : ole32: Copy the storage filename at stat time, not read time.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 6c3961f0c49a0684c5153282aa3293908c33cd04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c3961f0c49a0684c5153282a… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Mon Nov 23 15:39:35 2009 -0600 ole32: Copy the storage filename at stat time, not read time. --- dlls/ole32/stg_stream.c | 3 ++- dlls/ole32/storage32.c | 31 +++++++++++++++++++++---------- dlls/ole32/storage32.h | 4 ++-- 3 files changed, 25 insertions(+), 13 deletions(-) diff --git a/dlls/ole32/stg_stream.c b/dlls/ole32/stg_stream.c index 8b731dd..332ee8a 100644 --- a/dlls/ole32/stg_stream.c +++ b/dlls/ole32/stg_stream.c @@ -860,7 +860,8 @@ static HRESULT WINAPI StgStreamImpl_Stat( { StorageImpl *root = This->parentStorage->ancestorStorage; - StorageUtl_CopyDirEntryToSTATSTG(pstatstg, + StorageUtl_CopyDirEntryToSTATSTG(root, + pstatstg, &currentEntry, grfStatFlag); diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index 708f263..d4c6527 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -698,6 +698,7 @@ static HRESULT WINAPI StorageBaseImpl_Stat( if (readSuccessful) { StorageUtl_CopyDirEntryToSTATSTG( + This->ancestorStorage, pstatstg, &currentEntry, grfStatFlag); @@ -3241,14 +3242,10 @@ BOOL StorageImpl_ReadDirEntry( if (SUCCEEDED(readRes)) { - /* replace the name of root entry (often "Root Entry") by the file name */ - WCHAR *entryName = (index == This->base.storageDirEntry) ? - This->filename : (WCHAR *)currentEntry+OFFSET_PS_NAME; - memset(buffer->name, 0, sizeof(buffer->name)); memcpy( buffer->name, - entryName, + (WCHAR *)currentEntry+OFFSET_PS_NAME, DIRENTRY_NAME_BUFFER_LEN ); TRACE("storage name: %s\n", debugstr_w(buffer->name)); @@ -3750,7 +3747,8 @@ static HRESULT WINAPI IEnumSTATSTGImpl_Next( /* * Copy the information to the return buffer. */ - StorageUtl_CopyDirEntryToSTATSTG(currentReturnStruct, + StorageUtl_CopyDirEntryToSTATSTG(This->parentStorage, + currentReturnStruct, &currentEntry, STATFLAG_DEFAULT); @@ -4179,25 +4177,38 @@ void StorageUtl_WriteGUID(BYTE* buffer, ULONG offset, const GUID* value) } void StorageUtl_CopyDirEntryToSTATSTG( + StorageImpl* storage, STATSTG* destination, const DirEntry* source, int statFlags) { + LPCWSTR entryName; + + if (source->stgType == STGTY_ROOT) + { + /* replace the name of root entry (often "Root Entry") by the file name */ + entryName = storage->filename; + } + else + { + entryName = source->name; + } + /* * The copy of the string occurs only when the flag is not set */ if( ((statFlags & STATFLAG_NONAME) != 0) || - (source->name == NULL) || - (source->name[0] == 0) ) + (entryName == NULL) || + (entryName[0] == 0) ) { destination->pwcsName = 0; } else { destination->pwcsName = - CoTaskMemAlloc((lstrlenW(source->name)+1)*sizeof(WCHAR)); + CoTaskMemAlloc((lstrlenW(entryName)+1)*sizeof(WCHAR)); - strcpyW(destination->pwcsName, source->name); + strcpyW(destination->pwcsName, entryName); } switch (source->stgType) diff --git a/dlls/ole32/storage32.h b/dlls/ole32/storage32.h index 4a953c1..508bd70 100644 --- a/dlls/ole32/storage32.h +++ b/dlls/ole32/storage32.h @@ -428,8 +428,8 @@ void StorageUtl_WriteULargeInteger(BYTE* buffer, ULONG offset, const ULARGE_INTEGER *value); void StorageUtl_ReadGUID(const BYTE* buffer, ULONG offset, GUID* value); void StorageUtl_WriteGUID(BYTE* buffer, ULONG offset, const GUID* value); -void StorageUtl_CopyDirEntryToSTATSTG(STATSTG* destination, const DirEntry* source, - int statFlags); +void StorageUtl_CopyDirEntryToSTATSTG(StorageImpl *storage,STATSTG* destination, + const DirEntry* source, int statFlags); /**************************************************************************** * BlockChainStream definitions.

1 0

Rob Shearman : ole32: Fix error code returned by CoGetClassObject/ CoCreateInstance in CLSCTX_REMOTE_SERVER case.
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 9943facb260e6b33d8ba33e575a43607206e5358 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9943facb260e6b33d8ba33e57… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Nov 23 21:52:27 2009 +0000 ole32: Fix error code returned by CoGetClassObject/CoCreateInstance in CLSCTX_REMOTE_SERVER case. --- dlls/ole32/compobj.c | 2 +- dlls/ole32/tests/compobj.c | 1 - 2 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/ole32/compobj.c b/dlls/ole32/compobj.c index 7dc1568..321f2e9 100644 --- a/dlls/ole32/compobj.c +++ b/dlls/ole32/compobj.c @@ -2380,7 +2380,7 @@ HRESULT WINAPI CoGetClassObject( if (CLSCTX_REMOTE_SERVER & dwClsContext) { FIXME ("CLSCTX_REMOTE_SERVER not supported\n"); - hres = E_NOINTERFACE; + hres = REGDB_E_CLASSNOTREG; } if (FAILED(hres)) diff --git a/dlls/ole32/tests/compobj.c b/dlls/ole32/tests/compobj.c index 21293bc..9d469b4 100644 --- a/dlls/ole32/tests/compobj.c +++ b/dlls/ole32/tests/compobj.c @@ -270,7 +270,6 @@ static void test_CoCreateInstance(void) hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_LOCAL_SERVER, &IID_IUnknown, (void **)&pUnk); ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered local server should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_REMOTE_SERVER, &IID_IUnknown, (void **)&pUnk); - todo_wine ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered remote server should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); hr = CoCreateInstance(rclsid, NULL, CLSCTX_INPROC_SERVER, &IID_IUnknown, (void **)&pUnk);

1 0

Rob Shearman : ole32: Add tests for errors returned by CoCreateInstance for classes that are not registered .
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 8554ce5b18e8d7a23af20b3eeb3a97f779369fa7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8554ce5b18e8d7a23af20b3ee… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Nov 23 21:52:22 2009 +0000 ole32: Add tests for errors returned by CoCreateInstance for classes that are not registered. --- dlls/ole32/tests/compobj.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/ole32/tests/compobj.c b/dlls/ole32/tests/compobj.c index e868014..21293bc 100644 --- a/dlls/ole32/tests/compobj.c +++ b/dlls/ole32/tests/compobj.c @@ -261,6 +261,18 @@ static void test_CoCreateInstance(void) ok(pUnk == NULL, "CoCreateInstance should have changed the passed in pointer to NULL, instead of %p\n", pUnk); OleInitialize(NULL); + + /* test errors returned for non-registered clsids */ + hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_INPROC_SERVER, &IID_IUnknown, (void **)&pUnk); + ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered inproc server should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); + hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_INPROC_HANDLER, &IID_IUnknown, (void **)&pUnk); + ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered inproc handler should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); + hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_LOCAL_SERVER, &IID_IUnknown, (void **)&pUnk); + ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered local server should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); + hr = CoCreateInstance(&CLSID_non_existent, NULL, CLSCTX_REMOTE_SERVER, &IID_IUnknown, (void **)&pUnk); + todo_wine + ok(hr == REGDB_E_CLASSNOTREG, "CoCreateInstance for non-registered remote server should have returned REGDB_E_CLASSNOTREG instead of 0x%08x\n", hr); + hr = CoCreateInstance(rclsid, NULL, CLSCTX_INPROC_SERVER, &IID_IUnknown, (void **)&pUnk); if(hr == REGDB_E_CLASSNOTREG) {

1 0

Marcus Meissner : wined3d: Handle NULL ptrs (Coverity).
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: a7e599db450ccd6a8dad83bf802f30022d1df772 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a7e599db450ccd6a8dad83bf8… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Mon Nov 23 22:35:15 2009 +0100 wined3d: Handle NULL ptrs (Coverity). --- dlls/wined3d/volume.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/volume.c b/dlls/wined3d/volume.c index bb5827e..5a80cb6 100644 --- a/dlls/wined3d/volume.c +++ b/dlls/wined3d/volume.c @@ -187,11 +187,13 @@ static HRESULT WINAPI IWineD3DVolumeImpl_GetContainer(IWineD3DVolume *iface, REF if (!ppContainer) { ERR("Called without a valid ppContainer.\n"); + return E_FAIL; } /* Although surfaces can be standalone, volumes can't */ if (!This->container) { ERR("Volume without an container. Should not happen.\n"); + return E_FAIL; } TRACE("Relaying to QueryInterface\n");

1 0

Marcus Meissner : ole32: Some missing error checking in FileMonikerImpl_CommonPrefixWith (Coverity) .
by Alexandre Julliard 24 Nov '09

24 Nov '09

Module: wine Branch: master Commit: 000e1e0ce05fdbdb8c669c7da9ec9719c204f155 URL: http://source.winehq.org/git/wine.git/?a=commit;h=000e1e0ce05fdbdb8c669c7da… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Mon Nov 23 22:23:36 2009 +0100 ole32: Some missing error checking in FileMonikerImpl_CommonPrefixWith (Coverity). --- dlls/ole32/filemoniker.c | 18 +++++++++++++++--- 1 files changed, 15 insertions(+), 3 deletions(-) diff --git a/dlls/ole32/filemoniker.c b/dlls/ole32/filemoniker.c index c179b7f..ff543aa 100644 --- a/dlls/ole32/filemoniker.c +++ b/dlls/ole32/filemoniker.c @@ -960,20 +960,32 @@ FileMonikerImpl_CommonPrefixWith(IMoniker* iface,IMoniker* pmkOther,IMoniker** p if(mkSys==MKSYS_FILEMONIKER){ HRESULT ret; - CreateBindCtx(0,&pbind); + ret = CreateBindCtx(0,&pbind); + if (FAILED(ret)) + return ret; /* create a string based on common part of the two paths */ - IMoniker_GetDisplayName(iface,pbind,NULL,&pathThis); - IMoniker_GetDisplayName(pmkOther,pbind,NULL,&pathOther); + ret = IMoniker_GetDisplayName(iface,pbind,NULL,&pathThis); + if (FAILED(ret)) + return ret; + ret = IMoniker_GetDisplayName(pmkOther,pbind,NULL,&pathOther); + if (FAILED(ret)) + return ret; nb1=FileMonikerImpl_DecomposePath(pathThis,&stringTable1); + if (FAILED(nb1)) + return nb1; nb2=FileMonikerImpl_DecomposePath(pathOther,&stringTable2); + if (FAILED(nb2)) + return nb2; if (nb1==0 || nb2==0) return MK_E_NOPREFIX; commonPath=HeapAlloc(GetProcessHeap(),0,sizeof(WCHAR)*(min(lstrlenW(pathThis),lstrlenW(pathOther))+1)); + if (!commonPath) + return E_OUTOFMEMORY; *commonPath=0;

1 0

← Newer
1
...
8
9
10
11
12
13
14
...
91
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 November 2009