December 2009 - Wine-Commits

Paul Vriens : rpcrt4/tests: Don't crash on NT4.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: 4d1fa5833eb13f47557912bb365b16974ee53144 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d1fa5833eb13f47557912bb3… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Wed Dec 16 09:00:30 2009 +0100 rpcrt4/tests: Don't crash on NT4. --- dlls/rpcrt4/tests/server.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/dlls/rpcrt4/tests/server.c b/dlls/rpcrt4/tests/server.c index 353d0cd..f55bfcf 100644 --- a/dlls/rpcrt4/tests/server.c +++ b/dlls/rpcrt4/tests/server.c @@ -1346,7 +1346,7 @@ s_authinfo_test(unsigned int protseq, int secure) todo_wine ok(principal != NULL, "NULL principal\n"); } - if (protseq == RPC_PROTSEQ_LRPC && principal) + if (protseq == RPC_PROTSEQ_LRPC && principal && pGetUserNameExA) { int len; char *spn; @@ -1399,6 +1399,9 @@ set_auth_info(RPC_BINDING_HANDLE handle) RPC_STATUS status; RPC_SECURITY_QOS qos; + if (!pGetUserNameExA) + return; + qos.Version = 1; qos.Capabilities = RPC_C_QOS_CAPABILITIES_MUTUAL_AUTH; qos.IdentityTracking = RPC_C_QOS_IDENTITY_STATIC; @@ -1573,6 +1576,8 @@ START_TEST(server) domain_and_user = HeapAlloc(GetProcessHeap(), 0, size); ok(pGetUserNameExA(NameSamCompatible, domain_and_user, &size), "GetUserNameExA\n"); } + else + win_skip("GetUserNameExA is needed for some authentication tests\n"); argc = winetest_get_mainargs(&argv); progname = argv[0];

1 0

Jeremy White : winspool: Add tests for paths returned in GetPrinterDriver.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: e4897a827bc70a2c56731f02adb31b3172d88faa URL: http://source.winehq.org/git/wine.git/?a=commit;h=e4897a827bc70a2c56731f02a… Author: Jeremy White <jwhite(a)codeweavers.com> Date: Tue Dec 15 20:33:45 2009 -0600 winspool: Add tests for paths returned in GetPrinterDriver. --- dlls/winspool.drv/tests/info.c | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/dlls/winspool.drv/tests/info.c b/dlls/winspool.drv/tests/info.c index 1f876b3..5ab0ccd 100644 --- a/dlls/winspool.drv/tests/info.c +++ b/dlls/winspool.drv/tests/info.c @@ -2370,6 +2370,7 @@ static void test_GetPrinterDriver(void) { DRIVER_INFO_2 *di_2 = (DRIVER_INFO_2 *)buf; DWORD calculated = sizeof(*di_2); + HANDLE hf; /* MSDN is wrong: The Drivers on the win9x-CD's have cVersion=0x0400 NT351: 1, NT4.0+w2k(Kernelmode): 2, w2k and above(Usermode): 3 */ @@ -2393,6 +2394,24 @@ static void test_GetPrinterDriver(void) trace("pConfigFile %s\n", di_2->pConfigFile); calculated += strlen(di_2->pConfigFile) + 1; + hf = CreateFileA(di_2->pDriverPath, GENERIC_READ, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); + if(hf != INVALID_HANDLE_VALUE) + CloseHandle(hf); + todo_wine + ok(hf != INVALID_HANDLE_VALUE, "Could not open %s\n", di_2->pDriverPath); + + hf = CreateFileA(di_2->pDataFile, GENERIC_READ, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); + if(hf != INVALID_HANDLE_VALUE) + CloseHandle(hf); + todo_wine + ok(hf != INVALID_HANDLE_VALUE, "Could not open %s\n", di_2->pDataFile); + + hf = CreateFileA(di_2->pConfigFile, GENERIC_READ, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); + if(hf != INVALID_HANDLE_VALUE) + CloseHandle(hf); + todo_wine + ok(hf != INVALID_HANDLE_VALUE, "Could not open %s\n", di_2->pConfigFile); + /* XP allocates memory for both ANSI and unicode names */ ok(filled >= calculated,"calculated %d != filled %d\n", calculated, filled); }

1 0

James Hawkins : msi: Refactor freeing the import sql strings.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: b530f38baa571738d061b38f62db6c63465f8619 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b530f38baa571738d061b38f6… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:19:25 2009 -0800 msi: Refactor freeing the import sql strings. --- dlls/msi/database.c | 21 ++++++++++----------- 1 files changed, 10 insertions(+), 11 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index 3896454..6ce516a 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -493,10 +493,10 @@ done: static UINT msi_add_table_to_db(MSIDATABASE *db, LPWSTR *columns, LPWSTR *types, LPWSTR *labels, DWORD num_labels, DWORD num_columns) { - UINT r; + UINT r = ERROR_OUTOFMEMORY; DWORD size; MSIQUERY *view; - LPWSTR create_sql; + LPWSTR create_sql = NULL; LPWSTR prelude, columns_sql, postlude; prelude = msi_build_createsql_prelude(labels[0]); @@ -504,31 +504,30 @@ static UINT msi_add_table_to_db(MSIDATABASE *db, LPWSTR *columns, LPWSTR *types, postlude = msi_build_createsql_postlude(labels + 1, num_labels - 1); /* skip over table name */ if (!prelude || !columns_sql || !postlude) - return ERROR_OUTOFMEMORY; + goto done; size = lstrlenW(prelude) + lstrlenW(columns_sql) + lstrlenW(postlude) + 1; create_sql = msi_alloc(size * sizeof(WCHAR)); if (!create_sql) - return ERROR_OUTOFMEMORY; + goto done; lstrcpyW(create_sql, prelude); lstrcatW(create_sql, columns_sql); lstrcatW(create_sql, postlude); - msi_free(prelude); - msi_free(columns_sql); - msi_free(postlude); - r = MSI_DatabaseOpenViewW( db, create_sql, &view ); - msi_free(create_sql); - if (r != ERROR_SUCCESS) - return r; + goto done; r = MSI_ViewExecute(view, NULL); MSI_ViewClose(view); msiobj_release(&view->hdr); +done: + msi_free(prelude); + msi_free(columns_sql); + msi_free(postlude); + msi_free(create_sql); return r; }

1 0

James Hawkins : msi: Simplify ITERATE_WriteEnvironmentString.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: 1b7238a96983659c23eb5b35322d1427fedd6796 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b7238a96983659c23eb5b353… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:19:19 2009 -0800 msi: Simplify ITERATE_WriteEnvironmentString. Now we only allocate enough memory to exactly match the size of the constructed string. Before, we were leaving some memory uninitialized and overwriting the allocated memory in certain circum. --- dlls/msi/action.c | 68 ++++++++++++++++++++++++++++++---------------------- 1 files changed, 39 insertions(+), 29 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index ef30dda..a5c916e 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -5013,7 +5013,24 @@ static UINT ITERATE_WriteEnvironmentString( MSIRECORD *rec, LPVOID param ) (res == ERROR_SUCCESS && type != REG_SZ && type != REG_EXPAND_SZ)) goto done; - if (res != ERROR_FILE_NOT_FOUND) + if ((res == ERROR_FILE_NOT_FOUND || !(flags & ENV_MOD_MASK))) + { + /* Nothing to do. */ + if (!value) + { + res = ERROR_SUCCESS; + goto done; + } + + size = (lstrlenW(value) + 1) * sizeof(WCHAR); + newval = strdupW(value); + if (!newval) + { + res = ERROR_OUTOFMEMORY; + goto done; + } + } + else { if (flags & ENV_ACT_SETABSENT) { @@ -5038,8 +5055,18 @@ static UINT ITERATE_WriteEnvironmentString( MSIRECORD *rec, LPVOID param ) goto done; } - size += (lstrlenW(value) + 1) * sizeof(WCHAR); - newval = msi_alloc(size); + size = (lstrlenW(data) + 1) * sizeof(WCHAR); + if (flags & ENV_MOD_MASK) + { + DWORD mod_size; + int multiplier = 0; + if (flags & ENV_MOD_APPEND) multiplier++; + if (flags & ENV_MOD_PREFIX) multiplier++; + mod_size = (lstrlenW(value) + 1) * multiplier; + size += mod_size * sizeof(WCHAR); + } + + newval = msi_alloc(size); ptr = newval; if (!newval) { @@ -5047,37 +5074,20 @@ static UINT ITERATE_WriteEnvironmentString( MSIRECORD *rec, LPVOID param ) goto done; } - if (!(flags & ENV_MOD_MASK)) - lstrcpyW(newval, value); - else + if (flags & ENV_MOD_PREFIX) { - if (flags & ENV_MOD_PREFIX) - { - lstrcpyW(newval, value); - lstrcatW(newval, szSemiColon); - ptr = newval + lstrlenW(value) + 1; - } + lstrcpyW(newval, value); + lstrcatW(newval, szSemiColon); + ptr = newval + lstrlenW(value) + 1; + } - lstrcpyW(ptr, data); + lstrcpyW(ptr, data); - if (flags & ENV_MOD_APPEND) - { - lstrcatW(newval, szSemiColon); - lstrcatW(newval, value); - } - } - } - else if (value) - { - size = (lstrlenW(value) + 1) * sizeof(WCHAR); - newval = msi_alloc(size); - if (!newval) + if (flags & ENV_MOD_APPEND) { - res = ERROR_OUTOFMEMORY; - goto done; + lstrcatW(newval, szSemiColon); + lstrcatW(newval, value); } - - lstrcpyW(newval, value); } if (newval)

1 0

James Hawkins : msi: Free the temp buffer if we haven' t assigned it to *appValue.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: 8ed06175a0b36ae0bd509c8e490a5e84e55d9912 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8ed06175a0b36ae0bd509c8e4… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:19:31 2009 -0800 msi: Free the temp buffer if we haven't assigned it to *appValue. --- dlls/msi/appsearch.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/msi/appsearch.c b/dlls/msi/appsearch.c index 95daabb..b99b4f5 100644 --- a/dlls/msi/appsearch.c +++ b/dlls/msi/appsearch.c @@ -739,6 +739,7 @@ static UINT ACTION_RecurseSearchDirectory(MSIPACKAGE *package, LPWSTR *appValue, size_t dirLen = lstrlenW(dir), fileLen = lstrlenW(sig->File); WCHAR subpath[MAX_PATH]; WCHAR *buf; + DWORD len; static const WCHAR starDotStarW[] = { '*','.','*',0 }; @@ -753,7 +754,8 @@ static UINT ACTION_RecurseSearchDirectory(MSIPACKAGE *package, LPWSTR *appValue, * here. Add two because we might need to add a backslash if the dir name * isn't backslash-terminated. */ - buf = msi_alloc( (dirLen + max(fileLen, strlenW(starDotStarW)) + 2) * sizeof(WCHAR)); + len = dirLen + max(fileLen, strlenW(starDotStarW)) + 2; + buf = msi_alloc(len * sizeof(WCHAR)); if (!buf) return ERROR_OUTOFMEMORY; @@ -815,7 +817,7 @@ static UINT ACTION_RecurseSearchDirectory(MSIPACKAGE *package, LPWSTR *appValue, } } - if (!*appValue) + if (*appValue != buf) msi_free(buf); return rc;

1 0

James Hawkins : msi: Free the columns view if check_column_exists fails.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: be1b1605f9421c363ea98fc0f8c6f9c9cece91e9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=be1b1605f9421c363ea98fc0f… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:19:03 2009 -0800 msi: Free the columns view if check_column_exists fails. --- dlls/msi/alter.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/msi/alter.c b/dlls/msi/alter.c index a62b13e..93d322b 100644 --- a/dlls/msi/alter.c +++ b/dlls/msi/alter.c @@ -125,7 +125,10 @@ static UINT alter_add_column(MSIALTERVIEW *av) return r; if (check_column_exists(av->db, av->colinfo->table, av->colinfo->column)) + { + columns->ops->delete(columns); return ERROR_BAD_QUERY_SYNTAX; + } r = MSI_OpenQuery(av->db, &view, query, av->colinfo->table, av->colinfo->column); if (r == ERROR_SUCCESS)

1 0

James Hawkins : msi: Fix a leaking record handle.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: 1a4a112d768c4b586c504e44a3777e9d65978a8b URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a4a112d768c4b586c504e44a… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:18:47 2009 -0800 msi: Fix a leaking record handle. --- dlls/msi/tests/db.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c index c909f90..d3e9ad1 100644 --- a/dlls/msi/tests/db.c +++ b/dlls/msi/tests/db.c @@ -922,6 +922,8 @@ static void test_viewmodify(void) r = MsiViewModify(hview, MSIMODIFY_INSERT_TEMPORARY, hrec ); ok(r == ERROR_SUCCESS, "MsiViewModify failed\n"); + r = MsiCloseHandle(hrec); + ok(r == ERROR_SUCCESS, "MsiCloseHandle failed\n"); r = MsiViewClose(hview); ok(r == ERROR_SUCCESS, "MsiViewClose failed\n"); r = MsiCloseHandle(hview);

1 0

James Hawkins : msi: Free the join view if we fail to create the table view .
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: b93c2c354471df6eab642b350cfe15b21f3d82b6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b93c2c354471df6eab642b350… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Dec 15 18:18:40 2009 -0800 msi: Free the join view if we fail to create the table view. --- dlls/msi/join.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/msi/join.c b/dlls/msi/join.c index ef5cede..134c569 100644 --- a/dlls/msi/join.c +++ b/dlls/msi/join.c @@ -353,11 +353,13 @@ UINT JOIN_CreateView( MSIDATABASE *db, MSIVIEW **view, LPWSTR tables ) if( r != ERROR_SUCCESS ) { WARN("can't create table: %s\n", debugstr_w(tables)); + msi_free(table); r = ERROR_BAD_QUERY_SYNTAX; goto end; } - r = table->view->ops->get_dimensions( table->view, NULL, &table->columns ); + r = table->view->ops->get_dimensions( table->view, NULL, + &table->columns ); if( r != ERROR_SUCCESS ) { ERR("can't get table dimensions\n");

1 0

Vincent Povirk : ole32: Add a cache for block chain streams in StorageImpl.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: 393c5af205c5b25a50e0b2d06a5889162286abf2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=393c5af205c5b25a50e0b2d06… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue Dec 15 14:44:49 2009 -0600 ole32: Add a cache for block chain streams in StorageImpl. --- dlls/ole32/storage32.c | 88 +++++++++++++++++++++++++++++++++++++++-------- dlls/ole32/storage32.h | 7 ++++ 2 files changed, 80 insertions(+), 15 deletions(-) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index 90d2434..2a7e9f0 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -2268,6 +2268,63 @@ static HRESULT StorageImpl_BaseReadDirEntry(StorageBaseImpl *base, return StorageImpl_ReadDirEntry(This, index, data); } +static BlockChainStream **StorageImpl_GetFreeBlockChainCacheEntry(StorageImpl* This) +{ + int i; + + for (i=0; i<BLOCKCHAIN_CACHE_SIZE; i++) + { + if (!This->blockChainCache[i]) + { + return &This->blockChainCache[i]; + } + } + + i = This->blockChainToEvict; + + BlockChainStream_Destroy(This->blockChainCache[i]); + This->blockChainCache[i] = NULL; + + This->blockChainToEvict++; + if (This->blockChainToEvict == BLOCKCHAIN_CACHE_SIZE) + This->blockChainToEvict = 0; + + return &This->blockChainCache[i]; +} + +static BlockChainStream **StorageImpl_GetCachedBlockChainStream(StorageImpl *This, + DirRef index) +{ + int i, free_index=-1; + + for (i=0; i<BLOCKCHAIN_CACHE_SIZE; i++) + { + if (!This->blockChainCache[i]) + { + if (free_index == -1) free_index = i; + } + else if (This->blockChainCache[i]->ownerDirEntry == index) + { + return &This->blockChainCache[i]; + } + } + + if (free_index == -1) + { + free_index = This->blockChainToEvict; + + BlockChainStream_Destroy(This->blockChainCache[free_index]); + This->blockChainCache[free_index] = NULL; + + This->blockChainToEvict++; + if (This->blockChainToEvict == BLOCKCHAIN_CACHE_SIZE) + This->blockChainToEvict = 0; + } + + This->blockChainCache[free_index] = BlockChainStream_Construct(This, NULL, index); + return &This->blockChainCache[free_index]; +} + static HRESULT StorageImpl_StreamReadAt(StorageBaseImpl *base, DirRef index, ULARGE_INTEGER offset, ULONG size, void *buffer, ULONG *bytesRead) { @@ -2309,15 +2366,13 @@ static HRESULT StorageImpl_StreamReadAt(StorageBaseImpl *base, DirRef index, } else { - BlockChainStream *stream; + BlockChainStream *stream = NULL; - stream = BlockChainStream_Construct(This, NULL, index); + stream = *StorageImpl_GetCachedBlockChainStream(This, index); if (!stream) return E_OUTOFMEMORY; hr = BlockChainStream_ReadAt(stream, offset, bytesToRead, buffer, bytesRead); - BlockChainStream_Destroy(stream); - return hr; } } @@ -2329,7 +2384,7 @@ static HRESULT StorageImpl_StreamSetSize(StorageBaseImpl *base, DirRef index, DirEntry data; HRESULT hr; SmallBlockChainStream *smallblock=NULL; - BlockChainStream *bigblock=NULL; + BlockChainStream **pbigblock=NULL, *bigblock=NULL; hr = StorageImpl_ReadDirEntry(This, index, &data); if (FAILED(hr)) return hr; @@ -2351,7 +2406,8 @@ static HRESULT StorageImpl_StreamSetSize(StorageBaseImpl *base, DirRef index, } else { - bigblock = BlockChainStream_Construct(This, NULL, index); + pbigblock = StorageImpl_GetCachedBlockChainStream(This, index); + bigblock = *pbigblock; if (!bigblock) return E_OUTOFMEMORY; } } @@ -2362,7 +2418,8 @@ static HRESULT StorageImpl_StreamSetSize(StorageBaseImpl *base, DirRef index, } else { - bigblock = BlockChainStream_Construct(This, NULL, index); + pbigblock = StorageImpl_GetCachedBlockChainStream(This, index); + bigblock = *pbigblock; if (!bigblock) return E_OUTOFMEMORY; } @@ -2375,15 +2432,15 @@ static HRESULT StorageImpl_StreamSetSize(StorageBaseImpl *base, DirRef index, SmallBlockChainStream_Destroy(smallblock); return E_FAIL; } + + pbigblock = StorageImpl_GetFreeBlockChainCacheEntry(This); + *pbigblock = bigblock; } else if (bigblock && newsize.QuadPart < LIMIT_TO_USE_SMALL_BLOCK) { - smallblock = Storage32Impl_BigBlocksToSmallBlocks(This, &bigblock); + smallblock = Storage32Impl_BigBlocksToSmallBlocks(This, pbigblock); if (!smallblock) - { - BlockChainStream_Destroy(bigblock); return E_FAIL; - } } /* Set the size of the block chain. */ @@ -2395,7 +2452,6 @@ static HRESULT StorageImpl_StreamSetSize(StorageBaseImpl *base, DirRef index, else { BlockChainStream_SetSize(bigblock, newsize); - BlockChainStream_Destroy(bigblock); } /* Set the size in the directory entry. */ @@ -2450,13 +2506,11 @@ static HRESULT StorageImpl_StreamWriteAt(StorageBaseImpl *base, DirRef index, { BlockChainStream *stream; - stream = BlockChainStream_Construct(This, NULL, index); + stream = *StorageImpl_GetCachedBlockChainStream(This, index); if (!stream) return E_OUTOFMEMORY; hr = BlockChainStream_WriteAt(stream, offset, size, buffer, bytesWritten); - BlockChainStream_Destroy(stream); - return hr; } } @@ -2741,6 +2795,7 @@ static void StorageImpl_Invalidate(StorageBaseImpl* iface) static void StorageImpl_Destroy(StorageBaseImpl* iface) { StorageImpl *This = (StorageImpl*) iface; + int i; TRACE("(%p)\n", This); StorageImpl_Invalidate(iface); @@ -2751,6 +2806,9 @@ static void StorageImpl_Destroy(StorageBaseImpl* iface) BlockChainStream_Destroy(This->rootBlockChain); BlockChainStream_Destroy(This->smallBlockDepotChain); + for (i=0; i<BLOCKCHAIN_CACHE_SIZE; i++) + BlockChainStream_Destroy(This->blockChainCache[i]); + if (This->bigBlockFile) BIGBLOCKFILE_Destructor(This->bigBlockFile); HeapFree(GetProcessHeap(), 0, This); diff --git a/dlls/ole32/storage32.h b/dlls/ole32/storage32.h index 07a4b33..03328b2 100644 --- a/dlls/ole32/storage32.h +++ b/dlls/ole32/storage32.h @@ -325,6 +325,9 @@ static inline HRESULT StorageBaseImpl_StreamSetSize(StorageBaseImpl *This, void StorageBaseImpl_AddStream(StorageBaseImpl * stg, StgStreamImpl * strm); void StorageBaseImpl_RemoveStream(StorageBaseImpl * stg, StgStreamImpl * strm); +/* Number of BlockChainStream objects to cache in a StorageImpl */ +#define BLOCKCHAIN_CACHE_SIZE 4 + /**************************************************************************** * Storage32Impl definitions. * @@ -367,6 +370,10 @@ struct StorageImpl BlockChainStream* smallBlockDepotChain; BlockChainStream* smallBlockRootChain; + /* Cache of block chain streams objects for directory entries */ + BlockChainStream* blockChainCache[BLOCKCHAIN_CACHE_SIZE]; + UINT blockChainToEvict; + /* * Pointer to the big block file abstraction */

1 0

Huw Davies : ole32: Fix several memory leaks in the tests.
by Alexandre Julliard Dec. 16, 2009

Dec. 16, 2009

Module: wine Branch: master Commit: c6d4df8b1c39c50dfe929a7c485758298a4fc730 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c6d4df8b1c39c50dfe929a7c4… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Dec 15 21:04:42 2009 +0000 ole32: Fix several memory leaks in the tests. Found by Valgrind. --- dlls/ole32/tests/stg_prop.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/ole32/tests/stg_prop.c b/dlls/ole32/tests/stg_prop.c index 8260946..37e8e10 100644 --- a/dlls/ole32/tests/stg_prop.c +++ b/dlls/ole32/tests/stg_prop.c @@ -178,6 +178,7 @@ static void testProps(void) ok(var.vt == VT_LPSTR && !lstrcmpA(U(var).pszVal, val), "Didn't get expected type or value for property (got type %d, value %s)\n", var.vt, U(var).pszVal); + PropVariantClear(&var); /* read clipboard format */ spec.ulKind = PRSPEC_PROPID; @@ -282,6 +283,7 @@ static void testProps(void) ok(var.vt == VT_LPSTR && !lstrcmpA(U(var).pszVal, val), "Didn't get expected type or value for property (got type %d, value %s)\n", var.vt, U(var).pszVal); + PropVariantClear(&var); IPropertyStorage_Release(propertyStorage); IPropertySetStorage_Release(propSetStorage); @@ -364,11 +366,13 @@ static void testCodepage(void) ok(hr == S_OK, "ReadMultiple failed: 0x%08x\n", hr); ok(var.vt == VT_LPSTR && !strcmp(U(var).pszVal, "hi"), "Didn't get expected type or value for property\n"); + PropVariantClear(&var); /* This seemingly non-sensical test is to show that the string is indeed * interpreted according to the current system code page, not according to * the property set's code page. (If the latter were true, the whole * string would be maintained. As it is, only the first character is.) */ + var.vt = VT_LPSTR; U(var).pszVal = (LPSTR)wval; hr = IPropertyStorage_WriteMultiple(propertyStorage, 1, &spec, &var, 0); ok(hr == S_OK, "WriteMultiple failed: 0x%08x\n", hr); @@ -376,6 +380,8 @@ static void testCodepage(void) ok(hr == S_OK, "ReadMultiple failed: 0x%08x\n", hr); ok(var.vt == VT_LPSTR && !strcmp(U(var).pszVal, "h"), "Didn't get expected type or value for property\n"); + PropVariantClear(&var); + /* now that a property's been set, you can't change the code page */ spec.ulKind = PRSPEC_PROPID; U(spec).propid = PID_CODEPAGE;

1 0