July 2012 - Wine-commits - list.winehq.org

Nozomi Kodama : d3dx9: Implementation of D3DXSHMultiply2.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: f757f3fb7d8a1cccecc496834d8d82516676792f URL: http://source.winehq.org/git/wine.git/?a=commit;h=f757f3fb7d8a1cccecc496834… Author: Nozomi Kodama <Nozomi.Kodama(a)yahoo.com> Date: Fri Jul 13 02:53:18 2012 +0200 d3dx9: Implementation of D3DXSHMultiply2. --- dlls/d3dx9_36/d3dx9_36.spec | 2 +- dlls/d3dx9_36/math.c | 17 +++++++++++++++++ dlls/d3dx9_36/tests/math.c | 23 +++++++++++++++++++++++ include/d3dx9math.h | 1 + 4 files changed, 42 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/d3dx9_36.spec b/dlls/d3dx9_36/d3dx9_36.spec index 58e8636..9230d55 100644 --- a/dlls/d3dx9_36/d3dx9_36.spec +++ b/dlls/d3dx9_36/d3dx9_36.spec @@ -279,7 +279,7 @@ @ stub D3DXSHEvalDirectionalLight(long ptr long long long long ptr ptr ptr) @ stub D3DXSHEvalHemisphereLight(long ptr long long ptr ptr ptr) @ stub D3DXSHEvalSphericalLight(long ptr long long long long ptr ptr ptr) -@ stub D3DXSHMultiply2(ptr ptr ptr) +@ stdcall D3DXSHMultiply2(ptr ptr ptr) @ stdcall D3DXSHMultiply3(ptr ptr ptr) @ stub D3DXSHMultiply4(ptr ptr ptr) @ stub D3DXSHMultiply5(ptr ptr ptr) diff --git a/dlls/d3dx9_36/math.c b/dlls/d3dx9_36/math.c index 2406929..79a0bca 100644 --- a/dlls/d3dx9_36/math.c +++ b/dlls/d3dx9_36/math.c @@ -1995,6 +1995,23 @@ FLOAT WINAPI D3DXSHDot(UINT order, CONST FLOAT *a, CONST FLOAT *b) return s; } +FLOAT* WINAPI D3DXSHMultiply2(FLOAT *out, CONST FLOAT *a, CONST FLOAT *b) +{ + FLOAT ta, tb; + + TRACE("(%p, %p, %p)\n", out, a, b); + + ta = 0.28209479f * a[0]; + tb = 0.28209479f * b[0]; + + out[0]= 0.28209479f * D3DXSHDot(2, a, b); + out[1] = ta * b[1] + tb * a[1]; + out[2] = ta * b[2] + tb * a[2]; + out[3] = ta * b[3] + tb * a[3]; + + return out; +} + FLOAT* WINAPI D3DXSHMultiply3(FLOAT *out, CONST FLOAT *a, CONST FLOAT *b) { FLOAT t, ta, tb; diff --git a/dlls/d3dx9_36/tests/math.c b/dlls/d3dx9_36/tests/math.c index 018081c..66d4689 100644 --- a/dlls/d3dx9_36/tests/math.c +++ b/dlls/d3dx9_36/tests/math.c @@ -2422,6 +2422,28 @@ static void test_D3DXSHDot(void) return; } +static void test_D3DXSHMultiply2(void) +{ + unsigned int i; + FLOAT a[20], b[20], c[20]; + /* D3DXSHMultiply2 only modifies the first 4 elements of the array */ + const FLOAT expected[20] = + { 3.418594f, 1.698211f, 1.703853f, 1.709494f, 4.0f, 5.0f, + 6.0f, 7.0f, 8.0f, 9.0f, 10.0f, 11.0f, 12.0f, 13.0f, + 14.0f, 15.0f, 16.0f, 17.0f, 18.0f, 19.0f }; + + for (i = 0; i < 20; i++) + { + a[i] = 1.0f + i / 100.0f; + b[i] = 3.0f - i / 100.0f; + c[i] = i; + } + + D3DXSHMultiply2(c, a, b); + for (i = 0; i < 20; i++) + ok(relative_error(c[i], expected[i]) < admitted_error, "Expected[%d] = %f, received = %f\n", i, expected[i], c[i]); +} + static void test_D3DXSHMultiply3(void) { unsigned int i; @@ -2490,6 +2512,7 @@ START_TEST(math) test_D3DXFloat_Array(); test_D3DXSHAdd(); test_D3DXSHDot(); + test_D3DXSHMultiply2(); test_D3DXSHMultiply3(); test_D3DXSHScale(); } diff --git a/include/d3dx9math.h b/include/d3dx9math.h index e31d432..ab3e2b0 100644 --- a/include/d3dx9math.h +++ b/include/d3dx9math.h @@ -379,6 +379,7 @@ FLOAT *WINAPI D3DXFloat16To32Array(FLOAT *pout, CONST D3DXFLOAT16 *pin, UINT n); FLOAT* WINAPI D3DXSHAdd(FLOAT *out, UINT order, CONST FLOAT *a, CONST FLOAT *b); FLOAT WINAPI D3DXSHDot(UINT order, CONST FLOAT *a, CONST FLOAT *b); +FLOAT* WINAPI D3DXSHMultiply2(FLOAT *out, CONST FLOAT *a, CONST FLOAT *b); FLOAT* WINAPI D3DXSHMultiply3(FLOAT *out, CONST FLOAT *a, CONST FLOAT *b); FLOAT* WINAPI D3DXSHScale(FLOAT *out, UINT order, CONST FLOAT *a, CONST FLOAT scale);

1 0

Francois Gouget : appwiz.cpl: Only offer to download an addon if its installer is missing.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: 0f36dc8df8e057d3e97670df0d1d77a6690e45fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f36dc8df8e057d3e97670df0… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Jul 16 10:09:32 2012 +0200 appwiz.cpl: Only offer to download an addon if its installer is missing. --- dlls/appwiz.cpl/addons.c | 49 +++++++++++++++++++++++++++------------------ 1 files changed, 29 insertions(+), 20 deletions(-) diff --git a/dlls/appwiz.cpl/addons.c b/dlls/appwiz.cpl/addons.c index 8e93dee..b2445d4 100644 --- a/dlls/appwiz.cpl/addons.c +++ b/dlls/appwiz.cpl/addons.c @@ -175,25 +175,31 @@ static void set_status(DWORD id) SendMessageW(status, WM_SETTEXT, 0, (LPARAM)buf); } -static BOOL install_file(const WCHAR *file_name) +enum install_res { + INSTALL_OK = 0, + INSTALL_FAILED, + INSTALL_NEXT, +}; + +static enum install_res install_file(const WCHAR *file_name) { ULONG res; res = MsiInstallProductW(file_name, NULL); if(res != ERROR_SUCCESS) { ERR("MsiInstallProduct failed: %u\n", res); - return FALSE; + return INSTALL_FAILED; } - return TRUE; + return INSTALL_OK; } -static BOOL install_from_unix_file(const char *dir, const char *subdir, const char *file_name) +static enum install_res install_from_unix_file(const char *dir, const char *subdir, const char *file_name) { LPWSTR dos_file_name; char *file_path; int fd, len; - BOOL ret; + enum install_res ret; static WCHAR * (CDECL *wine_get_dos_file_name)(const char*); static const WCHAR kernel32W[] = {'k','e','r','n','e','l','3','2','.','d','l','l',0}; @@ -201,7 +207,7 @@ static BOOL install_from_unix_file(const char *dir, const char *subdir, const ch len = strlen(dir); file_path = heap_alloc(len+strlen(subdir)+strlen(file_name)+3); if(!file_path) - return FALSE; + return INSTALL_FAILED; memcpy(file_path, dir, len); if(len && file_path[len-1] != '/' && file_path[len-1] != '\\') @@ -217,7 +223,7 @@ static BOOL install_from_unix_file(const char *dir, const char *subdir, const ch if(fd == -1) { TRACE("%s not found\n", debugstr_a(file_path)); heap_free(file_path); - return FALSE; + return INSTALL_NEXT; } close(fd); @@ -230,7 +236,7 @@ static BOOL install_from_unix_file(const char *dir, const char *subdir, const ch if(!dos_file_name) { ERR("Could not get dos file name of %s\n", debugstr_a(file_path)); heap_free(file_path); - return FALSE; + return INSTALL_FAILED; } } else { /* Windows mode */ UINT res; @@ -265,16 +271,16 @@ static HKEY open_config_key(void) return res == ERROR_SUCCESS ? ret : NULL; } -static BOOL install_from_registered_dir(void) +static enum install_res install_from_registered_dir(void) { char *package_dir; HKEY hkey; DWORD res, type, size = MAX_PATH; - BOOL ret; + enum install_res ret; hkey = open_config_key(); if(!hkey) - return FALSE; + return INSTALL_NEXT; package_dir = heap_alloc(size); res = RegGetValueA(hkey, NULL, addon->dir_config_key, RRF_RT_ANY, &type, (PBYTE)package_dir, &size); @@ -283,9 +289,12 @@ static BOOL install_from_registered_dir(void) res = RegGetValueA(hkey, NULL, addon->dir_config_key, RRF_RT_ANY, &type, (PBYTE)package_dir, &size); } RegCloseKey(hkey); - if(res != ERROR_SUCCESS || (type != REG_SZ && type != REG_EXPAND_SZ)) { + if(res == ERROR_FILE_NOT_FOUND) { heap_free(package_dir); - return FALSE; + return INSTALL_NEXT; + } else if(res != ERROR_SUCCESS || (type != REG_SZ && type != REG_EXPAND_SZ)) { + heap_free(package_dir); + return INSTALL_FAILED; } TRACE("Trying %s/%s\n", debugstr_a(package_dir), debugstr_a(addon->file_name)); @@ -296,7 +305,7 @@ static BOOL install_from_registered_dir(void) return ret; } -static BOOL install_from_default_dir(void) +static enum install_res install_from_default_dir(void) { const char *data_dir, *package_dir; char *dir_buf = NULL; @@ -312,15 +321,15 @@ static BOOL install_from_default_dir(void) strcpy(dir_buf+len, "/../"); package_dir = dir_buf; }else { - return FALSE; + return INSTALL_NEXT; } ret = install_from_unix_file(package_dir, addon->subdir_name, addon->file_name); heap_free(dir_buf); - if (!ret) + if (ret == INSTALL_NEXT) ret = install_from_unix_file(INSTALL_DATADIR "/wine/", addon->subdir_name, addon->file_name); - if (!ret && strcmp(INSTALL_DATADIR, "/usr/share")) + if (ret == INSTALL_NEXT && strcmp(INSTALL_DATADIR, "/usr/share")) ret = install_from_unix_file("/usr/share/wine/", addon->subdir_name, addon->file_name); return ret; } @@ -588,9 +597,9 @@ BOOL install_addon(addon_t addon_type) * - /usr/share/wine/$addon_subdir/ * - download from URL stored in $url_config_key value of HKCU/Wine/Software/$config_key key */ - if(!install_from_registered_dir() - && !install_from_default_dir() - && (url = get_url())) + if (install_from_registered_dir() == INSTALL_NEXT + && install_from_default_dir() == INSTALL_NEXT + && (url = get_url())) DialogBoxW(hInst, addon->dialog_template, 0, installer_proc); heap_free(url);

1 0

Nikolay Sivov : ole32: COM cleanup of IPropertySetStorage interface.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: ac032593c050ca0b6fb7234cbba458b39db25dff URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac032593c050ca0b6fb7234cb… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Jul 14 20:42:48 2012 +0400 ole32: COM cleanup of IPropertySetStorage interface. --- dlls/ole32/stg_prop.c | 16 +++++++--------- dlls/ole32/stg_stream.c | 4 ++-- dlls/ole32/storage32.c | 13 ++++++------- dlls/ole32/storage32.h | 2 +- 4 files changed, 16 insertions(+), 19 deletions(-) diff --git a/dlls/ole32/stg_prop.c b/dlls/ole32/stg_prop.c index 5e22a40..9e3df45 100644 --- a/dlls/ole32/stg_prop.c +++ b/dlls/ole32/stg_prop.c @@ -60,7 +60,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(storage); static inline StorageImpl *impl_from_IPropertySetStorage( IPropertySetStorage *iface ) { - return (StorageImpl *)((char*)iface - FIELD_OFFSET(StorageImpl, base.pssVtbl)); + return CONTAINING_RECORD(iface, StorageImpl, base.IPropertySetStorage_iface); } /* These are documented in MSDN, @@ -2100,7 +2100,7 @@ static HRESULT WINAPI IPropertySetStorage_fnQueryInterface( void** ppvObject) { StorageImpl *This = impl_from_IPropertySetStorage(ppstg); - return IStorage_QueryInterface( (IStorage*)This, riid, ppvObject ); + return IStorage_QueryInterface( &This->base.IStorage_iface, riid, ppvObject ); } /************************************************************************ @@ -2112,7 +2112,7 @@ static ULONG WINAPI IPropertySetStorage_fnAddRef( IPropertySetStorage *ppstg) { StorageImpl *This = impl_from_IPropertySetStorage(ppstg); - return IStorage_AddRef( (IStorage*)This ); + return IStorage_AddRef( &This->base.IStorage_iface ); } /************************************************************************ @@ -2124,7 +2124,7 @@ static ULONG WINAPI IPropertySetStorage_fnRelease( IPropertySetStorage *ppstg) { StorageImpl *This = impl_from_IPropertySetStorage(ppstg); - return IStorage_Release( (IStorage*)This ); + return IStorage_Release( &This->base.IStorage_iface ); } /************************************************************************ @@ -2173,7 +2173,7 @@ static HRESULT WINAPI IPropertySetStorage_fnCreate( if (FAILED(r)) goto end; - r = IStorage_CreateStream( (IStorage*)This, name, grfMode, 0, 0, &stm ); + r = IStorage_CreateStream( &This->base.IStorage_iface, name, grfMode, 0, 0, &stm ); if (FAILED(r)) goto end; @@ -2218,7 +2218,7 @@ static HRESULT WINAPI IPropertySetStorage_fnOpen( if (FAILED(r)) goto end; - r = IStorage_OpenStream((IStorage*) This, name, 0, grfMode, 0, &stm ); + r = IStorage_OpenStream( &This->base.IStorage_iface, name, 0, grfMode, 0, &stm ); if (FAILED(r)) goto end; @@ -2237,7 +2237,6 @@ static HRESULT WINAPI IPropertySetStorage_fnDelete( REFFMTID rfmtid) { StorageImpl *This = impl_from_IPropertySetStorage(ppstg); - IStorage *stg = NULL; WCHAR name[CCH_MAX_PROPSTG_NAME]; HRESULT r; @@ -2250,8 +2249,7 @@ static HRESULT WINAPI IPropertySetStorage_fnDelete( if (FAILED(r)) return r; - stg = (IStorage*) This; - return IStorage_DestroyElement(stg, name); + return IStorage_DestroyElement(&This->base.IStorage_iface, name); } /************************************************************************ diff --git a/dlls/ole32/stg_stream.c b/dlls/ole32/stg_stream.c index 019a789..6a39b94 100644 --- a/dlls/ole32/stg_stream.c +++ b/dlls/ole32/stg_stream.c @@ -57,7 +57,7 @@ static void StgStreamImpl_Destroy(StgStreamImpl* This) /* * Release the reference we are holding on the parent storage. - * IStorage_Release((IStorage*)This->parentStorage); + * IStorage_Release(&This->parentStorage->IStorage_iface); * * No, don't do this. Some apps call IStorage_Release without * calling IStream_Release first. If we grab a reference the @@ -719,7 +719,7 @@ StgStreamImpl* StgStreamImpl_Construct( * We want to nail-down the reference to the storage in case the * stream out-lives the storage in the client application. * - * -- IStorage_AddRef((IStorage*)newStream->parentStorage); + * -- IStorage_AddRef(&newStream->parentStorage->IStorage_iface); * * No, don't do this. Some apps call IStorage_Release without * calling IStream_Release first. If we grab a reference the diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index 4e802f5..df84108 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -373,11 +373,11 @@ static HRESULT WINAPI StorageBaseImpl_QueryInterface( if (IsEqualGUID(&IID_IUnknown, riid) || IsEqualGUID(&IID_IStorage, riid)) { - *ppvObject = This; + *ppvObject = &This->IStorage_iface; } else if (IsEqualGUID(&IID_IPropertySetStorage, riid)) { - *ppvObject = &This->pssVtbl; + *ppvObject = &This->IPropertySetStorage_iface; } if ((*ppvObject)==0) @@ -2718,7 +2718,7 @@ static HRESULT StorageImpl_Construct( list_init(&This->base.storageHead); This->base.IStorage_iface.lpVtbl = &Storage32Impl_Vtbl; - This->base.pssVtbl = &IPropertySetStorage_Vtbl; + This->base.IPropertySetStorage_iface.lpVtbl = &IPropertySetStorage_Vtbl; This->base.baseVtbl = &StorageImpl_BaseVtbl; This->base.openFlags = (openFlags & ~STGM_CREATE); This->base.ref = 1; @@ -5095,8 +5095,7 @@ static HRESULT TransactedSnapshotImpl_Construct(StorageBaseImpl *parentStorage, (*result)->base.IStorage_iface.lpVtbl = &TransactedSnapshotImpl_Vtbl; /* This is OK because the property set storage functions use the IStorage functions. */ - (*result)->base.pssVtbl = parentStorage->pssVtbl; - + (*result)->base.IPropertySetStorage_iface.lpVtbl = parentStorage->IPropertySetStorage_iface.lpVtbl; (*result)->base.baseVtbl = &TransactedSnapshotImpl_BaseVtbl; list_init(&(*result)->base.strmHead); @@ -5693,7 +5692,7 @@ static StorageInternalImpl* StorageInternalImpl_Construct( * Initialize the virtual function table. */ newStorage->base.IStorage_iface.lpVtbl = &Storage32InternalImpl_Vtbl; - newStorage->base.pssVtbl = &IPropertySetStorage_Vtbl; + newStorage->base.IPropertySetStorage_iface.lpVtbl = &IPropertySetStorage_Vtbl; newStorage->base.baseVtbl = &StorageInternalImpl_BaseVtbl; newStorage->base.openFlags = (openFlags & ~STGM_CREATE); @@ -9519,7 +9518,7 @@ HRESULT WINAPI ReadClassStm(IStream *pStm,CLSID *pclsid) /* clear the output args */ *pclsid = CLSID_NULL; - res = IStream_Read(pStm,(void*)pclsid,sizeof(CLSID),&nbByte); + res = IStream_Read(pStm, pclsid, sizeof(CLSID), &nbByte); if (FAILED(res)) return res; diff --git a/dlls/ole32/storage32.h b/dlls/ole32/storage32.h index d94740d..80fe01b 100644 --- a/dlls/ole32/storage32.h +++ b/dlls/ole32/storage32.h @@ -176,7 +176,7 @@ HRESULT OLECONVERT_CreateCompObjStream(LPSTORAGE pStorage, LPCSTR strOleTypeName struct StorageBaseImpl { IStorage IStorage_iface; - const IPropertySetStorageVtbl *pssVtbl; /* interface for adding a properties stream */ + IPropertySetStorage IPropertySetStorage_iface; /* interface for adding a properties stream */ LONG ref; /*

1 0

Nikolay Sivov : ole32: No need to test for null implementation pointer in methods.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: a3a97558de148116f5bfea66c142e8ff4b9f5990 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a3a97558de148116f5bfea66c… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Jul 14 21:13:40 2012 +0400 ole32: No need to test for null implementation pointer in methods. --- dlls/ole32/storage32.c | 17 ++++++++--------- 1 files changed, 8 insertions(+), 9 deletions(-) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index df84108..c04675d 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -365,7 +365,7 @@ static HRESULT WINAPI StorageBaseImpl_QueryInterface( { StorageBaseImpl *This = impl_from_IStorage(iface); - if ( (This==0) || (ppvObject==0) ) + if (!ppvObject) return E_INVALIDARG; *ppvObject = 0; @@ -379,8 +379,7 @@ static HRESULT WINAPI StorageBaseImpl_QueryInterface( { *ppvObject = &This->IPropertySetStorage_iface; } - - if ((*ppvObject)==0) + else return E_NOINTERFACE; IStorage_AddRef(iface); @@ -578,7 +577,7 @@ static HRESULT WINAPI StorageBaseImpl_OpenStorage( iface, debugstr_w(pwcsName), pstgPriority, grfMode, snbExclude, reserved, ppstg); - if ( (This==0) || (pwcsName==NULL) || (ppstg==0) ) + if ((pwcsName==NULL) || (ppstg==0) ) { res = E_INVALIDARG; goto end; @@ -709,7 +708,7 @@ static HRESULT WINAPI StorageBaseImpl_EnumElements( TRACE("(%p, %d, %p, %d, %p)\n", iface, reserved1, reserved2, reserved3, ppenum); - if ( (This==0) || (ppenum==0)) + if (!ppenum) return E_INVALIDARG; if (This->reverted) @@ -719,7 +718,7 @@ static HRESULT WINAPI StorageBaseImpl_EnumElements( This, This->storageDirEntry); - if (newEnum!=0) + if (newEnum) { *ppenum = &newEnum->IEnumSTATSTG_iface; @@ -750,7 +749,7 @@ static HRESULT WINAPI StorageBaseImpl_Stat( TRACE("(%p, %p, %x)\n", iface, pstatstg, grfStatFlag); - if ( (This==0) || (pstatstg==0)) + if (!pstatstg) { res = E_INVALIDARG; goto end; @@ -2076,7 +2075,7 @@ static HRESULT deleteStorageContents( /* * Open the storage and enumerate it */ - hr = StorageBaseImpl_OpenStorage( + hr = IStorage_OpenStorage( &parentStorage->IStorage_iface, entryDataToDelete.name, 0, @@ -2209,11 +2208,11 @@ static HRESULT removeFromTree( DirRef parentStorageIndex, DirRef deletedIndex) { - HRESULT hr = S_OK; DirEntry entryToDelete; DirEntry parentEntry; DirRef parentEntryRef; ULONG typeOfRelation; + HRESULT hr; hr = StorageBaseImpl_ReadDirEntry(This, deletedIndex, &entryToDelete);

1 0

Nikolay Sivov : ole32: Implement SetConvertStg().
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: 39562961283ccb7c838c5650b11873e69a8d8e56 URL: http://source.winehq.org/git/wine.git/?a=commit;h=39562961283ccb7c838c5650b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Jul 14 22:39:15 2012 +0400 ole32: Implement SetConvertStg(). --- dlls/ole32/ole2stubs.c | 9 ----- dlls/ole32/storage32.c | 78 +++++++++++++++++++++++++++++++++++++++--- dlls/ole32/tests/storage32.c | 5 ++- 3 files changed, 76 insertions(+), 16 deletions(-) diff --git a/dlls/ole32/ole2stubs.c b/dlls/ole32/ole2stubs.c index 6988dfe..5a90e2d 100644 --- a/dlls/ole32/ole2stubs.c +++ b/dlls/ole32/ole2stubs.c @@ -44,15 +44,6 @@ HRESULT WINAPI OleCreateLinkToFile(LPCOLESTR lpszFileName, REFIID riid, } /****************************************************************************** - * SetConvertStg [OLE32.@] - */ -HRESULT WINAPI SetConvertStg(LPSTORAGE pStg, BOOL fConvert) -{ - FIXME("(%p,%x), stub!\n", pStg, fConvert); - return E_NOTIMPL; -} - -/****************************************************************************** * OleCreateLink [OLE32.@] */ HRESULT WINAPI OleCreateLink(LPMONIKER pmkLinkSrc, REFIID riid, DWORD renderopt, LPFORMATETC lpFormatEtc, diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index c04675d..a1c925f 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -9383,7 +9383,7 @@ HRESULT WINAPI OleConvertIStorageToOLESTREAM ( enum stream_1ole_flags { OleStream_LinkedObject = 0x00000001, - OleStream_Convert = 0x00000100 + OleStream_Convert = 0x00000004 }; /*********************************************************************** @@ -9396,7 +9396,6 @@ HRESULT WINAPI GetConvertStg(IStorage *stg) DWORD header[2]; IStream *stream; HRESULT hr; - ULONG len; TRACE("%p\n", stg); @@ -9405,20 +9404,89 @@ HRESULT WINAPI GetConvertStg(IStorage *stg) hr = IStorage_OpenStream(stg, stream_1oleW, NULL, STGM_READ | STGM_SHARE_EXCLUSIVE, 0, &stream); if (FAILED(hr)) return hr; - len = 0; - hr = IStream_Read(stream, header, sizeof(header), &len); + hr = IStream_Read(stream, header, sizeof(header), NULL); IStream_Release(stream); if (FAILED(hr)) return hr; if (header[0] != version_magic) { - ERR("got wrong version magic for \1Ole stream, 0x%08x\n", header[0]); + ERR("got wrong version magic for 1Ole stream, 0x%08x\n", header[0]); return E_FAIL; } return header[1] & OleStream_Convert ? S_OK : S_FALSE; } +/*********************************************************************** + * SetConvertStg (OLE32.@) + */ +HRESULT WINAPI SetConvertStg(IStorage *storage, BOOL convert) +{ + static const WCHAR stream_1oleW[] = {1,'O','l','e',0}; + static const DWORD version_magic = 0x02000001; + DWORD flags = convert ? OleStream_Convert : 0; + IStream *stream; + HRESULT hr; + + TRACE("(%p, %d)\n", storage, convert); + + hr = IStorage_CreateStream(storage, stream_1oleW, STGM_WRITE | STGM_SHARE_EXCLUSIVE, 0, 0, &stream); + if (hr == S_OK) + { + struct empty_1ole_stream { + DWORD version_magic; + DWORD flags; + BYTE padding[12]; + }; + struct empty_1ole_stream stream_data; + + stream_data.version_magic = version_magic; + stream_data.flags = flags; + memset(stream_data.padding, 0, sizeof(stream_data.padding)); + + hr = IStream_Write(stream, &stream_data, sizeof(stream_data), NULL); + IStream_Release(stream); + } + else if (hr == STG_E_FILEALREADYEXISTS) + { + DWORD header[2]; + + hr = IStorage_OpenStream(storage, stream_1oleW, NULL, STGM_READWRITE | STGM_SHARE_EXCLUSIVE, 0, &stream); + if (FAILED(hr)) return hr; + + hr = IStream_Read(stream, header, sizeof(header), NULL); + if (FAILED(hr)) + { + IStream_Release(stream); + return hr; + } + + /* update flag if differs */ + if ((header[1] ^ flags) & OleStream_Convert) + { + LARGE_INTEGER pos; + + if (header[1] & OleStream_Convert) + flags = header[1] & ~OleStream_Convert; + else + flags = header[1] | OleStream_Convert; + + pos.QuadPart = sizeof(DWORD); + hr = IStream_Seek(stream, pos, STREAM_SEEK_SET, NULL); + if (FAILED(hr)) + { + IStream_Release(stream); + return hr; + } + + hr = IStream_Write(stream, &flags, sizeof(flags), NULL); + } + IStream_Release(stream); + } + + return hr; +} + /****************************************************************************** * StgIsStorageFile [OLE32.@] * Verify if the file contains a storage object diff --git a/dlls/ole32/tests/storage32.c b/dlls/ole32/tests/storage32.c index 86b8d8e..c79b5d1 100644 --- a/dlls/ole32/tests/storage32.c +++ b/dlls/ole32/tests/storage32.c @@ -3009,7 +3009,8 @@ static void test_convert(void) hr = GetConvertStg(stg); ok(hr == STG_E_FILENOTFOUND, "got 0x%08x\n", hr); hr = SetConvertStg(stg, TRUE); -todo_wine { + ok(hr == S_OK, "got 0x%08x\n", hr); + hr = SetConvertStg(stg, TRUE); ok(hr == S_OK, "got 0x%08x\n", hr); hr = GetConvertStg(stg); ok(hr == S_OK, "got 0x%08x\n", hr); @@ -3017,7 +3018,7 @@ todo_wine { ok(hr == S_OK, "got 0x%08x\n", hr); hr = GetConvertStg(stg); ok(hr == S_FALSE, "got 0x%08x\n", hr); -} + IStorage_Release(stg); DeleteFileW(filename);

1 0

Nikolay Sivov : msxml3: Protect from null pointer in load().
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: 13d95525c3f6886f29df658b2d03a415abcbf5e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=13d95525c3f6886f29df658b2… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Jul 14 22:52:35 2012 +0400 msxml3: Protect from null pointer in load(). --- dlls/msxml3/domdoc.c | 1 + dlls/msxml3/tests/domdoc.c | 7 +++++++ 2 files changed, 8 insertions(+), 0 deletions(-) diff --git a/dlls/msxml3/domdoc.c b/dlls/msxml3/domdoc.c index cbbebf2..0f3eb50 100644 --- a/dlls/msxml3/domdoc.c +++ b/dlls/msxml3/domdoc.c @@ -2093,6 +2093,7 @@ static HRESULT WINAPI domdoc_load( } break; case VT_UNKNOWN: + if (!V_UNKNOWN(&source)) return E_INVALIDARG; hr = IUnknown_QueryInterface(V_UNKNOWN(&source), &IID_IXMLDOMDocument3, (void**)&pNewDoc); if(hr == S_OK) { diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index 46b628f..dfa6126 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -10749,6 +10749,13 @@ static void test_load(void) doc = create_document(&IID_IXMLDOMDocument); + /* null pointer as input */ + V_VT(&src) = VT_UNKNOWN; + V_UNKNOWN(&src) = NULL; + hr = IXMLDOMDocument_load(doc, src, &b); + EXPECT_HR(hr, E_INVALIDARG); + ok(b == VARIANT_FALSE, "got %d\n", b); + path = _bstr_("test.xml"); /* load from path: VT_BSTR */

1 0

Ken Thomases : include/wincrypt.h: Add identifiers for Elliptic curve Diffie-Hellman algorithm.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: ba12844f325704531f0ec5903a47f8a94b4beab9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ba12844f325704531f0ec5903… Author: Ken Thomases <ken(a)codeweavers.com> Date: Fri Jul 13 15:19:09 2012 -0500 include/wincrypt.h: Add identifiers for Elliptic curve Diffie-Hellman algorithm. --- include/wincrypt.h | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/wincrypt.h b/include/wincrypt.h index 1311f31..a14edab 100644 --- a/include/wincrypt.h +++ b/include/wincrypt.h @@ -1578,6 +1578,7 @@ typedef const CERT_CRL_CONTEXT_PAIR *PCCERT_CRL_CONTEXT_PAIR; #define ALG_SID_DH_EPHEM 2 #define ALG_SID_AGREED_KEY_ANY 3 #define ALG_SID_KEA 4 +#define ALG_SID_ECDH 5 /* RC2 SIDs */ #define ALG_SID_RC4 1 #define ALG_SID_RC2 2 @@ -1627,6 +1628,7 @@ typedef const CERT_CRL_CONTEXT_PAIR *PCCERT_CRL_CONTEXT_PAIR; #define CALG_NO_SIGN (ALG_CLASS_SIGNATURE | ALG_TYPE_ANY | ALG_SID_ANY) #define CALG_DH_SF (ALG_CLASS_KEY_EXCHANGE | ALG_TYPE_DH | ALG_SID_DH_SANDF) #define CALG_DH_EPHEM (ALG_CLASS_KEY_EXCHANGE | ALG_TYPE_DH | ALG_SID_DH_EPHEM) +#define CALG_ECDH (ALG_CLASS_KEY_EXCHANGE | ALG_TYPE_DH | ALG_SID_ECDH) #define CALG_RSA_KEYX (ALG_CLASS_KEY_EXCHANGE | ALG_TYPE_RSA | ALG_SID_RSA_ANY) #define CALG_DES (ALG_CLASS_DATA_ENCRYPT | ALG_TYPE_BLOCK | ALG_SID_DES) #define CALG_RC2 (ALG_CLASS_DATA_ENCRYPT | ALG_TYPE_BLOCK | ALG_SID_RC2)

1 0

Ken Thomases : secur32: On Mac, recognize key exchange algorithm for cipher suites using ECDH.
by Alexandre Julliard 16 Jul '12

16 Jul '12

Module: wine Branch: master Commit: 4ccb8746fde08a21e533558a417816279e2c8620 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ccb8746fde08a21e533558a4… Author: Ken Thomases <ken(a)codeweavers.com> Date: Fri Jul 13 15:19:23 2012 -0500 secur32: On Mac, recognize key exchange algorithm for cipher suites using ECDH. --- dlls/secur32/schannel_macosx.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/secur32/schannel_macosx.c b/dlls/secur32/schannel_macosx.c index 01493b1..d4576b9 100644 --- a/dlls/secur32/schannel_macosx.c +++ b/dlls/secur32/schannel_macosx.c @@ -330,6 +330,11 @@ static ALG_ID schan_get_kx_algid(const struct cipher_suite* c) case schan_kx_DHE_DSS: case schan_kx_DHE_RSA_EXPORT: case schan_kx_DHE_RSA: return CALG_DH_EPHEM; + case schan_kx_ECDH_anon: + case schan_kx_ECDH_ECDSA: + case schan_kx_ECDH_RSA: + case schan_kx_ECDHE_ECDSA: + case schan_kx_ECDHE_RSA: return CALG_ECDH; case schan_kx_NULL: return 0; case schan_kx_RSA: return CALG_RSA_KEYX; @@ -339,11 +344,6 @@ static ALG_ID schan_get_kx_algid(const struct cipher_suite* c) case schan_kx_DH_DSS: case schan_kx_DH_RSA_EXPORT: case schan_kx_DH_RSA: - case schan_kx_ECDH_anon: - case schan_kx_ECDH_ECDSA: - case schan_kx_ECDH_RSA: - case schan_kx_ECDHE_ECDSA: - case schan_kx_ECDHE_RSA: case schan_kx_FORTEZZA_DMS: case schan_kx_RSA_EXPORT: FIXME("Don't know CALG for key exchange algorithm %d for cipher suite %#x, returning 0\n", c->kx_alg, (unsigned)c->suite);

1 0

Kyle Auble : Moved icon notes from contributing page to wiki (retry)
by Jeremy Newman 16 Jul '12

16 Jul '12

Module: website Branch: master Commit: 85888b471e2459593326cc2e9284479acf29615a URL: http://source.winehq.org/git/website.git/?a=commit;h=85888b471e2459593326cc… Author: Kyle Auble <randomidman48(a)yahoo.com> Date: Fri Jul 13 14:51:44 2012 -0700 Moved icon notes from contributing page to wiki (retry) --- templates/en/contributing.template | 13 +------------ 1 files changed, 1 insertions(+), 12 deletions(-) diff --git a/templates/en/contributing.template b/templates/en/contributing.template index 84989e7..a2a0b18 100644 --- a/templates/en/contributing.template +++ b/templates/en/contributing.template @@ -19,7 +19,7 @@ <li><a href="#web_site">General maintenance</a> <ol> <li><a href="http://wiki.winehq.org/CreateFonts">Creating fonts</a></li> - <li><a href="#draw">Draw icons for Wine</a></li> + <li><a href="http://wiki.winehq.org/CreateIcons">Drawing icons</a></li> <li><a href="#app_owners">Application owners</a></li> <li><a href="#support">Bug hunting and support</a></li> </ol> @@ -43,17 +43,6 @@ </li> </ol> -<h3 id="draw">Draw icons for Wine</h3> - -<p>Wine cannot reuse the Windows icons verbatim because of copyright issues. So we have to redraw - them, keeping a similar design so that users are not completely lost, all while making them - different enough (and possibly nicer). You'll see this if you open a file selection dialog.</p> -<p>Required skills:</p> -<ul> - <li>Drawing skills</li> - <li>Vector graphic skills</li> -</ul> - <h3 id="app_owners">Application database: application maintainers</h3> <p>Another very important aspect of Wine testing is to use or test real

1 0

Kyle Auble : Moved AppDB notes from contributing page to wiki (retry)
by Jeremy Newman 16 Jul '12

16 Jul '12

Module: website Branch: master Commit: 8efce17cc58485e7ad94bff14e2a43850c585ef2 URL: http://source.winehq.org/git/website.git/?a=commit;h=8efce17cc58485e7ad94bf… Author: Kyle Auble <randomidman48(a)yahoo.com> Date: Fri Jul 13 14:51:59 2012 -0700 Moved AppDB notes from contributing page to wiki (retry) --- templates/en/contributing.template | 34 +--------------------------------- 1 files changed, 1 insertions(+), 33 deletions(-) diff --git a/templates/en/contributing.template b/templates/en/contributing.template index a2a0b18..0d019f3 100644 --- a/templates/en/contributing.template +++ b/templates/en/contributing.template @@ -20,7 +20,7 @@ <ol> <li><a href="http://wiki.winehq.org/CreateFonts">Creating fonts</a></li> <li><a href="http://wiki.winehq.org/CreateIcons">Drawing icons</a></li> - <li><a href="#app_owners">Application owners</a></li> + <li><a href="http://wiki.winehq.org/AppDB_Maintainers">Maintaining applications</a></li> <li><a href="#support">Bug hunting and support</a></li> </ol> </li> @@ -43,38 +43,6 @@ </li> </ol> -<h3 id="app_owners">Application database: application maintainers</h3> - -<p>Another very important aspect of Wine testing is to use or test real - applications regularly and report regressions as soon as possible to the Wine - developers. This way regressions are caught early, while the set of changes - to search through is still small and the Wine developer who made the change - still around. Another reason why <b>your</b> contribution is especially - valuable is that Wine developers probably do not have access to the - application (they cannot buy all the applications out there) or do not know - it well enough to test it properly.</p> -<p>You can help even further by becoming an 'application maintainer', that is by - maintaining that application entry in the <a href="http://appdb.winehq.org/">Application Database</a>. Many - applications can be made to run (or run better) by using just the right mix - of native (Windows) and built-in (Wine) libraries. By testing the application - with various library mix and then documenting your results in the - corresponding Application Database entry, you will help other Wine users be - productive with that application, thus making Wine useful to a greater number - of people.</p> -<p>Required skills:</p> -<ul> - <li>Good knowledge of a particular application</li> - <li>Being dedicated to using / testing this application with each new Wine - release or more often. It is more important to test one application - regularly than ten applications once a year.</li> - <li>Report all regressions to wine-devel, and cooperate with the developers - to diagnose the issue.</li> - <li>Create and maintain a HOWTO note on the application page summarizing - all actions required to get the application to run properly, e.g. - collected from the comments of visitors on the application/versions pages.</li> - <li>Coding skills are <strong>not necessary</strong>.</li> -</ul> - <h3 id="support">Bug hunting and support</h3> <p>Because there are so many Windows applications out there, and because very few of them work perfectly, we get a lot of questions from users,

1 0