wine-commits November 2010

wine-commits@winehq.org

2 participants
866 discussions

Hans Leidekker : sxs: Correct the spec entry for CreateAssemblyCache.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: da831f4bbe2136e530cfc110e7651761d51213f9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=da831f4bbe2136e530cfc110e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:08:26 2010 +0100 sxs: Correct the spec entry for CreateAssemblyCache. --- dlls/sxs/sxs.spec | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/sxs/sxs.spec b/dlls/sxs/sxs.spec index cd32a2e..e199bb1 100644 --- a/dlls/sxs/sxs.spec +++ b/dlls/sxs/sxs.spec @@ -1,2 +1,2 @@ -@ stdcall CreateAssemblyCache(long long) +@ stdcall CreateAssemblyCache(ptr long) @ stub CreateAssemblyNameObject

1 0

Hans Leidekker : include: Add winsxs.idl.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: 58352b9443df584bd1100b337030b18066890674 URL: http://source.winehq.org/git/wine.git/?a=commit;h=58352b9443df584bd1100b337… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:07:18 2010 +0100 include: Add winsxs.idl. --- .gitignore | 1 + include/Makefile.in | 1 + include/winsxs.idl | 101 +++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 103 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index fdfe55b..d1090c1 100644 --- a/.gitignore +++ b/.gitignore @@ -240,6 +240,7 @@ include/wine/itss.h include/wine/svcctl.h include/wine/wined3d.h include/wine/winedxgi.h +include/winsxs.h include/wtypes.h include/wuapi.h include/xmldom.h diff --git a/include/Makefile.in b/include/Makefile.in index d112efa..733e4cc 100644 --- a/include/Makefile.in +++ b/include/Makefile.in @@ -96,6 +96,7 @@ PUBLIC_IDL_H_SRCS = \ wincodec.idl \ wine/itss.idl \ wine/svcctl.idl \ + winsxs.idl \ wtypes.idl \ wuapi.idl \ xmldom.idl \ diff --git a/include/winsxs.idl b/include/winsxs.idl new file mode 100644 index 0000000..c2c38f2 --- /dev/null +++ b/include/winsxs.idl @@ -0,0 +1,101 @@ +/* + * Copyright 2010 Hans Leidekker for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +import "objidl.idl"; + +interface IAssemblyCache; +interface IAssemblyCacheItem; + +typedef struct _FUSION_INSTALL_REFERENCE_ +{ + DWORD cbSize; + DWORD dwFlags; + GUID guidScheme; + LPCWSTR szIdentifier; + LPCWSTR szNonCannonicalData; +} FUSION_INSTALL_REFERENCE, *LPFUSION_INSTALL_REFERENCE; + +typedef struct _ASSEMBLY_INFO +{ + ULONG cbAssemblyInfo; + DWORD dwAssemblyFlags; + ULARGE_INTEGER uliAssemblySizeInKB; + LPWSTR pszCurrentAssemblyPathBuf; + ULONG cchBuf; +} ASSEMBLY_INFO; + +typedef const struct _FUSION_INSTALL_REFERENCE_ *LPCFUSION_INSTALL_REFERENCE; + +[ + object, + uuid(e707dcde-d1cd-11d2-bab9-00c04f8eceae), + pointer_default(unique), + local +] +interface IAssemblyCache : IUnknown +{ + HRESULT UninstallAssembly( + [in] DWORD flags, + [in] LPCWSTR name, + [in] LPCFUSION_INSTALL_REFERENCE ref, + [out, optional] ULONG *disp); + + HRESULT QueryAssemblyInfo( + [in] DWORD flags, + [in] LPCWSTR name, + [in, out] ASSEMBLY_INFO *info); + + HRESULT CreateAssemblyCacheItem( + [in] DWORD flags, + [in] PVOID reserved, + [out] IAssemblyCacheItem **item, + [in, optional] LPCWSTR name); + + HRESULT Reserved( + [out] IUnknown **reserved); + + HRESULT InstallAssembly( + [in] DWORD flags, + [in] LPCWSTR path, + [in] LPCFUSION_INSTALL_REFERENCE ref); +} + +[ + object, + uuid(9e3aaeb4-d1cd-11d2-bab9-00c04f8eceae), + pointer_default(unique), + local +] +interface IAssemblyCacheItem : IUnknown +{ + HRESULT CreateStream( + [in] DWORD flags, + [in] LPCWSTR name, + [in] DWORD format, + [in] DWORD format_flags, + [out] IStream **stream, + [in, optional] ULARGE_INTEGER *max_size); + + HRESULT Commit( + [in] DWORD flags, + [out, optional] ULONG *disp); + + HRESULT AbortItem(); +} + +cpp_quote("HRESULT WINAPI CreateAssemblyCache(IAssemblyCache**,DWORD);")

1 0

Hans Leidekker : msi: Revert 3cd55cc916ad8d74710f435845d8f970161f77fe.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: 5871ad6d00224b025a755ecdb486f77a3d6b6a25 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5871ad6d00224b025a755ecdb… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:06:19 2010 +0100 msi: Revert 3cd55cc916ad8d74710f435845d8f970161f77fe. It breaks the Office 2003 installer. --- dlls/msi/action.c | 4 ---- dlls/msi/tests/package.c | 10 +++++----- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index f56fcf7..2b41b25 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -212,10 +212,6 @@ void msi_feature_set_state( MSIPACKAGE *package, MSIFEATURE *feature, INSTALLSTA feature->ActionRequest = state; feature->Action = state; } - if (feature->Attributes & msidbFeatureAttributesUIDisallowAbsent) - { - feature->Action = INSTALLSTATE_UNKNOWN; - } } void msi_component_set_state( MSIPACKAGE *package, MSICOMPONENT *comp, INSTALLSTATE state ) diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index d19a10a..7e236e4 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -5581,7 +5581,7 @@ static void test_states(void) r = MsiGetFeatureState(hpkg, "eleven", &state, &action); ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); ok( state == INSTALLSTATE_SOURCE, "Expected INSTALLSTATE_SOURCE, got %d\n", state); - ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); + todo_wine ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); state = 0xdeadbee; action = 0xdeadbee; @@ -6534,7 +6534,7 @@ static void test_states(void) r = MsiGetFeatureState(hpkg, "eleven", &state, &action); ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); ok( state == INSTALLSTATE_SOURCE, "Expected INSTALLSTATE_SOURCE, got %d\n", state); - ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); + todo_wine ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); state = 0xdeadbee; action = 0xdeadbee; @@ -7483,7 +7483,7 @@ static void test_states(void) r = MsiGetFeatureState(hpkg, "eleven", &state, &action); ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); ok( state == INSTALLSTATE_SOURCE, "Expected INSTALLSTATE_SOURCE, got %d\n", state); - ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); + todo_wine ok( action == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", action); state = 0xdeadbee; action = 0xdeadbee; @@ -9711,7 +9711,7 @@ static void test_featureparents(void) r = MsiGetFeatureState(hpkg, "lyra", &state, &action); ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); ok( state == INSTALLSTATE_ABSENT, "Expected INSTALLSTATE_ABSENT, got %d\n", state); - todo_wine ok( action == INSTALLSTATE_LOCAL, "Expected INSTALLSTATE_LOCAL, got %d\n", action); + ok( action == INSTALLSTATE_LOCAL, "Expected INSTALLSTATE_LOCAL, got %d\n", action); state = 0xdeadbee; action = 0xdeadbee; @@ -9839,7 +9839,7 @@ static void test_featureparents(void) r = MsiGetFeatureState(hpkg, "lyra", &state, &action); ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); ok( state == INSTALLSTATE_ABSENT, "Expected lyra INSTALLSTATE_ABSENT, got %d\n", state); - todo_wine ok( action == INSTALLSTATE_ABSENT, "Expected lyra INSTALLSTATE_ABSENT, got %d\n", action); + ok( action == INSTALLSTATE_ABSENT, "Expected lyra INSTALLSTATE_ABSENT, got %d\n", action); state = 0xdeadbee; action = 0xdeadbee;

1 0

Hans Leidekker : msi: Downgrade some warnings.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: ee7d8547b6b918bd2aee2c80ef2a091f8b8a02d2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ee7d8547b6b918bd2aee2c80e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:05:22 2010 +0100 msi: Downgrade some warnings. --- dlls/msi/action.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 8a25c1a..f56fcf7 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -2651,7 +2651,7 @@ static void delete_reg_key_or_value( HKEY hkey_root, LPCWSTR key, LPCWSTR value, { if ((res = RegDeleteTreeW( hkey_root, key ))) { - WARN("Failed to delete key %s (%d)\n", debugstr_w(key), res); + TRACE("Failed to delete key %s (%d)\n", debugstr_w(key), res); } return; } @@ -2660,12 +2660,11 @@ static void delete_reg_key_or_value( HKEY hkey_root, LPCWSTR key, LPCWSTR value, { if ((res = RegDeleteValueW( hkey, value ))) { - WARN("Failed to delete value %s (%d)\n", debugstr_w(value), res); + TRACE("Failed to delete value %s (%d)\n", debugstr_w(value), res); } res = RegQueryInfoKeyW( hkey, NULL, NULL, NULL, &num_subkeys, NULL, NULL, &num_values, NULL, NULL, NULL, NULL ); RegCloseKey( hkey ); - if (!res && !num_subkeys && !num_values) { TRACE("Removing empty key %s\n", debugstr_w(key)); @@ -2673,7 +2672,7 @@ static void delete_reg_key_or_value( HKEY hkey_root, LPCWSTR key, LPCWSTR value, } return; } - WARN("Failed to open key %s (%d)\n", debugstr_w(key), res); + TRACE("Failed to open key %s (%d)\n", debugstr_w(key), res); }

1 0

Hans Leidekker : msi: Components don't require a key path.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: 566e67a491306e31e21773252255e9d04b9d7737 URL: http://source.winehq.org/git/wine.git/?a=commit;h=566e67a491306e31e21773252… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:04:49 2010 +0100 msi: Components don't require a key path. --- dlls/msi/action.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 3ebf286..8a25c1a 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -3358,10 +3358,11 @@ static UINT ITERATE_RegisterTypeLibraries(MSIRECORD *row, LPVOID param) } comp->Action = INSTALLSTATE_LOCAL; - file = get_loaded_file( package, comp->KeyPath ); - if (!file) + if (!comp->KeyPath || !(file = get_loaded_file( package, comp->KeyPath ))) + { + TRACE("component has no key path\n"); return ERROR_SUCCESS; - + } ui_actiondata( package, szRegisterTypeLibraries, row ); module = LoadLibraryExW( file->TargetPath, NULL, LOAD_LIBRARY_AS_DATAFILE );

1 0

Hans Leidekker : msi: Don't crash if there is no feature parent.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: aef0aa30fd8ef50cf3915bc94a26fbfb572c1407 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aef0aa30fd8ef50cf3915bc94… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 3 15:04:07 2010 +0100 msi: Don't crash if there is no feature parent. --- dlls/msi/dialog.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c index 1221531..a68d953 100644 --- a/dlls/msi/dialog.c +++ b/dlls/msi/dialog.c @@ -2258,7 +2258,11 @@ msi_seltree_add_child_features( MSIPACKAGE *package, HWND hwnd, LIST_FOR_EACH_ENTRY( feature, &package->features, MSIFEATURE, entry ) { - if ( strcmpW( parent, feature->Feature_Parent ) ) + if ( parent && feature->Feature_Parent && strcmpW( parent, feature->Feature_Parent )) + continue; + else if ( parent && !feature->Feature_Parent ) + continue; + else if ( !parent && feature->Feature_Parent ) continue; if ( !feature->Title )

1 0

Detlef Riekenberg : wer: Add a stub for WerReportSubmit.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: 6f5421631c5b835a4123b32aed07bf55602fc341 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f5421631c5b835a4123b32ae… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Tue Nov 2 21:24:08 2010 +0100 wer: Add a stub for WerReportSubmit. --- dlls/wer/main.c | 28 ++++++++++++++++++++++++++++ dlls/wer/wer.spec | 2 +- 2 files changed, 29 insertions(+), 1 deletions(-) diff --git a/dlls/wer/main.c b/dlls/wer/main.c index d78da3e..585ac91 100644 --- a/dlls/wer/main.c +++ b/dlls/wer/main.c @@ -291,3 +291,31 @@ HRESULT WINAPI WerReportSetParameter(HREPORT hreport, DWORD id, PCWSTR name, PCW return E_NOTIMPL; } + +/*********************************************************************** + * WerReportSubmit (wer.@) + * + * Ask the user for permission and send the error report + * then kill or restart the application, when requested + * + * PARAMS + * hreport [i] error reporting handle to send + * consent [i] current transmit permission + * flags [i] flag to select dialog, transmission snd restart options + * presult [o] ptr, where the transmission result should be saved + * + * RETURNS + * Success: S_OK + * Failure: A HRESULT error code + * + */ +HRESULT WINAPI WerReportSubmit(HREPORT hreport, WER_CONSENT consent, DWORD flags, PWER_SUBMIT_RESULT presult) +{ + FIXME("(%p, %d, 0x%x, %p) :stub\n", hreport, consent, flags, presult); + + if(!presult) + return E_INVALIDARG; + + *presult = WerDisabled; + return E_NOTIMPL; +} diff --git a/dlls/wer/wer.spec b/dlls/wer/wer.spec index 5e6fb52..4303c20 100644 --- a/dlls/wer/wer.spec +++ b/dlls/wer/wer.spec @@ -69,7 +69,7 @@ @ stdcall WerReportCreate(wstr long ptr ptr) @ stdcall WerReportSetParameter(ptr long wstr wstr) @ stub WerReportSetUIOption -@ stub WerReportSubmit +@ stdcall WerReportSubmit(ptr long long ptr) @ stub WerpGetReportConsent @ stub WerpIsDisabled @ stub WerpOpenUserQueue

1 0

Detlef Riekenberg : include: Add WerReportSetParameter + WerReportSubmit.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: d7ed75496806443a15b026093edfa6adde110151 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d7ed75496806443a15b026093… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Mon Nov 1 23:08:48 2010 +0100 include: Add WerReportSetParameter + WerReportSubmit. --- include/werapi.h | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 56 insertions(+), 0 deletions(-) diff --git a/include/werapi.h b/include/werapi.h index a1db799..4b79c04 100644 --- a/include/werapi.h +++ b/include/werapi.h @@ -25,9 +25,48 @@ extern "C" { #endif +/* Only 10 parameter are allowed in WerReportSetParameter */ +#define WER_MAX_PARAM_COUNT 10 +#define WER_P0 0 +#define WER_P1 1 +#define WER_P2 2 +#define WER_P3 3 +#define WER_P4 4 +#define WER_P5 5 +#define WER_P6 6 +#define WER_P7 7 +#define WER_P8 8 +#define WER_P9 9 + +/* Flags for WerReportSubmit */ +#define WER_SUBMIT_HONOR_RECOVERY 0x0001 +#define WER_SUBMIT_HONOR_RESTART 0x0002 +#define WER_SUBMIT_QUEUE 0x0004 +#define WER_SUBMIT_SHOW_DEBUG 0x0008 +#define WER_SUBMIT_ADD_REGISTERED_DATA 0x0010 +#define WER_SUBMIT_OUTOFPROCESS 0x0020 +#define WER_SUBMIT_NO_CLOSE_UI 0x0040 +#define WER_SUBMIT_NO_QUEUE 0x0080 +#define WER_SUBMIT_NO_ARCHIVE 0x0100 +#define WER_SUBMIT_START_MINIMIZED 0x0200 +#define WER_SUBMIT_OUTOFPROCESS_ASYNC 0x0400 +#define WER_SUBMIT_BYPASS_DATA_THROTTLING 0x0800 +#define WER_SUBMIT_ARCHIVE_PARAMETERS_ONLY 0x1000 +#define WER_SUBMIT_REPORT_MACHINE_ID 0x2000 + +/* #### */ typedef HANDLE HREPORT; +typedef enum _WER_CONSENT +{ + WerConsentNotAsked = 1, + WerConsentApproved, + WerConsentDenied, + WerConsentAlwaysPrompt, + WerConsentMax +} WER_CONSENT; + typedef enum _WER_REGISTER_FILE_TYPE { WerRegFileTypeUserDocument = 1, @@ -58,11 +97,28 @@ typedef enum _WER_REPORT_TYPE WerReportInvalid } WER_REPORT_TYPE; +typedef enum _WER_SUBMIT_RESULT +{ + WerReportQueued = 1, + WerReportUploaded, + WerReportDebug, + WerReportFailed, + WerDisabled, + WerReportCancelled, + WerDisabledQueue, + WerReportAsync, + WerCustomAction +} WER_SUBMIT_RESULT, *PWER_SUBMIT_RESULT; + +/* #### */ + HRESULT WINAPI WerAddExcludedApplication(PCWSTR, BOOL); HRESULT WINAPI WerRegisterFile(PCWSTR file, WER_REGISTER_FILE_TYPE regfiletype, DWORD flags); HRESULT WINAPI WerRemoveExcludedApplication(PCWSTR, BOOL); HRESULT WINAPI WerReportCloseHandle(HREPORT); HRESULT WINAPI WerReportCreate(PCWSTR, WER_REPORT_TYPE, PWER_REPORT_INFORMATION, HREPORT*); +HRESULT WINAPI WerReportSetParameter(HREPORT, DWORD, PCWSTR, PCWSTR); +HRESULT WINAPI WerReportSubmit(HREPORT, WER_CONSENT, DWORD, PWER_SUBMIT_RESULT); #ifdef __cplusplus }

1 0

Detlef Riekenberg : wer/tests: Add more tests for Wer*ExcludedApplication.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: 5858af901189c8ff5674e1714c4a9096987b6f44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5858af901189c8ff5674e1714… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Mon Nov 1 23:08:47 2010 +0100 wer/tests: Add more tests for Wer*ExcludedApplication. --- dlls/wer/tests/main.c | 66 ++++++++++++++++++++++++++++++++++++++++--------- 1 files changed, 54 insertions(+), 12 deletions(-) diff --git a/dlls/wer/tests/main.c b/dlls/wer/tests/main.c index f63b7bd..b10a819 100644 --- a/dlls/wer/tests/main.c +++ b/dlls/wer/tests/main.c @@ -31,6 +31,7 @@ #include "wine/test.h" static const WCHAR appcrash[] = {'A','P','P','C','R','A','S','H',0}; +static const WCHAR backslash[] = {'\\',0}; static const WCHAR empty[] = {0}; static const WCHAR winetest_wer[] = {'w','i','n','e','t','e','s','t','_','w','e','r','.','e','x','e',0}; @@ -39,14 +40,12 @@ static const WCHAR winetest_wer[] = {'w','i','n','e','t','e','s','t','_','w','e' static void test_WerAddExcludedApplication(void) { HRESULT hr; + WCHAR buffer[MAX_PATH]; + UINT res; /* clean state */ hr = WerRemoveExcludedApplication(winetest_wer, FALSE); - if (hr == E_NOTIMPL) { - skip("Wer*ExcludedApplication not implemented\n"); - return; - } - ok((hr == S_OK) || (hr == E_FAIL) || __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND), + ok((hr == S_OK) || (hr == E_FAIL) || (hr == __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND)), "got 0x%x (expected S_OK, E_FAIL or HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND))\n", hr); hr = WerAddExcludedApplication(NULL, FALSE); @@ -65,6 +64,27 @@ static void test_WerAddExcludedApplication(void) /* cleanup */ hr = WerRemoveExcludedApplication(winetest_wer, FALSE); ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + + /* appname has a path */ + res = GetWindowsDirectoryW(buffer, sizeof(buffer) / sizeof(buffer[0])); + if (res > 0) { + /* the last part from the path is added to the inclusion list */ + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + + lstrcatW(buffer, backslash); + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == E_INVALIDARG, "got 0x%x (expected E_INVALIDARG)\n", hr); + + lstrcatW(buffer, winetest_wer); + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + } + } /* #### */ @@ -72,15 +92,12 @@ static void test_WerAddExcludedApplication(void) static void test_WerRemoveExcludedApplication(void) { HRESULT hr; + WCHAR buffer[MAX_PATH]; + UINT res; /* clean state */ hr = WerRemoveExcludedApplication(winetest_wer, FALSE); - if (hr == E_NOTIMPL) { - skip("Wer*ExcludedApplication not implemented\n"); - return; - } - - ok((hr == S_OK) || (hr == E_FAIL) || __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND), + ok((hr == S_OK) || (hr == E_FAIL) || (hr == __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND)), "got 0x%x (expected S_OK, E_FAIL or HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND))\n", hr); hr = WerAddExcludedApplication(winetest_wer, FALSE); @@ -97,9 +114,34 @@ static void test_WerRemoveExcludedApplication(void) /* app not in the list */ hr = WerRemoveExcludedApplication(winetest_wer, FALSE); - ok((hr == E_FAIL) || __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND), + ok((hr == E_FAIL) || (hr == __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND)), "got 0x%x (expected E_FAIL or HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND))\n", hr); + /* appname has a path */ + res = GetWindowsDirectoryW(buffer, sizeof(buffer) / sizeof(buffer[0])); + if (res > 0) { + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok((hr == E_FAIL) || (hr == __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND)), + "got 0x%x (expected E_FAIL or HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND))\n", hr); + + /* the last part from the path is added to the inclusion list */ + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + + lstrcatW(buffer, backslash); + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == E_INVALIDARG, "got 0x%x (expected E_INVALIDARG)\n", hr); + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok(hr == E_INVALIDARG, "got 0x%x (expected E_INVALIDARG)\n", hr); + + lstrcatW(buffer, winetest_wer); + hr = WerAddExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + hr = WerRemoveExcludedApplication(buffer, FALSE); + ok(hr == S_OK, "got 0x%x (expected S_OK)\n", hr); + } } /* #### */

1 0

Detlef Riekenberg : wer: Implement Wer*ExcludedApplication.
by Alexandre Julliard 03 Nov '10

03 Nov '10

Module: wine Branch: master Commit: f47d2118b148418db65f55a9421ceb4079081270 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f47d2118b148418db65f55a94… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Tue Nov 2 21:08:54 2010 +0100 wer: Implement Wer*ExcludedApplication. --- dlls/wer/Makefile.in | 1 + dlls/wer/main.c | 86 +++++++++++++++++++++++++++++++++++++++++++------ 2 files changed, 76 insertions(+), 11 deletions(-) diff --git a/dlls/wer/Makefile.in b/dlls/wer/Makefile.in index e90dcc0..6916c19 100644 --- a/dlls/wer/Makefile.in +++ b/dlls/wer/Makefile.in @@ -1,5 +1,6 @@ MODULE = wer.dll IMPORTLIB = wer +IMPORTS = advapi32 C_SRCS = \ main.c diff --git a/dlls/wer/main.c b/dlls/wer/main.c index 539eac4..d78da3e 100644 --- a/dlls/wer/main.c +++ b/dlls/wer/main.c @@ -1,5 +1,6 @@ /* * Copyright 2010 Louis Lenders + * Copyright 2010 Detlef Riekenberg * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -22,8 +23,10 @@ #include "windef.h" #include "winbase.h" +#include "winreg.h" #include "werapi.h" #include "wine/list.h" +#include "wine/unicode.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(wer); @@ -47,6 +50,11 @@ static CRITICAL_SECTION report_table_cs = { &report_table_cs_debug, -1, 0, 0, 0, static struct list report_table = LIST_INIT(report_table); +static WCHAR regpath_exclude[] = {'S','o','f','t','w','a','r','e','\\', + 'M','i','c','r','o','s','o','f','t','\\', + 'W','i','n','d','o','w','s',' ','E','r','r','o','r',' ','R','e','p','o','r','t','i','n','g','\\', + 'E','x','c','l','u','d','e','d','A','p','p','l','i','c','a','t','i','o','n','s',0}; + /*********************************************************************** * Memory alloccation helper */ @@ -61,12 +69,6 @@ static inline BOOL heap_free(void *mem) return HeapFree(GetProcessHeap(), 0, mem); } -HRESULT WINAPI WerAddExcludedApplication(PCWSTR exeName, BOOL allUsers) -{ - FIXME("(%s, %d) stub\n",debugstr_w(exeName), allUsers); - return E_NOTIMPL; -} - BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) { TRACE("(0x%p, %d, %p)\n", hinstDLL, fdwReason, lpvReserved); @@ -86,6 +88,47 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) } /*********************************************************************** + * WerAddExcludedApplication (wer.@) + * + * Add an application to the user specific or the system wide exclusion list + * + * PARAMS + * exeName [i] The application name + * allUsers [i] for all users (TRUE) or for the current user (FALSE) + * + * RETURNS + * Success: S_OK + * Faulure: A HRESULT error code + * + */ +HRESULT WINAPI WerAddExcludedApplication(PCWSTR exeName, BOOL allUsers) +{ + HKEY hkey; + DWORD value = 1; + LPWSTR bs; + + TRACE("(%s, %d)\n",debugstr_w(exeName), allUsers); + if (!exeName || !exeName[0]) + return E_INVALIDARG; + + bs = strrchrW(exeName, '\\'); + if (bs) { + bs++; /* skip the backslash */ + if (!bs[0]) { + return E_INVALIDARG; + } + } else + bs = (LPWSTR) exeName; + + if (!RegCreateKeyW(allUsers ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER, regpath_exclude, &hkey)) { + RegSetValueExW(hkey, bs, 0, REG_DWORD, (LPBYTE)&value, sizeof(DWORD)); + RegCloseKey(hkey); + return S_OK; + } + return E_ACCESSDENIED; +} + +/*********************************************************************** * WerRemoveExcludedApplication (wer.@) * * remove an application from the exclusion list @@ -94,15 +137,36 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) * exeName [i] The application name * allUsers [i] for all users (TRUE) or for the current user (FALSE) * - * RESULTS - * SUCCESS S_OK - * FAILURE A HRESULT error code + * RETURNS + * Success: S_OK + * Faulure: A HRESULT error code * */ HRESULT WINAPI WerRemoveExcludedApplication(PCWSTR exeName, BOOL allUsers) { - FIXME("(%s, %d) :stub\n",debugstr_w(exeName), allUsers); - return E_NOTIMPL; + HKEY hkey; + LPWSTR bs; + LONG lres; + + TRACE("(%s, %d)\n",debugstr_w(exeName), allUsers); + if (!exeName || !exeName[0]) + return E_INVALIDARG; + + bs = strrchrW(exeName, '\\'); + if (bs) { + bs++; /* skip the backslash */ + if (!bs[0]) { + return E_INVALIDARG; + } + } else + bs = (LPWSTR) exeName; + + if (!RegCreateKeyW(allUsers ? HKEY_LOCAL_MACHINE : HKEY_CURRENT_USER, regpath_exclude, &hkey)) { + lres = RegDeleteValueW(hkey, bs); + RegCloseKey(hkey); + return lres ? __HRESULT_FROM_WIN32(ERROR_ENVVAR_NOT_FOUND) : S_OK; + } + return E_ACCESSDENIED; } /***********************************************************************

1 0

← Newer
1
...
67
68
69
70
71
72
73
...
87
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 2010