March 2019 - Wine-devel - list.winehq.org

[PATCH] mferror: Add error messages dll.
by Nikolay Sivov 15 Mar '19

15 Mar '19

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- Some applications try to load this. configure | 2 + configure.ac | 1 + dlls/mferror/Makefile.in | 4 + dlls/mferror/mferror.mc | 542 ++++++++++++++++++++++++++++++++++++++ dlls/mferror/mferror.spec | 1 + include/mferror.h | 2 + 6 files changed, 552 insertions(+) create mode 100644 dlls/mferror/Makefile.in create mode 100644 dlls/mferror/mferror.mc create mode 100644 dlls/mferror/mferror.spec diff --git a/configure b/configure index 4597eae4d1..0885b4979b 100755 --- a/configure +++ b/configure @@ -1374,6 +1374,7 @@ enable_mciseq enable_mciwave enable_mf enable_mf3216 +enable_mferror enable_mfplat enable_mfplay enable_mfreadwrite @@ -19672,6 +19673,7 @@ wine_fn_config_makefile dlls/mciwave enable_mciwave wine_fn_config_makefile dlls/mf enable_mf wine_fn_config_makefile dlls/mf/tests enable_tests wine_fn_config_makefile dlls/mf3216 enable_mf3216 +wine_fn_config_makefile dlls/mferror enable_mferror wine_fn_config_makefile dlls/mfplat enable_mfplat wine_fn_config_makefile dlls/mfplat/tests enable_tests wine_fn_config_makefile dlls/mfplay enable_mfplay diff --git a/configure.ac b/configure.ac index 744bf61ec9..d5640ed0e9 100644 --- a/configure.ac +++ b/configure.ac @@ -3394,6 +3394,7 @@ WINE_CONFIG_MAKEFILE(dlls/mciwave) WINE_CONFIG_MAKEFILE(dlls/mf) WINE_CONFIG_MAKEFILE(dlls/mf/tests) WINE_CONFIG_MAKEFILE(dlls/mf3216) +WINE_CONFIG_MAKEFILE(dlls/mferror) WINE_CONFIG_MAKEFILE(dlls/mfplat) WINE_CONFIG_MAKEFILE(dlls/mfplat/tests) WINE_CONFIG_MAKEFILE(dlls/mfplay) diff --git a/dlls/mferror/Makefile.in b/dlls/mferror/Makefile.in new file mode 100644 index 0000000000..25cbdf6172 --- /dev/null +++ b/dlls/mferror/Makefile.in @@ -0,0 +1,4 @@ +MODULE = mferror.dll + +MC_SRCS = \ + mferror.mc diff --git a/dlls/mferror/mferror.mc b/dlls/mferror/mferror.mc new file mode 100644 index 0000000000..89672c9c31 --- /dev/null +++ b/dlls/mferror/mferror.mc @@ -0,0 +1,542 @@ +; +; Copyright 2019 Nikolay Sivov 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 +; + +LanguageNames=(ENU=0x409:mferror) + +SeverityNames=(Error=0x3:STATUS_SEVERITY_ERROR +Success=0x0:STATUS_SEVERITY_SUCCESS) +FacilityNames=(MF=0xd:FACILITY_MF) + +MessageId=14000 +Severity=Error +Facility=MF +SymbolicName=MF_E_PLATFORM_NOT_INITIALIZED +Language=ENU +Media Foundation platform is not initialized. +. +MessageId=14001 +Severity=Error +Facility=MF +SymbolicName=MF_E_BUFFERTOOSMALL +Language=ENU +Buffer is too small. +. +MessageId=14002 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDREQUEST +Language=ENU +Invalid request. +. +MessageId=14003 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDSTREAMNUMBER +Language=ENU +Invalid stream number. +. +MessageId=14004 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDMEDIATYPE +Language=ENU +Invalid media type. +. +MessageId=14005 +Severity=Error +Facility=MF +SymbolicName=MF_E_NOTACCEPTING +Language=ENU +No more input is accepted. +. +MessageId=14006 +Severity=Error +Facility=MF +SymbolicName=MF_E_NOT_INITIALIZED +Language=ENU +Object is not initialized. +. +MessageId=14007 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_REPRESENTATION +Language=ENU +Representation is not supported. +. +MessageId=14009 +Severity=Error +Facility=MF +SymbolicName=MF_E_NO_MORE_TYPES +Language=ENU +No more types in the list of suggested media types. +. +MessageId=14010 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_SERVICE +Language=ENU +Unsupported service. +. +MessageId=14011 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNEXPECTED +Language=ENU +Unexpected error. +. +MessageId=14012 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDNAME +Language=ENU +Invalid name. +. +MessageId=14013 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDTYPE +Language=ENU +Invalid type. +. +MessageId=14014 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_FILE_FORMAT +Language=ENU +Invalid file format. +. +MessageId=14015 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALIDINDEX +Language=ENU +Invalid index. +. +MessageId=14016 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_TIMESTAMP +Language=ENU +Invalid timestamp. +. +MessageId=14019 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_SCHEME +Language=ENU +Unsupported scheme. +. +MessageId=14020 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_BYTESTREAM_TYPE +Language=ENU +Unsupported bytestream type. +. +MessageId=14021 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_TIME_FORMAT +Language=ENU +Unsupported time format. +. +MessageId=14024 +Severity=Error +Facility=MF +SymbolicName=MF_E_NO_SAMPLE_TIMESTAMP +Language=ENU +Timestamp is not set for the sample. +. +MessageId=14025 +Severity=Error +Facility=MF +SymbolicName=MF_E_NO_SAMPLE_DURATION +Language=ENU +No duration set for the sample. +. +MessageId=14027 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_STREAM_DATA +Language=ENU +Invalid stream data. +. +MessageId=14031 +Severity=Error +Facility=MF +SymbolicName=MF_E_RT_UNAVAILABLE +Language=ENU +Realtime support is not available. +. +MessageId=14032 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_RATE +Language=ENU +Unsupported rate. +. +MessageId=14033 +Severity=Error +Facility=MF +SymbolicName=MF_E_THINNING_UNSUPPORTED +Language=ENU +Unsupported thinning. +. +MessageId=14034 +Severity=Error +Facility=MF +SymbolicName=MF_E_REVERSE_UNSUPPORTED +Language=ENU +Reversing is not supported. +. +MessageId=14035 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_RATE_TRANSITION +Language=ENU +Unsupported rate transition. +. +MessageId=14036 +Severity=Error +Facility=MF +SymbolicName=MF_E_RATE_CHANGE_PREEMPTED +Language=ENU +Rate change was preempted. +. +MessageId=14037 +Severity=Error +Facility=MF +SymbolicName=MF_E_NOT_FOUND +Language=ENU +Object or value wasn't found. +. +MessageId=14038 +Severity=Error +Facility=MF +SymbolicName=MF_E_NOT_AVAILABLE +Language=ENU +Value is not available. +. +MessageId=14039 +Severity=Error +Facility=MF +SymbolicName=MF_E_NO_CLOCK +Language=ENU +Clock is not available. +. +MessageId=14040 +Severity=Success +Facility=MF +SymbolicName=MF_S_MULTIPLE_BEGIN +Language=ENU +Begin request has already been made. +. +MessageId=14041 +Severity=Error +Facility=MF +SymbolicName=MF_E_MULTIPLE_BEGIN +Language=ENU +Begin request has already been made. +. +MessageId=14042 +Severity=Error +Facility=MF +SymbolicName=MF_E_MULTIPLE_SUBSCRIBERS +Language=ENU +Multiple subscribers are not supported. +. +MessageId=14043 +Severity=Error +Facility=MF +SymbolicName=MF_E_TIMER_ORPHANED +Language=ENU +The timer was orphaned. +. +MessageId=14044 +Severity=Error +Facility=MF +SymbolicName=MF_E_STATE_TRANSITION_PENDING +Language=ENU +State transition is pending. +. +MessageId=14045 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_STATE_TRANSITION +Language=ENU +Unsupported state transition. +. +MessageId=14046 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNRECOVERABLE_ERROR_OCCURRED +Language=ENU +Unrecoverable error occurred. +. +MessageId=14047 +Severity=Error +Facility=MF +SymbolicName=MF_E_SAMPLE_HAS_TOO_MANY_BUFFERS +Language=ENU +Sample has too many buffers. +. +MessageId=14048 +Severity=Error +Facility=MF +SymbolicName=MF_E_SAMPLE_NOT_WRITABLE +Language=ENU +Sample is not writable. +. +MessageId=14050 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_KEY +Language=ENU +Key is invalid. +. +MessageId=14051 +Severity=Error +Facility=MF +SymbolicName=MF_E_BAD_STARTUP_VERSION +Language=ENU +Bad startup version. +. +MessageId=14052 +Severity=Error +Facility=MF +SymbolicName=MF_E_UNSUPPORTED_CAPTION +Language=ENU +Unsupported caption. +. +MessageId=14053 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_POSITION +Language=ENU +Invalid position. +. +MessageId=14054 +Severity=Error +Facility=MF +SymbolicName=MF_E_ATTRIBUTENOTFOUND +Language=ENU +Attribute is not found. +. +MessageId=14055 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_TYPE_NOT_ALLOWED +Language=ENU +Property type is not allowed. +. +MessageId=14056 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_TYPE_NOT_SUPPORTED +Language=ENU +Property type is not supported. +. +MessageId=14057 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_EMPTY +Language=ENU +Property is empty. +. +MessageId=14058 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_NOT_EMPTY +Language=ENU +Property is not empty. +. +MessageId=14059 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_VECTOR_NOT_ALLOWED +Language=ENU +Vector property is not allowed. +. +MessageId=14060 +Severity=Error +Facility=MF +SymbolicName=MF_E_PROPERTY_VECTOR_REQUIRED +Language=ENU +Vector property is required. +. +MessageId=14061 +Severity=Error +Facility=MF +SymbolicName=MF_E_OPERATION_CANCELLED +Language=ENU +Operation was cancelled. +. +MessageId=14062 +Severity=Error +Facility=MF +SymbolicName=MF_E_BYTESTREAM_NOT_SEEKABLE +Language=ENU +Bytestream is not seekable. +. +MessageId=14063 +Severity=Error +Facility=MF +SymbolicName=MF_E_DISABLED_IN_SAFEMODE +Language=ENU +Platform is disabled in safe mode. +. +MessageId=14064 +Severity=Error +Facility=MF +SymbolicName=MF_E_CANNOT_PARSE_BYTESTREAM +Language=ENU +Cannot parse bytestream. +. +MessageId=14065 +Severity=Error +Facility=MF +SymbolicName=MF_E_SOURCERESOLVER_MUTUALLY_EXCLUSIVE_FLAGS +Language=ENU +Mutually exclusive flags passed to source resolver. +. +MessageId=14075 +Severity=Error +Facility=MF +SymbolicName=MF_E_BYTESTREAM_UNKNOWN_LENGTH +Language=ENU +Unknown bytestream length. +. +MessageId=14079 +Severity=Error +Facility=MF +SymbolicName=MF_E_INVALID_WORKQUEUE +Language=ENU +Invalid work queue index. +. +MessageId=16000 +Severity=Error +Facility=MF +SymbolicName=MF_E_NO_EVENTS_AVAILABLE +Language=ENU +No events available. +. +MessageId=16005 +Severity=Error +Facility=MF +SymbolicName=MF_E_SHUTDOWN +Language=ENU +Shutdown() was called. +. +MessageId=21006 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_INVALID_OPTIONAL_NODE +Language=ENU +Optional node is invalid. +. +MessageId=21009 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_CANNOT_FIND_DECRYPTOR +Language=ENU +Cannot find decryptor. +. +MessageId=21010 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_CODEC_NOT_FOUND +Language=ENU +Codec was not found. +. +MessageId=21011 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_CANNOT_CONNECT +Language=ENU +Cannot connect topology nodes. +. +MessageId=21012 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_UNSUPPORTED +Language=ENU +Topology request is not supported. +. +MessageId=21013 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_INVALID_TIME_ATTRIBUTES +Language=ENU +Invalid topology time attributes. +. +MessageId=21014 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_LOOPS_IN_TOPOLOGY +Language=ENU +Found loops in topology. +. +MessageId=21015 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_MISSING_PRESENTATION_DESCRIPTOR +Language=ENU +Presentation descriptor is missing. +. +MessageId=21016 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_MISSING_STREAM_DESCRIPTOR +Language=ENU +Stream descriptor is missing. +. +MessageId=21017 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_STREAM_DESCRIPTOR_NOT_SELECTED +Language=ENU +Stream descriptor is not selected. +. +MessageId=21018 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_MISSING_SOURCE +Language=ENU +Source is missing. +. +MessageId=21019 +Severity=Error +Facility=MF +SymbolicName=MF_E_TOPO_SINK_ACTIVATES_UNSUPPORTED +Language=ENU +Topology loader does not support sink activates. +. +MessageId=40001 +Severity=Error +Facility=MF +SymbolicName=MF_E_CLOCK_NO_TIME_SOURCE +Language=ENU +Clock has no time source set. +. +MessageId=40002 +Severity=Error +Facility=MF +SymbolicName=MF_E_CLOCK_STATE_ALREADY_SET +Language=ENU +Clock state was already set. +. diff --git a/dlls/mferror/mferror.spec b/dlls/mferror/mferror.spec new file mode 100644 index 0000000000..2792acf360 --- /dev/null +++ b/dlls/mferror/mferror.spec @@ -0,0 +1 @@ +# This is resource dll diff --git a/include/mferror.h b/include/mferror.h index 17a006c96d..46d25b6a31 100644 --- a/include/mferror.h +++ b/include/mferror.h @@ -19,6 +19,8 @@ #ifndef __WINE_MFERROR_H #define __WINE_MFERROR_H +#define FACILITY_MF 0xd + #define MF_E_PLATFORM_NOT_INITIALIZED _HRESULT_TYPEDEF_(0xc00d36b0) #define MF_E_BUFFERTOOSMALL _HRESULT_TYPEDEF_(0xc00d36b1) #define MF_E_INVALIDREQUEST _HRESULT_TYPEDEF_(0xc00d36b2) -- 2.20.1

2 1

[v2 PATCH 1/8] propsys: Add support for VT_CLSID in PropVariantCompareEx().
by Nikolay Sivov 15 Mar '19

15 Mar '19

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- v2: should fix test failures on wine. dlls/propsys/propvar.c | 9 ++++++++- dlls/propsys/tests/propsys.c | 35 +++++++++++++++++++++++++++++++++++ 2 files changed, 43 insertions(+), 1 deletion(-) diff --git a/dlls/propsys/propvar.c b/dlls/propsys/propvar.c index c56a737ede..1b2cb157a4 100644 --- a/dlls/propsys/propvar.c +++ b/dlls/propsys/propvar.c @@ -807,6 +807,9 @@ static BOOL isemptyornull(const PROPVARIANT *propvar) } return i == propvar->u.parray->cDims; } + if (propvar->vt == VT_CLSID) + return !propvar->u.puuid; + /* FIXME: vectors, byrefs, errors? */ return FALSE; } @@ -893,8 +896,12 @@ INT WINAPI PropVariantCompareEx(REFPROPVARIANT propvar1, REFPROPVARIANT propvar2 else res = lstrcmpA(propvar1->u.pszVal, propvar2_converted->u.pszVal); break; + case VT_CLSID: + res = memcmp(propvar1->u.puuid, propvar2->u.puuid, sizeof(*propvar1->u.puuid)); + if (res) res = res > 0 ? 1 : -1; + break; default: - FIXME("vartype %d not handled\n", propvar1->vt); + FIXME("vartype %#x not handled\n", propvar1->vt); res = -1; break; } diff --git a/dlls/propsys/tests/propsys.c b/dlls/propsys/tests/propsys.c index c47729ea97..fab2d5e89e 100644 --- a/dlls/propsys/tests/propsys.c +++ b/dlls/propsys/tests/propsys.c @@ -660,6 +660,7 @@ static void test_PropVariantToStringAlloc(void) static void test_PropVariantCompare(void) { PROPVARIANT empty, null, emptyarray, i2_0, i2_2, i4_large, i4_largeneg, i4_2, str_2, str_02, str_b; + PROPVARIANT clsid_null, clsid, clsid2; INT res; static const WCHAR str_2W[] = {'2', 0}; static const WCHAR str_02W[] = {'0', '2', 0}; @@ -704,6 +705,12 @@ static void test_PropVariantCompare(void) str_02.u.bstrVal = SysAllocString(str_02W); str_b.vt = VT_BSTR; str_b.u.bstrVal = SysAllocString(str_bW); + clsid_null.vt = VT_CLSID; + clsid_null.u.puuid = NULL; + clsid.vt = VT_CLSID; + clsid.u.puuid = (GUID *)&dummy_guid; + clsid2.vt = VT_CLSID; + clsid2.u.puuid = (GUID *)&GUID_NULL; res = PropVariantCompareEx(&empty, &empty, 0, 0); ok(res == 0, "res=%i\n", res); @@ -772,6 +779,34 @@ static void test_PropVariantCompare(void) res = PropVariantCompareEx(&i4_large, &str_b, 0, 0); todo_wine ok(res == -5 /* ??? */, "res=%i\n", res); + /* VT_CLSID */ + res = PropVariantCompareEx(&clsid_null, &clsid_null, 0, 0); + ok(res == 0, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid_null, &clsid_null, 0, PVCF_TREATEMPTYASGREATERTHAN); + ok(res == 0, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid, &clsid, 0, 0); + ok(res == 0, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid, &clsid2, 0, 0); + ok(res == 1, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid2, &clsid, 0, 0); + ok(res == -1, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid_null, &clsid, 0, 0); + ok(res == -1, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid, &clsid_null, 0, 0); + ok(res == 1, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid_null, &clsid, 0, PVCF_TREATEMPTYASGREATERTHAN); + ok(res == 1, "res=%i\n", res); + + res = PropVariantCompareEx(&clsid, &clsid_null, 0, PVCF_TREATEMPTYASGREATERTHAN); + ok(res == -1, "res=%i\n", res); + SysFreeString(str_2.u.bstrVal); SysFreeString(str_02.u.bstrVal); SysFreeString(str_b.u.bstrVal); -- 2.20.1

2 15

[PATCH 9/9] msidb: Add support for wildcard (full database) export.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msidb allows a full database export with the special "*" database name. Note: It does not support true wildcards, just this special indication that you wish to export all the tables in the database. v2: Use Unicode functions, ARRAY_SIZE macro. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- programs/msidb/main.c | 82 +++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 80 insertions(+), 2 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index 3916e0d721..4bd065abe0 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -439,7 +439,7 @@ static int import_table( struct msidb_state *state, const WCHAR *table_name ) WCHAR table_path[MAX_PATH]; UINT ret; - snprintfW( table_path, sizeof(table_path)/sizeof(WCHAR), format, table_name ); + snprintfW( table_path, ARRAY_SIZE(table_path), format, table_name ); ret = MsiDatabaseImportW( state->database_handle, state->table_folder, table_path ); if (ret != ERROR_SUCCESS) { @@ -469,7 +469,7 @@ static int export_table( struct msidb_state *state, const WCHAR *table_name ) WCHAR table_path[MAX_PATH]; UINT ret; - snprintfW( table_path, sizeof(table_path)/sizeof(WCHAR), format, table_name ); + snprintfW( table_path, ARRAY_SIZE(table_path), format, table_name ); ret = MsiDatabaseExportW( state->database_handle, table_name, state->table_folder, table_path ); if (ret != ERROR_SUCCESS) { @@ -479,12 +479,90 @@ static int export_table( struct msidb_state *state, const WCHAR *table_name ) return 1; } +static int export_all_tables( struct msidb_state *state ) +{ + static const WCHAR summary_information[] = + {'_','S','u','m','m','a','r','y','I','n','f','o','r','m','a','t','i','o','n',0}; + static const WCHAR query_command[] = + {'S','E','L','E','C','T',' ','N','a','m','e',' ','F','R','O','M',' ','_','T','a','b','l','e','s',0}; + MSIHANDLE view = 0; + UINT ret; + + ret = MsiDatabaseOpenViewW( state->database_handle, query_command, &view ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to open _Tables table.\n" ); + goto cleanup; + } + ret = MsiViewExecute( view, 0 ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to query list from _Tables table.\n" ); + goto cleanup; + } + while( 1 ) + { + MSIHANDLE record = 0; + WCHAR table[256]; + DWORD size; + + ret = MsiViewFetch( view, &record ); + if (ret == ERROR_NO_MORE_ITEMS) + break; + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to query row from _Tables table.\n" ); + goto cleanup; + } + size = ARRAY_SIZE(table); + ret = MsiRecordGetStringW( record, 1, table, &size ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to retrieve name string.\n" ); + goto cleanup; + } + if (!export_table( state, table )) + { + ret = ERROR_FUNCTION_FAILED; + goto cleanup; + } + ret = MsiCloseHandle( record ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to close record handle.\n" ); + goto cleanup; + } + } + ret = ERROR_SUCCESS; + /* the _SummaryInformation table is not listed in _Tables */ + if (!export_table( state, summary_information )) + { + ret = ERROR_FUNCTION_FAILED; + goto cleanup; + } + +cleanup: + if (view && MsiViewClose( view ) != ERROR_SUCCESS) + { + ERR( "Failed to close _Streams table.\n" ); + return 0; + } + return (ret == ERROR_SUCCESS); +} + static int export_tables( struct msidb_state *state ) { + const WCHAR wildcard[] = { '*',0 }; struct msidb_listentry *data; LIST_FOR_EACH_ENTRY( data, &state->table_list, struct msidb_listentry, entry ) { + if (strcmpW( data->name, wildcard ) == 0) + { + if (!export_all_tables( state )) + return 0; /* failed, do not commit changes */ + continue; + } if (!export_table( state, data->name )) return 0; /* failed, do not commit changes */ } -- 2.20.1

1 0

[PATCH 8/9] msidb: Add support for exporting with short (DOS) filenames.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msidb's "-s" flag modifies the database export to use short (DOS) filenames instead of full table names. This flag is convenient because it uses the same filenames that the import (-i) option expects. For example, the InstallExecuteSequence table gets imported (or exported with this flag) as InstallE.idt where the normal export option uses InstallExecuteSequence.idt. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- programs/msidb/main.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index 1cb16df788..3916e0d721 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -48,6 +48,7 @@ struct msidb_state BOOL create_database; BOOL import_tables; BOOL export_tables; + BOOL short_filenames; struct list add_stream_list; struct list extract_stream_list; struct list kill_stream_list; @@ -92,6 +93,7 @@ static void show_usage( void ) " -f folder Folder in which to open/save the tables.\n" " -i Import tables into database.\n" " -k file.cab Kill (remove) stream/cabinet file from _Streams table.\n" + " -s Export with short filenames (eight character max).\n" " -x file.cab Extract stream/cabinet file from _Streams table.\n" ); } @@ -163,6 +165,9 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * state->kill_streams = TRUE; list_append( &state->kill_stream_list, argv[i + 1] ); return 2; + case 's': + state->short_filenames = TRUE; + return 1; case 'x': if (i + 1 >= argc) return 0; state->extract_streams = TRUE; @@ -458,7 +463,9 @@ static int import_tables( struct msidb_state *state ) static int export_table( struct msidb_state *state, const WCHAR *table_name ) { - const WCHAR format[] = { '%','s','.','i','d','t',0 }; + const WCHAR format_dos[] = { '%','.','8','s','.','i','d','t',0 }; /* truncate to 8 characters */ + const WCHAR format_full[] = { '%','s','.','i','d','t',0 }; + const WCHAR *format = (state->short_filenames ? format_dos : format_full); WCHAR table_path[MAX_PATH]; UINT ret; -- 2.20.1

1 0

[PATCH 7/9] msidb: Add support for exporting database tables.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msidb allows developers to export tables from a database with the "-e" mode flag followed by a list of tables. For example, this call would export three tables to the current directory: msidb -d package.msi -f . -e ActionText Component InstallExecuteSequence Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- programs/msidb/main.c | 41 ++++++++++++++++++++++++++++++++++++++--- 1 file changed, 38 insertions(+), 3 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index ee79b5c196..1cb16df788 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -47,6 +47,7 @@ struct msidb_state BOOL kill_streams; BOOL create_database; BOOL import_tables; + BOOL export_tables; struct list add_stream_list; struct list extract_stream_list; struct list kill_stream_list; @@ -87,6 +88,7 @@ static void show_usage( void ) " -a file.cab Add stream/cabinet file to _Streams table.\n" " -c Create database file (instead of opening existing file).\n" " -d package.msi Path to the database file.\n" + " -e Export tables from database.\n" " -f folder Folder in which to open/save the tables.\n" " -i Import tables into database.\n" " -k file.cab Kill (remove) stream/cabinet file from _Streams table.\n" @@ -108,10 +110,10 @@ static int valid_state( struct msidb_state *state ) show_usage(); return 0; } - if (!state->create_database && !state->import_tables && !state->add_streams - && !state->kill_streams && !state->extract_streams) + if (!state->create_database && !state->import_tables && !state->export_tables + && !state->add_streams&& !state->kill_streams && !state->extract_streams) { - ERR( "No mode flag specified (-a, -c, -i, -k, -x).\n" ); + ERR( "No mode flag specified (-a, -c, -e, -i, -k, -x).\n" ); show_usage(); return 0; } @@ -146,6 +148,9 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * if (i + 1 >= argc) return 0; state->database_file = argv[i + 1]; return 2; + case 'e': + state->export_tables = TRUE; + return 1; case 'f': if (i + 1 >= argc) return 0; state->table_folder = argv[i + 1]; @@ -451,6 +456,34 @@ static int import_tables( struct msidb_state *state ) return 1; } +static int export_table( struct msidb_state *state, const WCHAR *table_name ) +{ + const WCHAR format[] = { '%','s','.','i','d','t',0 }; + WCHAR table_path[MAX_PATH]; + UINT ret; + + snprintfW( table_path, sizeof(table_path)/sizeof(WCHAR), format, table_name ); + ret = MsiDatabaseExportW( state->database_handle, table_name, state->table_folder, table_path ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to export table '%s', error %d.\n", wine_dbgstr_w(table_name), ret ); + return 0; + } + return 1; +} + +static int export_tables( struct msidb_state *state ) +{ + struct msidb_listentry *data; + + LIST_FOR_EACH_ENTRY( data, &state->table_list, struct msidb_listentry, entry ) + { + if (!export_table( state, data->name )) + return 0; /* failed, do not commit changes */ + } + return 1; +} + int wmain( int argc, WCHAR *argv[] ) { struct msidb_state state; @@ -479,6 +512,8 @@ int wmain( int argc, WCHAR *argv[] ) } if (state.add_streams && !add_streams( &state )) goto cleanup; /* failed, do not commit changes */ + if (state.export_tables && !export_tables( &state )) + goto cleanup; /* failed, do not commit changes */ if (state.extract_streams && !extract_streams( &state )) goto cleanup; /* failed, do not commit changes */ if (state.import_tables && !import_tables( &state )) -- 2.20.1

1 0

[PATCH 6/9] msidb: Add support for extracting stream/cabinet files from MSI databases.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msidb allows developers to extract "streams" (cabinet files) from a database with the "-x" mode flag followed by the filename for the stream in the database, example: msidb -d package.msi -x cabinet.cab v2: Use Unicode functions. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- programs/msidb/main.c | 113 ++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 109 insertions(+), 4 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index e2e4e77ce1..ee79b5c196 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -43,10 +43,12 @@ struct msidb_state WCHAR *table_folder; MSIHANDLE database_handle; BOOL add_streams; + BOOL extract_streams; BOOL kill_streams; BOOL create_database; BOOL import_tables; struct list add_stream_list; + struct list extract_stream_list; struct list kill_stream_list; struct list table_list; }; @@ -88,6 +90,7 @@ static void show_usage( void ) " -f folder Folder in which to open/save the tables.\n" " -i Import tables into database.\n" " -k file.cab Kill (remove) stream/cabinet file from _Streams table.\n" + " -x file.cab Extract stream/cabinet file from _Streams table.\n" ); } @@ -98,20 +101,22 @@ static int valid_state( struct msidb_state *state ) FIXME( "GUI operation is not currently supported.\n" ); return 0; } - if (state->table_folder == NULL && !state->add_streams && !state->kill_streams) + if (state->table_folder == NULL && !state->add_streams && !state->kill_streams + && !state->extract_streams) { ERR( "No table folder specified (-f option).\n" ); show_usage(); return 0; } if (!state->create_database && !state->import_tables && !state->add_streams - && !state->kill_streams) + && !state->kill_streams && !state->extract_streams) { - ERR( "No mode flag specified (-a, -c, -i, -k).\n" ); + ERR( "No mode flag specified (-a, -c, -i, -k, -x).\n" ); show_usage(); return 0; } - if (list_empty( &state->table_list ) && !state->add_streams && !state->kill_streams) + if (list_empty( &state->table_list ) && !state->add_streams && !state->kill_streams + && !state->extract_streams) { ERR( "No tables specified.\n" ); return 0; @@ -153,6 +158,11 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * state->kill_streams = TRUE; list_append( &state->kill_stream_list, argv[i + 1] ); return 2; + case 'x': + if (i + 1 >= argc) return 0; + state->extract_streams = TRUE; + list_append( &state->extract_stream_list, argv[i + 1] ); + return 2; default: break; } @@ -322,6 +332,97 @@ static int kill_streams( struct msidb_state *state ) return 1; } +static int extract_stream( struct msidb_state *state, const WCHAR *stream_filename ) +{ + static const WCHAR select_command[] = + {'S','E','L','E','C','T',' ','D','a','t','a',' ','F','R','O','M',' ','_','S','t','r','e','a','m','s',' ', + 'W','H','E','R','E',' ','N','a','m','e',' ','=',' ','?',0}; + HANDLE file = INVALID_HANDLE_VALUE; + MSIHANDLE view = 0, record = 0; + DWORD read_size, write_size; + char buffer[1024]; + UINT ret; + + file = CreateFileW( stream_filename, GENERIC_WRITE, FILE_SHARE_READ | FILE_SHARE_WRITE, + NULL, CREATE_NEW, FILE_ATTRIBUTE_NORMAL, NULL ); + if (file == INVALID_HANDLE_VALUE) + { + ret = ERROR_FILE_NOT_FOUND; + ERR( "Failed to open destination file %s.\n", wine_dbgstr_w(stream_filename) ); + goto cleanup; + } + ret = MsiDatabaseOpenViewW( state->database_handle, select_command, &view ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to open _Streams table.\n" ); + goto cleanup; + } + record = MsiCreateRecord( 1 ); + if (record == 0) + { + ERR( "Failed to create MSI record.\n" ); + ret = ERROR_OUTOFMEMORY; + goto cleanup; + } + ret = MsiRecordSetStringW( record, 1, stream_filename ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to add stream filename to MSI record.\n" ); + goto cleanup; + } + ret = MsiViewExecute( view, record ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to query stream from _Streams table.\n" ); + goto cleanup; + } + MsiCloseHandle( record ); + record = 0; + ret = MsiViewFetch( view, &record ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to query row from _Streams table.\n" ); + goto cleanup; + } + read_size = sizeof(buffer); + while (read_size == sizeof(buffer)) + { + ret = MsiRecordReadStream( record, 1, buffer, &read_size ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to read stream from _Streams table.\n" ); + goto cleanup; + } + if (!WriteFile( file, buffer, read_size, &write_size, NULL ) || read_size != write_size) + { + ret = ERROR_WRITE_FAULT; + ERR( "Failed to write stream to destination file.\n" ); + goto cleanup; + } + } + +cleanup: + if (record) + MsiCloseHandle( record ); + if (view) + MsiViewClose( view ); + if (file != INVALID_HANDLE_VALUE) + CloseHandle( file ); + return (ret == ERROR_SUCCESS); +} + +static int extract_streams( struct msidb_state *state ) +{ + struct msidb_listentry *data; + + LIST_FOR_EACH_ENTRY( data, &state->extract_stream_list, struct msidb_listentry, entry ) + { + if (!extract_stream( state, data->name )) + return 0; /* failed, do not commit changes */ + } + return 1; +} + static int import_table( struct msidb_state *state, const WCHAR *table_name ) { const WCHAR format[] = { '%','.','8','s','.','i','d','t',0 }; /* truncate to 8 characters */ @@ -358,6 +459,7 @@ int wmain( int argc, WCHAR *argv[] ) memset( &state, 0x0, sizeof(state) ); list_init( &state.add_stream_list ); + list_init( &state.extract_stream_list ); list_init( &state.kill_stream_list ); list_init( &state.table_list ); /* process and validate all the command line flags */ @@ -377,6 +479,8 @@ int wmain( int argc, WCHAR *argv[] ) } if (state.add_streams && !add_streams( &state )) goto cleanup; /* failed, do not commit changes */ + if (state.extract_streams && !extract_streams( &state )) + goto cleanup; /* failed, do not commit changes */ if (state.import_tables && !import_tables( &state )) goto cleanup; /* failed, do not commit changes */ if (state.kill_streams && !kill_streams( &state )) @@ -386,6 +490,7 @@ int wmain( int argc, WCHAR *argv[] ) cleanup: close_database( &state, ret == 0 ); list_free( &state.add_stream_list ); + list_free( &state.extract_stream_list ); list_free( &state.kill_stream_list ); list_free( &state.table_list ); return ret; -- 2.20.1

1 0

[PATCH 5/9] msidb: Add support for removing stream/cabinet files from MSI databases.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msidb allows developers to remove "streams" (cabinet files) from a database with the "-k" mode flag followed by the filename for the stream in the database, example: msidb -d package.msi -k cabinet.cab v2: Use Unicode functions. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- programs/msidb/main.c | 82 +++++++++++++++++++++++++++++++++++++++---- 1 file changed, 76 insertions(+), 6 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index d8372994a6..e2e4e77ce1 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -43,9 +43,11 @@ struct msidb_state WCHAR *table_folder; MSIHANDLE database_handle; BOOL add_streams; + BOOL kill_streams; BOOL create_database; BOOL import_tables; struct list add_stream_list; + struct list kill_stream_list; struct list table_list; }; @@ -85,6 +87,7 @@ static void show_usage( void ) " -d package.msi Path to the database file.\n" " -f folder Folder in which to open/save the tables.\n" " -i Import tables into database.\n" + " -k file.cab Kill (remove) stream/cabinet file from _Streams table.\n" ); } @@ -95,19 +98,20 @@ static int valid_state( struct msidb_state *state ) FIXME( "GUI operation is not currently supported.\n" ); return 0; } - if (state->table_folder == NULL && !state->add_streams) + if (state->table_folder == NULL && !state->add_streams && !state->kill_streams) { ERR( "No table folder specified (-f option).\n" ); show_usage(); return 0; } - if (!state->create_database && !state->import_tables && !state->add_streams) + if (!state->create_database && !state->import_tables && !state->add_streams + && !state->kill_streams) { - ERR( "No mode flag specified (-a, -c, -i).\n" ); + ERR( "No mode flag specified (-a, -c, -i, -k).\n" ); show_usage(); return 0; } - if (list_empty( &state->table_list ) && !state->add_streams) + if (list_empty( &state->table_list ) && !state->add_streams && !state->kill_streams) { ERR( "No tables specified.\n" ); return 0; @@ -144,6 +148,11 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * case 'i': state->import_tables = TRUE; return 1; + case 'k': + if (i + 1 >= argc) return 0; + state->kill_streams = TRUE; + list_append( &state->kill_stream_list, argv[i + 1] ); + return 2; default: break; } @@ -199,11 +208,13 @@ static const WCHAR *basenameW( const WCHAR *filename ) static int add_stream( struct msidb_state *state, const WCHAR *stream_filename ) { - static const char insert_command[] = "INSERT INTO _Streams (Name, Data) VALUES (?, ?)"; + static const WCHAR insert_command[] = + {'I','N','S','E','R','T',' ','I','N','T','O',' ','_','S','t','r','e','a','m','s',' ', + '(','N','a','m','e',',',' ','D','a','t','a',')',' ','V','A','L','U','E','S',' ','(','?',',',' ','?',')',0}; MSIHANDLE view = 0, record = 0; UINT ret; - ret = MsiDatabaseOpenViewA( state->database_handle, insert_command, &view ); + ret = MsiDatabaseOpenViewW( state->database_handle, insert_command, &view ); if (ret != ERROR_SUCCESS) { ERR( "Failed to open _Streams table.\n" ); @@ -256,6 +267,61 @@ static int add_streams( struct msidb_state *state ) return 1; } +static int kill_stream( struct msidb_state *state, const WCHAR *stream_filename ) +{ + static const WCHAR delete_command[] = + {'D','E','L','E','T','E',' ','F','R','O','M',' ','_','S','t','r','e','a','m','s',' ', + 'W','H','E','R','E',' ','N','a','m','e',' ','=',' ','?',0}; + MSIHANDLE view = 0, record = 0; + UINT ret; + + ret = MsiDatabaseOpenViewW( state->database_handle, delete_command, &view ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to open _Streams table.\n" ); + goto cleanup; + } + record = MsiCreateRecord( 1 ); + if (record == 0) + { + ERR( "Failed to create MSI record.\n" ); + ret = ERROR_OUTOFMEMORY; + goto cleanup; + } + ret = MsiRecordSetStringW( record, 1, stream_filename ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to add stream filename to MSI record.\n" ); + goto cleanup; + } + ret = MsiViewExecute( view, record ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to delete stream from _Streams table.\n" ); + goto cleanup; + } + +cleanup: + if (record) + MsiCloseHandle( record ); + if (view) + MsiViewClose( view ); + + return (ret == ERROR_SUCCESS); +} + +static int kill_streams( struct msidb_state *state ) +{ + struct msidb_listentry *data; + + LIST_FOR_EACH_ENTRY( data, &state->kill_stream_list, struct msidb_listentry, entry ) + { + if (!kill_stream( state, data->name )) + return 0; /* failed, do not commit changes */ + } + return 1; +} + static int import_table( struct msidb_state *state, const WCHAR *table_name ) { const WCHAR format[] = { '%','.','8','s','.','i','d','t',0 }; /* truncate to 8 characters */ @@ -292,6 +358,7 @@ int wmain( int argc, WCHAR *argv[] ) memset( &state, 0x0, sizeof(state) ); list_init( &state.add_stream_list ); + list_init( &state.kill_stream_list ); list_init( &state.table_list ); /* process and validate all the command line flags */ for (i = 1; n && i < argc; i += n) @@ -312,11 +379,14 @@ int wmain( int argc, WCHAR *argv[] ) goto cleanup; /* failed, do not commit changes */ if (state.import_tables && !import_tables( &state )) goto cleanup; /* failed, do not commit changes */ + if (state.kill_streams && !kill_streams( &state )) + goto cleanup; /* failed, do not commit changes */ ret = 0; cleanup: close_database( &state, ret == 0 ); list_free( &state.add_stream_list ); + list_free( &state.kill_stream_list ); list_free( &state.table_list ); return ret; } -- 2.20.1

1 0

[PATCH 4/9] msi: Add support for exporting binary streams (Binary/Icon tables).
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> The Binary and Icon tables store their data in a different format from other tables, one column of the data is stored as a "stream" instead of a normal text field. With this patch those tables can now be exported properly by the MSI export functionality. v2: Avoid fixed size path buffers. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- dlls/msi/database.c | 85 +++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 79 insertions(+), 6 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index f302610771..4537ed5836 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -961,8 +961,68 @@ static UINT msi_export_field( HANDLE handle, MSIRECORD *row, UINT field ) return ret ? ERROR_SUCCESS : ERROR_FUNCTION_FAILED; } -static UINT msi_export_record( HANDLE handle, MSIRECORD *row, UINT start ) +static UINT msi_export_stream( const WCHAR *folder, const WCHAR *table, MSIRECORD *row, UINT field, UINT start ) { + static const WCHAR fmt[] = {'%','s','\\','%','s',0}; + WCHAR stream[MAX_STREAM_NAME_LEN + 1], *path; + DWORD sz, read_size, write_size; + char buffer[1024]; + HANDLE file; + UINT len, r; + + sz = ARRAY_SIZE( stream ); + r = MSI_RecordGetStringW( row, start, stream, &sz ); + if (r != ERROR_SUCCESS) + return r; + + len = (sz + strlenW( folder ) + strlenW( table ) + ARRAY_SIZE( fmt ) + 1) * sizeof(WCHAR); + if (!(path = msi_alloc( len ))) + return ERROR_OUTOFMEMORY; + + len = sprintfW( path, fmt, folder, table ); + if (!CreateDirectoryW( path, NULL ) && GetLastError() != ERROR_ALREADY_EXISTS) + { + msi_free( path ); + return ERROR_FUNCTION_FAILED; + } + + path[len++] = '\\'; + strcpyW( path + len, stream ); + file = CreateFileW( path, GENERIC_WRITE, FILE_SHARE_READ | FILE_SHARE_WRITE, + NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL ); + msi_free( path ); + if (file == INVALID_HANDLE_VALUE) + return ERROR_FUNCTION_FAILED; + + read_size = sizeof(buffer); + while (read_size == sizeof(buffer)) + { + r = MSI_RecordReadStream( row, field, buffer, &read_size ); + if (r != ERROR_SUCCESS) + { + CloseHandle( file ); + return r; + } + if (!WriteFile( file, buffer, read_size, &write_size, NULL ) || read_size != write_size) + { + CloseHandle( file ); + return ERROR_WRITE_FAULT; + } + } + CloseHandle( file ); + return r; +} + +struct row_export_info +{ + HANDLE handle; + const WCHAR *folder; + const WCHAR *table; +}; + +static UINT msi_export_record( struct row_export_info *row_export_info, MSIRECORD *row, UINT start ) +{ + HANDLE handle = row_export_info->handle; UINT i, count, r = ERROR_SUCCESS; const char *sep; DWORD sz; @@ -971,7 +1031,18 @@ static UINT msi_export_record( HANDLE handle, MSIRECORD *row, UINT start ) for (i = start; i <= count; i++) { r = msi_export_field( handle, row, i ); - if (r != ERROR_SUCCESS) + if (r == ERROR_INVALID_PARAMETER) + { + r = msi_export_stream( row_export_info->folder, row_export_info->table, row, i, start ); + if (r != ERROR_SUCCESS) + return r; + + /* exporting a binary stream, repeat the "Name" field */ + r = msi_export_field( handle, row, start ); + if (r != ERROR_SUCCESS) + return r; + } + else if (r != ERROR_SUCCESS) return r; sep = (i < count) ? "\t" : "\r\n"; @@ -1062,11 +1133,13 @@ static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, LPCWSTR folder, r = MSI_OpenQuery( db, &view, query, table ); if (r == ERROR_SUCCESS) { + struct row_export_info row_export_info = { handle, folder, table }; + /* write out row 1, the column names */ r = MSI_ViewGetColumnInfo(view, MSICOLINFO_NAMES, &rec); if (r == ERROR_SUCCESS) { - msi_export_record( handle, rec, 1 ); + msi_export_record( &row_export_info, rec, 1 ); msiobj_release( &rec->hdr ); } @@ -1074,7 +1147,7 @@ static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, LPCWSTR folder, r = MSI_ViewGetColumnInfo(view, MSICOLINFO_TYPES, &rec); if (r == ERROR_SUCCESS) { - msi_export_record( handle, rec, 1 ); + msi_export_record( &row_export_info, rec, 1 ); msiobj_release( &rec->hdr ); } @@ -1083,12 +1156,12 @@ static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, LPCWSTR folder, if (r == ERROR_SUCCESS) { MSI_RecordSetStringW( rec, 0, table ); - msi_export_record( handle, rec, 0 ); + msi_export_record( &row_export_info, rec, 0 ); msiobj_release( &rec->hdr ); } /* write out row 4 onwards, the data */ - r = MSI_IterateRecords( view, 0, msi_export_row, handle ); + r = MSI_IterateRecords( view, 0, msi_export_row, &row_export_info ); msiobj_release( &view->hdr ); } -- 2.20.1

1 0

[PATCH 3/9] msi: Break out field exporting into a separate routine.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> msi_export_record is a lot more complicated than necessary, this patch splits out the field export part of that functionality as msi_export_field. This also needs to be separate for exporting binary stream data (next patch). v2: Fix memory leak on error path. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- dlls/msi/database.c | 74 +++++++++++++++++++++++++++------------------ 1 file changed, 45 insertions(+), 29 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index 589dce510d..f302610771 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -918,50 +918,66 @@ end: return r; } -static UINT msi_export_record( HANDLE handle, MSIRECORD *row, UINT start ) +static UINT msi_export_field( HANDLE handle, MSIRECORD *row, UINT field ) { - UINT i, count, len, r = ERROR_SUCCESS; - const char *sep; char *buffer; - DWORD sz; + BOOL ret; + DWORD sz = 0x100; + UINT r; - len = 0x100; - buffer = msi_alloc( len ); - if ( !buffer ) + buffer = msi_alloc( sz ); + if (!buffer) return ERROR_OUTOFMEMORY; - count = MSI_RecordGetFieldCount( row ); - for ( i=start; i<=count; i++ ) + r = MSI_RecordGetStringA( row, field, buffer, &sz ); + if (r == ERROR_MORE_DATA) { - sz = len; - r = MSI_RecordGetStringA( row, i, buffer, &sz ); - if (r == ERROR_MORE_DATA) + char *tmp; + + sz++; /* leave room for NULL terminator */ + tmp = msi_realloc( buffer, sz ); + if (!tmp) { - char *p = msi_realloc( buffer, sz + 1 ); - if (!p) - break; - len = sz + 1; - buffer = p; + msi_free( buffer ); + return ERROR_OUTOFMEMORY; } - sz = len; - r = MSI_RecordGetStringA( row, i, buffer, &sz ); - if (r != ERROR_SUCCESS) - break; + buffer = tmp; - if (!WriteFile( handle, buffer, sz, &sz, NULL )) + r = MSI_RecordGetStringA( row, field, buffer, &sz ); + if (r != ERROR_SUCCESS) { - r = ERROR_FUNCTION_FAILED; - break; + msi_free( buffer ); + return r; } + } + else if (r != ERROR_SUCCESS) + { + msi_free( buffer ); + return r; + } + + ret = WriteFile( handle, buffer, sz, &sz, NULL ); + msi_free( buffer ); + return ret ? ERROR_SUCCESS : ERROR_FUNCTION_FAILED; +} + +static UINT msi_export_record( HANDLE handle, MSIRECORD *row, UINT start ) +{ + UINT i, count, r = ERROR_SUCCESS; + const char *sep; + DWORD sz; + + count = MSI_RecordGetFieldCount( row ); + for (i = start; i <= count; i++) + { + r = msi_export_field( handle, row, i ); + if (r != ERROR_SUCCESS) + return r; sep = (i < count) ? "\t" : "\r\n"; if (!WriteFile( handle, sep, strlen(sep), &sz, NULL )) - { - r = ERROR_FUNCTION_FAILED; - break; - } + return ERROR_FUNCTION_FAILED; } - msi_free( buffer ); return r; } -- 2.20.1

1 0

[PATCH 2/9] msi: Add support for exporting the _SummaryInformation table.
by Hans Leidekker 15 Mar '19

15 Mar '19

From: "Erich E. Hoover" <erich.e.hoover(a)gmail.com> The _SummaryInformation table stores a lot of important information about an MSI database. This patch adds the ability to export that table since, like _ForceCodepage, it is stored in a different format than the other tables. v2: Avoid lstrlenA calls. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> --- dlls/msi/database.c | 29 +++++++++--- dlls/msi/msipriv.h | 1 + dlls/msi/suminfo.c | 109 ++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 133 insertions(+), 6 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index b8ec39322b..589dce510d 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -974,24 +974,35 @@ static UINT msi_export_forcecodepage( HANDLE handle, UINT codepage ) { static const char fmt[] = "\r\n\r\n%u\t_ForceCodepage\r\n"; char data[sizeof(fmt) + 10]; - DWORD sz; - - sprintf( data, fmt, codepage ); + DWORD sz = sprintf( data, fmt, codepage ); - sz = lstrlenA(data) + 1; if (!WriteFile(handle, data, sz, &sz, NULL)) return ERROR_FUNCTION_FAILED; return ERROR_SUCCESS; } -static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, - LPCWSTR folder, LPCWSTR file ) +static UINT msi_export_summaryinformation( MSIDATABASE *db, HANDLE handle ) +{ + static const char header[] = "PropertyId\tValue\r\n" + "i2\tl255\r\n" + "_SummaryInformation\tPropertyId\r\n"; + DWORD sz = ARRAY_SIZE(header) - 1; + + if (!WriteFile(handle, header, sz, &sz, NULL)) + return ERROR_WRITE_FAULT; + + return msi_export_suminfo( db, handle ); +} + +static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, LPCWSTR folder, LPCWSTR file ) { static const WCHAR query[] = { 's','e','l','e','c','t',' ','*',' ','f','r','o','m',' ','%','s',0 }; static const WCHAR forcecodepage[] = { '_','F','o','r','c','e','C','o','d','e','p','a','g','e',0 }; + static const WCHAR summaryinformation[] = { + '_','S','u','m','m','a','r','y','I','n','f','o','r','m','a','t','i','o','n',0 }; MSIRECORD *rec = NULL; MSIQUERY *view = NULL; LPWSTR filename; @@ -1026,6 +1037,12 @@ static UINT MSI_DatabaseExport( MSIDATABASE *db, LPCWSTR table, goto done; } + if (!strcmpW( table, summaryinformation )) + { + r = msi_export_summaryinformation( db, handle ); + goto done; + } + r = MSI_OpenQuery( db, &view, query, table ); if (r == ERROR_SUCCESS) { diff --git a/dlls/msi/msipriv.h b/dlls/msi/msipriv.h index 4b2b65a13b..39219a646f 100644 --- a/dlls/msi/msipriv.h +++ b/dlls/msi/msipriv.h @@ -961,6 +961,7 @@ extern LPWSTR msi_suminfo_dup_string( MSISUMMARYINFO *si, UINT uiProperty ) DECL extern INT msi_suminfo_get_int32( MSISUMMARYINFO *si, UINT uiProperty ) DECLSPEC_HIDDEN; extern LPWSTR msi_get_suminfo_product( IStorage *stg ) DECLSPEC_HIDDEN; extern UINT msi_add_suminfo( MSIDATABASE *db, LPWSTR **records, int num_records, int num_columns ) DECLSPEC_HIDDEN; +extern UINT msi_export_suminfo( MSIDATABASE *db, HANDLE handle ) DECLSPEC_HIDDEN; extern UINT msi_load_suminfo_properties( MSIPACKAGE *package ) DECLSPEC_HIDDEN; /* undocumented functions */ diff --git a/dlls/msi/suminfo.c b/dlls/msi/suminfo.c index 98f6330d9f..c812ae09a4 100644 --- a/dlls/msi/suminfo.c +++ b/dlls/msi/suminfo.c @@ -23,6 +23,7 @@ #define COBJMACROS #define NONAMELESSUNION +#include "stdio.h" #include "windef.h" #include "winbase.h" #include "winreg.h" @@ -1096,6 +1097,114 @@ end: return r; } +static UINT save_prop( MSISUMMARYINFO *si, HANDLE handle, UINT row ) +{ + static const char fmt_systemtime[] = "%04u/%02u/%02u %02u:%02u:%02u"; + char data[36]; /* largest string: YYYY/MM/DD hh:mm:ss */ + static const char fmt_begin[] = "%u\t"; + static const char data_end[] = "\r\n"; + static const char fmt_int[] = "%u"; + UINT r, data_type, len; + SYSTEMTIME system_time; + FILETIME file_time; + INT int_value; + awstring str; + DWORD sz; + + str.unicode = FALSE; + str.str.a = NULL; + len = 0; + r = get_prop( si, row, &data_type, &int_value, &file_time, &str, &len ); + if (r != ERROR_SUCCESS && r != ERROR_MORE_DATA) + return r; + if (data_type == VT_EMPTY) + return ERROR_SUCCESS; /* property not set */ + sz = sprintf( data, fmt_begin, row ); + if (!WriteFile( handle, data, sz, &sz, NULL )) + return ERROR_WRITE_FAULT; + + switch( data_type ) + { + case VT_I2: + case VT_I4: + sz = sprintf( data, fmt_int, int_value ); + if (!WriteFile( handle, data, sz, &sz, NULL )) + return ERROR_WRITE_FAULT; + break; + case VT_LPSTR: + len++; + if (!(str.str.a = msi_alloc( len ))) + return ERROR_OUTOFMEMORY; + r = get_prop( si, row, NULL, NULL, NULL, &str, &len ); + if (r != ERROR_SUCCESS) + { + msi_free( str.str.a ); + return r; + } + sz = len; + if (!WriteFile( handle, str.str.a, sz, &sz, NULL )) + { + msi_free( str.str.a ); + return ERROR_WRITE_FAULT; + } + msi_free( str.str.a ); + break; + case VT_FILETIME: + if (!FileTimeToSystemTime( &file_time, &system_time )) + return ERROR_FUNCTION_FAILED; + sz = sprintf( data, fmt_systemtime, system_time.wYear, system_time.wMonth, + system_time.wDay, system_time.wHour, system_time.wMinute, + system_time.wSecond ); + if (!WriteFile( handle, data, sz, &sz, NULL )) + return ERROR_WRITE_FAULT; + break; + case VT_EMPTY: + /* cannot reach here, property not set */ + break; + default: + FIXME( "Unknown property variant type\n" ); + return ERROR_FUNCTION_FAILED; + } + + sz = ARRAY_SIZE(data_end) - 1; + if (!WriteFile( handle, data_end, sz, &sz, NULL )) + return ERROR_WRITE_FAULT; + + return ERROR_SUCCESS; +} + +UINT msi_export_suminfo( MSIDATABASE *db, HANDLE handle ) +{ + UINT i, r, num_rows; + MSISUMMARYINFO *si; + + r = msi_get_suminfo( db->storage, 0, &si ); + if (r != ERROR_SUCCESS) + r = msi_get_db_suminfo( db, 0, &si ); + if (r != ERROR_SUCCESS) + return r; + + num_rows = get_property_count( si->property ); + if (!num_rows) + { + msiobj_release( &si->hdr ); + return ERROR_FUNCTION_FAILED; + } + + for (i = 0; i < num_rows; i++) + { + r = save_prop( si, handle, i ); + if (r != ERROR_SUCCESS) + { + msiobj_release( &si->hdr ); + return r; + } + } + + msiobj_release( &si->hdr ); + return ERROR_SUCCESS; +} + UINT WINAPI MsiSummaryInfoPersist( MSIHANDLE handle ) { MSISUMMARYINFO *si; -- 2.20.1

1 0