wine-commits May 2022

wine-commits@winehq.org

2 participants
907 discussions

Hans Leidekker : mshtml: Implement HTMLStorage_removeItem().
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 617ea8b930b7223af87111a24b0fad552b24874f URL: https://source.winehq.org/git/wine.git/?a=commit;h=617ea8b930b7223af87111a2… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon May 30 10:29:48 2022 +0200 mshtml: Implement HTMLStorage_removeItem(). Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlstorage.c | 56 +++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 54 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlstorage.c b/dlls/mshtml/htmlstorage.c index f7b92114622..1983d87a296 100644 --- a/dlls/mshtml/htmlstorage.c +++ b/dlls/mshtml/htmlstorage.c @@ -454,11 +454,63 @@ static HRESULT WINAPI HTMLStorage_setItem(IHTMLStorage *iface, BSTR bstrKey, BST return hres; } +static HRESULT remove_item(const WCHAR *filename, BSTR key) +{ + IXMLDOMDocument *doc; + IXMLDOMNode *root = NULL, *node = NULL; + BSTR query = NULL; + HRESULT hres; + + hres = open_document(filename, &doc); + if(hres != S_OK) + return hres; + + hres = get_root_node(doc, &root); + if(hres != S_OK) + goto done; + + query = build_query(key); + if(!query) { + hres = E_OUTOFMEMORY; + goto done; + } + + hres = IXMLDOMNode_selectSingleNode(root, query, &node); + if(hres == S_OK) { + hres = IXMLDOMNode_removeChild(root, node, NULL); + if(hres != S_OK) + goto done; + } + + hres = save_document(doc, filename); + +done: + SysFreeString(query); + if(root) + IXMLDOMNode_Release(root); + if(node) + IXMLDOMNode_Release(node); + IXMLDOMDocument_Release(doc); + return hres; +} + static HRESULT WINAPI HTMLStorage_removeItem(IHTMLStorage *iface, BSTR bstrKey) { HTMLStorage *This = impl_from_IHTMLStorage(iface); - FIXME("(%p)->(%s)\n", This, debugstr_w(bstrKey)); - return E_NOTIMPL; + HRESULT hres; + + TRACE("(%p)->(%s)\n", This, debugstr_w(bstrKey)); + + if(!This->filename) { + FIXME("session storage not supported\n"); + return E_NOTIMPL; + } + + WaitForSingleObject(This->mutex, INFINITE); + hres = remove_item(This->filename, bstrKey); + ReleaseMutex(This->mutex); + + return hres; } static HRESULT WINAPI HTMLStorage_clear(IHTMLStorage *iface)

1 0

Hans Leidekker : mshtml: Implement HTMLStorage_setItem().
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 076281c016fdf5bc2955f62cff6090475c992bee URL: https://source.winehq.org/git/wine.git/?a=commit;h=076281c016fdf5bc2955f62c… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon May 30 10:29:47 2022 +0200 mshtml: Implement HTMLStorage_setItem(). Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlstorage.c | 113 +++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 111 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlstorage.c b/dlls/mshtml/htmlstorage.c index 8ef92453b95..f7b92114622 100644 --- a/dlls/mshtml/htmlstorage.c +++ b/dlls/mshtml/htmlstorage.c @@ -338,11 +338,120 @@ static HRESULT WINAPI HTMLStorage_getItem(IHTMLStorage *iface, BSTR bstrKey, VAR return hres; } +static HRESULT set_attribute(IXMLDOMElement *elem, const WCHAR *name, BSTR value) +{ + BSTR str; + VARIANT var; + HRESULT hres; + + str = SysAllocString(name); + if(!str) + return E_OUTOFMEMORY; + V_VT(&var) = VT_BSTR; + V_BSTR(&var) = value; + + hres = IXMLDOMElement_setAttribute(elem, str, var); + SysFreeString(str); + return hres; +} + +static HRESULT save_document(IXMLDOMDocument *doc, const WCHAR *filename) +{ + VARIANT var; + HRESULT hres; + + V_VT(&var) = VT_BSTR; + V_BSTR(&var) = SysAllocString(filename); + if(!V_BSTR(&var)) + return E_OUTOFMEMORY; + + hres = IXMLDOMDocument_save(doc, var); + SysFreeString(V_BSTR(&var)); + return hres; +} + +static HRESULT set_item(const WCHAR *filename, BSTR key, BSTR value) +{ + IXMLDOMDocument *doc; + IXMLDOMNode *root = NULL, *node = NULL; + IXMLDOMElement *elem = NULL; + BSTR query = NULL; + HRESULT hres; + + hres = open_document(filename, &doc); + if(hres != S_OK) + return hres; + + hres = get_root_node(doc, &root); + if(hres != S_OK) + goto done; + + query = build_query(key); + if(!query) { + hres = E_OUTOFMEMORY; + goto done; + } + + hres = IXMLDOMNode_selectSingleNode(root, query, &node); + if(hres == S_OK) { + hres = IXMLDOMNode_QueryInterface(node, &IID_IXMLDOMElement, (void**)&elem); + if(hres != S_OK) + goto done; + + hres = set_attribute(elem, L"value", value); + if(hres != S_OK) + goto done; + }else { + BSTR str = SysAllocString(L"item"); + hres = IXMLDOMDocument_createElement(doc, str, &elem); + SysFreeString(str); + if(hres != S_OK) + goto done; + + hres = set_attribute(elem, L"name", key); + if(hres != S_OK) + goto done; + + hres = set_attribute(elem, L"value", value); + if(hres != S_OK) + goto done; + + hres = IXMLDOMNode_appendChild(root, (IXMLDOMNode*)elem, NULL); + if(hres != S_OK) + goto done; + } + + hres = save_document(doc, filename); + +done: + SysFreeString(query); + if(root) + IXMLDOMNode_Release(root); + if(node) + IXMLDOMNode_Release(node); + if(elem) + IXMLDOMElement_Release(elem); + IXMLDOMDocument_Release(doc); + return hres; +} + static HRESULT WINAPI HTMLStorage_setItem(IHTMLStorage *iface, BSTR bstrKey, BSTR bstrValue) { HTMLStorage *This = impl_from_IHTMLStorage(iface); - FIXME("(%p)->(%s %s)\n", This, debugstr_w(bstrKey), debugstr_w(bstrValue)); - return E_NOTIMPL; + HRESULT hres; + + TRACE("(%p)->(%s %s)\n", This, debugstr_w(bstrKey), debugstr_w(bstrValue)); + + if(!This->filename) { + FIXME("session storage not supported\n"); + return E_NOTIMPL; + } + + WaitForSingleObject(This->mutex, INFINITE); + hres = set_item(This->filename, bstrKey, bstrValue); + ReleaseMutex(This->mutex); + + return hres; } static HRESULT WINAPI HTMLStorage_removeItem(IHTMLStorage *iface, BSTR bstrKey)

1 0

Hans Leidekker : mshtml: Implement HTMLStorage_getItem().
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: b810aec3cd5954d3f1e789918814df9cd543b1cd URL: https://source.winehq.org/git/wine.git/?a=commit;h=b810aec3cd5954d3f1e78991… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon May 30 10:29:46 2022 +0200 mshtml: Implement HTMLStorage_getItem(). Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlstorage.c | 269 ++++++++++++++++++++++++++++++++++++++++++- dlls/mshtml/htmlwindow.c | 4 +- dlls/mshtml/mshtml_private.h | 2 +- 3 files changed, 267 insertions(+), 8 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=b810aec3cd5954d3f1e7…

1 0

Hugh McMaster : conhost: Don't prompt the user to save their app-specific settings.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 25f012edcba62a58797079b60286ee0f9f7fb124 URL: https://source.winehq.org/git/wine.git/?a=commit;h=25f012edcba62a58797079b6… Author: Hugh McMaster <hugh.mcmaster(a)outlook.com> Date: Fri May 27 19:57:36 2022 +1000 conhost: Don't prompt the user to save their app-specific settings. On Windows, app-specific console settings are always saved to the registry unless the user cancels the console properties dialog. Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- po/ar.po | 75 ++++++++++++++++++------------------------- po/ast.po | 75 ++++++++++++++++++------------------------- po/bg.po | 76 ++++++++++++++++++-------------------------- po/ca.po | 75 ++++++++++++++++++------------------------- po/cs.po | 75 ++++++++++++++++++------------------------- po/da.po | 75 ++++++++++++++++++------------------------- po/de.po | 75 ++++++++++++++++++------------------------- po/el.po | 76 ++++++++++++++++++-------------------------- po/en.po | 75 ++++++++++++++++++------------------------- po/en_US.po | 75 ++++++++++++++++++------------------------- po/eo.po | 75 ++++++++++++++++++------------------------- po/es.po | 75 ++++++++++++++++++------------------------- po/fa.po | 75 ++++++++++++++++++------------------------- po/fi.po | 75 ++++++++++++++++++------------------------- po/fr.po | 75 ++++++++++++++++++------------------------- po/he.po | 75 ++++++++++++++++++------------------------- po/hi.po | 75 ++++++++++++++++++------------------------- po/hr.po | 75 ++++++++++++++++++------------------------- po/hu.po | 75 ++++++++++++++++++------------------------- po/it.po | 75 ++++++++++++++++++------------------------- po/ja.po | 75 ++++++++++++++++++------------------------- po/ko.po | 75 ++++++++++++++++++------------------------- po/lt.po | 75 ++++++++++++++++++------------------------- po/ml.po | 75 ++++++++++++++++++------------------------- po/nb_NO.po | 75 ++++++++++++++++++------------------------- po/nl.po | 75 ++++++++++++++++++------------------------- po/or.po | 75 ++++++++++++++++++------------------------- po/pa.po | 75 ++++++++++++++++++------------------------- po/pl.po | 75 ++++++++++++++++++------------------------- po/pt_BR.po | 75 ++++++++++++++++++------------------------- po/pt_PT.po | 75 ++++++++++++++++++------------------------- po/rm.po | 75 ++++++++++++++++++------------------------- po/ro.po | 75 ++++++++++++++++++------------------------- po/ru.po | 75 ++++++++++++++++++------------------------- po/si.po | 75 ++++++++++++++++++------------------------- po/sk.po | 75 ++++++++++++++++++------------------------- po/sl.po | 75 ++++++++++++++++++------------------------- po/sr_RS(a)cyrillic.po | 77 ++++++++++++++++++--------------------------- po/sr_RS(a)latin.po | 77 ++++++++++++++++++--------------------------- po/sv.po | 75 ++++++++++++++++++------------------------- po/ta.po | 75 ++++++++++++++++++------------------------- po/te.po | 75 ++++++++++++++++++------------------------- po/th.po | 75 ++++++++++++++++++------------------------- po/tr.po | 75 ++++++++++++++++++------------------------- po/uk.po | 75 ++++++++++++++++++------------------------- po/wa.po | 75 ++++++++++++++++++------------------------- po/wine.pot | 75 ++++++++++++++++++------------------------- po/zh_CN.po | 75 ++++++++++++++++++------------------------- po/zh_TW.po | 75 ++++++++++++++++++------------------------- programs/conhost/conhost.h | 4 --- programs/conhost/conhost.rc | 12 ------- programs/conhost/window.c | 69 ++++++---------------------------------- 52 files changed, 1528 insertions(+), 2238 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=25f012edcba62a587970…

1 0

Hugh McMaster : conhost: Only save app-specific settings that differ from the defaults.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: bb2b1d0ed16081e3a76f5b7119035a5c01680410 URL: https://source.winehq.org/git/wine.git/?a=commit;h=bb2b1d0ed16081e3a76f5b71… Author: Hugh McMaster <hugh.mcmaster(a)outlook.com> Date: Fri May 27 19:57:35 2022 +1000 conhost: Only save app-specific settings that differ from the defaults. On Windows, app-specific subkeys of HKCU\Console only hold console settings that differ from the global defaults. Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/window.c | 140 ++++++++++++++++++++++++++++++++-------------- 1 file changed, 99 insertions(+), 41 deletions(-) diff --git a/programs/conhost/window.c b/programs/conhost/window.c index 019925789fd..d5aa01dca65 100644 --- a/programs/conhost/window.c +++ b/programs/conhost/window.c @@ -280,69 +280,128 @@ static void load_config( const WCHAR *key_name, struct console_config *config ) TRACE( "%s\n", debugstr_config( config )); } -static void save_registry_key( HKEY key, const struct console_config *config ) +static void save_registry_key( HKEY key, const struct console_config *config, BOOL save_all ) { + struct console_config default_config; DWORD val, width, height, i; WCHAR color_name[13]; TRACE( "%s\n", debugstr_config( config )); + if (!save_all) + load_config( NULL, &default_config ); + for (i = 0; i < ARRAY_SIZE(config->color_map); i++) { - wsprintfW( color_name, L"ColorTable%02d", i ); - val = config->color_map[i]; - RegSetValueExW( key, color_name, 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->color_map[i] != default_config.color_map[i]) + { + wsprintfW( color_name, L"ColorTable%02d", i ); + val = config->color_map[i]; + RegSetValueExW( key, color_name, 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } } - val = config->cursor_size; - RegSetValueExW( key, L"CursorSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->cursor_size != default_config.cursor_size) + { + val = config->cursor_size; + RegSetValueExW( key, L"CursorSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->cursor_visible; - RegSetValueExW( key, L"CursorVisible", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->cursor_visible != default_config.cursor_visible) + { + val = config->cursor_visible; + RegSetValueExW( key, L"CursorVisible", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->edition_mode; - RegSetValueExW( key, L"EditionMode", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->edition_mode != default_config.edition_mode) + { + val = config->edition_mode; + RegSetValueExW( key, L"EditionMode", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - RegSetValueExW( key, L"FaceName", 0, REG_SZ, (BYTE *)&config->face_name, - (lstrlenW(config->face_name) + 1) * sizeof(WCHAR) ); + if (save_all || lstrcmpW( config->face_name, default_config.face_name )) + { + RegSetValueExW( key, L"FaceName", 0, REG_SZ, (BYTE *)&config->face_name, + (lstrlenW(config->face_name) + 1) * sizeof(WCHAR) ); + } - val = config->font_pitch_family; - RegSetValueExW( key, L"FontPitchFamily", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->font_pitch_family != default_config.font_pitch_family) + { + val = config->font_pitch_family; + RegSetValueExW( key, L"FontPitchFamily", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - width = MulDiv( config->cell_width, USER_DEFAULT_SCREEN_DPI, GetDpiForSystem() ); - height = MulDiv( config->cell_height, USER_DEFAULT_SCREEN_DPI, GetDpiForSystem() ); - val = MAKELONG( width, height ); - RegSetValueExW( key, L"FontSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->cell_height != default_config.cell_height || + config->cell_width != default_config.cell_width) + { + width = MulDiv( config->cell_width, USER_DEFAULT_SCREEN_DPI, GetDpiForSystem() ); + height = MulDiv( config->cell_height, USER_DEFAULT_SCREEN_DPI, GetDpiForSystem() ); + val = MAKELONG( width, height ); - val = config->font_weight; - RegSetValueExW( key, L"FontWeight", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + RegSetValueExW( key, L"FontSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->history_size; - RegSetValueExW( key, L"HistoryBufferSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->font_weight != default_config.font_weight) + { + val = config->font_weight; + RegSetValueExW( key, L"FontWeight", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->history_mode; - RegSetValueExW( key, L"HistoryNoDup", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->history_size != default_config.history_size) + { + val = config->history_size; + RegSetValueExW( key, L"HistoryBufferSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->insert_mode; - RegSetValueExW( key, L"InsertMode", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->history_mode != default_config.history_mode) + { + val = config->history_mode; + RegSetValueExW( key, L"HistoryNoDup", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->menu_mask; - RegSetValueExW( key, L"MenuMask", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->insert_mode != default_config.insert_mode) + { + val = config->insert_mode; + RegSetValueExW( key, L"InsertMode", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->popup_attr; - RegSetValueExW( key, L"PopupColors", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->menu_mask != default_config.menu_mask) + { + val = config->menu_mask; + RegSetValueExW( key, L"MenuMask", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } + + if (save_all || config->popup_attr != default_config.popup_attr) + { + val = config->popup_attr; + RegSetValueExW( key, L"PopupColors", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->quick_edit; - RegSetValueExW( key, L"QuickEdit", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->quick_edit != default_config.quick_edit) + { + val = config->quick_edit; + RegSetValueExW( key, L"QuickEdit", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = MAKELONG(config->sb_width, config->sb_height); - RegSetValueExW( key, L"ScreenBufferSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->sb_width != default_config.sb_width || + config->sb_height != default_config.sb_height) + { + val = MAKELONG(config->sb_width, config->sb_height); + RegSetValueExW( key, L"ScreenBufferSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = config->attr; - RegSetValueExW( key, L"ScreenColors", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->attr != default_config.attr) + { + val = config->attr; + RegSetValueExW( key, L"ScreenColors", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } - val = MAKELONG( config->win_width, config->win_height ); - RegSetValueExW( key, L"WindowSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + if (save_all || config->win_width != default_config.win_width || + config->win_height != default_config.win_height) + { + val = MAKELONG( config->win_width, config->win_height ); + RegSetValueExW( key, L"WindowSize", 0, REG_DWORD, (BYTE *)&val, sizeof(val) ); + } } static void save_config( const WCHAR *key_name, const struct console_config *config ) @@ -365,12 +424,11 @@ static void save_config( const WCHAR *key_name, const struct console_config *con } else { - /* FIXME: maybe only save the values different from the default value ? */ - save_registry_key( app_key, config ); + save_registry_key( app_key, config, FALSE ); RegCloseKey( app_key ); } } - else save_registry_key( key, config ); + else save_registry_key( key, config, TRUE ); RegCloseKey(key); }

1 0

Hugh McMaster : conhost: Save console settings as global defaults on first run.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 37bcbe6bcace68ddc56bd2757ca993e2c82e4e7e URL: https://source.winehq.org/git/wine.git/?a=commit;h=37bcbe6bcace68ddc56bd275… Author: Hugh McMaster <hugh.mcmaster(a)outlook.com> Date: Fri May 27 19:57:34 2022 +1000 conhost: Save console settings as global defaults on first run. On Windows, HKCU\Console holds global default settings, while subkeys hold any app-specific settings that differ from the defaults. Wine's conhost.exe implementation currently saves all console settings to an app-specific subkey on the first run, while global defaults are only saved to HKCU\Console if the user selects 'Set Defaults' from the pop-up menu and saves new settings. This is the opposite of console behaviour on Windows. Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/window.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/programs/conhost/window.c b/programs/conhost/window.c index 5e8c3e15a4f..019925789fd 100644 --- a/programs/conhost/window.c +++ b/programs/conhost/window.c @@ -850,7 +850,7 @@ static void set_first_font( struct console *console, struct console_config *conf if (fc.pass > 5) ERR("Unable to find a valid console font\n"); - /* Save font configuration to the registry */ + /* Update active configuration */ config->cell_width = console->active->font.width; config->cell_height = console->active->font.height; config->font_pitch_family = console->active->font.pitch_family; @@ -858,7 +858,8 @@ static void set_first_font( struct console *console, struct console_config *conf console->active->font.face_len * sizeof(WCHAR) ); config->face_name[console->active->font.face_len] = 0; - save_config( console->window->config_key, config ); + /* Save default console configuration to the registry */ + save_config( NULL, config ); } /* Sets the font specified in the LOGFONT as the new console font */

1 0

Robert Wilhelm : scrrun: Check for null arguments in MoveFile.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: e5ae7a4026f46181d6a45a2026e78a4e177ba279 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e5ae7a4026f46181d6a45a20… Author: Robert Wilhelm <robert.wilhelm(a)gmx.net> Date: Wed May 25 14:53:11 2022 +0200 scrrun: Check for null arguments in MoveFile. Signed-off-by: Robert Wilhelm <robert.wilhelm(a)gmx.net> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/scrrun/filesystem.c | 3 +++ dlls/scrrun/tests/filesystem.c | 7 +++++++ 2 files changed, 10 insertions(+) diff --git a/dlls/scrrun/filesystem.c b/dlls/scrrun/filesystem.c index d7ad255e517..35b43e2f124 100644 --- a/dlls/scrrun/filesystem.c +++ b/dlls/scrrun/filesystem.c @@ -3758,6 +3758,9 @@ static HRESULT WINAPI filesys_MoveFile(IFileSystem3 *iface, BSTR source, BSTR de { TRACE("%p %s %s\n", iface, debugstr_w(source), debugstr_w(destination)); + if(!source || !destination) + return E_INVALIDARG; + return MoveFileW(source, destination) ? S_OK : create_error(GetLastError()); } diff --git a/dlls/scrrun/tests/filesystem.c b/dlls/scrrun/tests/filesystem.c index 79af87591ff..1ba0e81a2d1 100644 --- a/dlls/scrrun/tests/filesystem.c +++ b/dlls/scrrun/tests/filesystem.c @@ -2591,6 +2591,13 @@ static void test_MoveFile(void) hr = IFileSystem3_DeleteFile(fs3, str, VARIANT_TRUE); ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); SysFreeString(str); + + str = SysAllocString(L"null.txt"); + hr = IFileSystem3_MoveFile(fs3, str, NULL); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + hr = IFileSystem3_MoveFile(fs3, NULL, str); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + SysFreeString(str); } static void test_DoOpenPipeStream(void)

1 0

Alexandre Julliard : kernel32/tests: Add an option to generate sortkeys from an input file.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: d8bea38f96034b77ed1f5318f70ee7f072bf32b5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d8bea38f96034b77ed1f5318… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon May 30 13:15:56 2022 +0200 kernel32/tests: Add an option to generate sortkeys from an input file. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/locale.c | 68 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c index e15626ec9ed..75e9104ba03 100644 --- a/dlls/kernel32/tests/locale.c +++ b/dlls/kernel32/tests/locale.c @@ -8116,10 +8116,78 @@ static void test_EnumCalendarInfoExW(void) } } +/* Generate sort keys for a list of Unicode code points. + * Possible source files: + * The Unicode collation test suite: https://www.unicode.org/Public/UCA/latest/CollationTest.zip + * The list of supported char compressions: winedump nls/sortdefault.nls | grep \\-\> + */ +static void dump_sortkeys( char *argv[] ) +{ + WCHAR data[128]; + WCHAR locale[LOCALE_NAME_MAX_LENGTH]; + BYTE key[256]; + unsigned int i, val, pos, res, flags = 0; + char *p, *end, buffer[1024]; + FILE *f = fopen( argv[1], "r" ); + + locale[0] = 0; + if (argv[2]) + { + MultiByteToWideChar( CP_ACP, 0, argv[2], -1, locale, LOCALE_NAME_MAX_LENGTH ); + if (argv[3]) flags = strtoul( argv[3], NULL, 0 ); + } + + if (!f) + { + fprintf( stderr, "cannot open %s\n", argv[1] ); + return; + } + while (fgets( buffer, sizeof(buffer), f )) + { + if (buffer[0] && buffer[strlen(buffer)-1] == '\n') buffer[strlen(buffer)-1] = 0; + p = buffer; + while (*p == ' ' || *p == '\t') p++; + if (*p == '#') continue; + pos = 0; + while (*p && *p != ';' && *p != '-') + { + val = strtoul( p, &end, 16 ); + if (end == p) break; + if (val >= 0x10000) + { + data[pos++] = 0xd800 | (val >> 10); + data[pos++] = 0xdc00 | (val & 0x3ff); + } + else data[pos++] = val; + p = end; + while (*p == ' ' || *p == '\t') p++; + } + *p = 0; + res = LCMapStringEx( locale, flags | LCMAP_SORTKEY, data, pos, + (WCHAR *)key, sizeof(key), NULL, NULL, 0 ); + printf( "%s:", buffer ); + for (i = 0; i < res; i++) printf( " %02x", key[i] ); + printf( "\n" ); + } + fclose( f ); +} + START_TEST(locale) { + char **argv; + int argc = winetest_get_mainargs( &argv ); + InitFunctionPointers(); + if (argc >= 4) + { + if (!strcmp( argv[2], "sortkeys" )) + { + dump_sortkeys( argv + 2 ); + return; + } + } + test_EnumTimeFormatsA(); test_EnumTimeFormatsW(); test_EnumDateFormatsA();

1 0

Alexandre Julliard : kernel32/tests: Add a few more sortkey tests.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 2756fec551a0ad74f23484c7c9508a77c8b91f59 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2756fec551a0ad74f23484c7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon May 30 16:06:36 2022 +0200 kernel32/tests: Add a few more sortkey tests. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/locale.c | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c index 9327a074179..e15626ec9ed 100644 --- a/dlls/kernel32/tests/locale.c +++ b/dlls/kernel32/tests/locale.c @@ -2587,6 +2587,25 @@ static void test_lcmapstring_unicode(lcmapstring_wrapper func_ptr, const char *f ok(ret == ret2, "%s lengths of sort keys must be equal\n", func_name); ok(!memcmp(p_buf, p_buf2, ret), "%s sort keys must be equal\n", func_name); + /* test contents with short buffer */ + memset( buf, 0xcc, sizeof(buf) ); + ret = func_ptr(LCMAP_SORTKEY, upper_case, -1, buf, ret - 1); + ok( !ret, "%s succeeded with %u\n", func_name, ret ); + ok( GetLastError() == ERROR_INSUFFICIENT_BUFFER, "%s wrong error %lu\n", func_name, GetLastError() ); + ret = (char *)memchr( p_buf, 0xcc, sizeof(buf) ) - p_buf; + ok( ret, "%s buffer not filled\n", func_name ); + ok( p_buf[ret - 1] == 0x01, "%s buffer filled up to %02x\n", func_name, (BYTE)p_buf[ret - 1] ); + ok( !memcmp( p_buf, p_buf2, ret - 1 ), "%s buffers differ\n", func_name ); + + memset( buf, 0xcc, sizeof(buf) ); + ret = func_ptr(LCMAP_SORTKEY, upper_case, -1, buf, ret2 - 20); + ok( !ret, "%s succeeded with %u\n", func_name, ret ); + ok( GetLastError() == ERROR_INSUFFICIENT_BUFFER, "%s wrong error %lu\n", func_name, GetLastError() ); + ret = (char *)memchr( p_buf, 0xcc, sizeof(buf) ) - p_buf; + ok( ret, "%s buffer not filled\n", func_name ); + ok( p_buf[ret - 1] == 0x01, "%s buffer filled up to %02x\n", func_name, (BYTE)p_buf[ret - 1] ); + ok( !memcmp( p_buf, p_buf2, ret - 1 ), "%s buffers differ\n", func_name ); + /* test LCMAP_SORTKEY | NORM_IGNORECASE */ ret = func_ptr(LCMAP_SORTKEY | NORM_IGNORECASE, upper_case, -1, buf, sizeof(buf)); @@ -3522,6 +3541,8 @@ static const struct sorting_test_entry unicode_sorting_tests[] = { L"en-US", -1, CSTR_LESS_THAN, NORM_IGNORECASE, L"\xffb7", L"\x3147" }, { L"en-US", -1, CSTR_LESS_THAN, NORM_IGNORECASE, L"\xffb6", L"\x3146" }, { L"en-US", 1, CSTR_GREATER_THAN, NORM_IGNORECASE, L"\x3145", L"\xffb5" }, + { L"en-US", 1, CSTR_GREATER_THAN, 0, L"\x3075\x30fc", L"\x30d5\x30fc" }, + { L"en-US", -1, CSTR_LESS_THAN, 0, L"\x30a1\x30fc", L"\x30a2\x30fc" }, /* Coptic < Japanese */ { L"en-US", -1, CSTR_LESS_THAN, NORM_IGNORECASE, L"\x2cff", L"\x30ba" }, { L"en-US", -1, CSTR_LESS_THAN, NORM_IGNORECASE, L"\x2cdb", L"\x32de" }, @@ -3537,6 +3558,7 @@ static const struct sorting_test_entry unicode_sorting_tests[] = { L"en-US", -1, CSTR_EQUAL, 0, L"\x0f75", L"\x0f71\x0f74" }, { L"en-US", -1, CSTR_EQUAL, 0, L"\xfc5e", L"\x064c\x0651" }, { L"en-US", -1, CSTR_EQUAL, 0, L"\xfb2b", L"\x05e9\x05c2" }, + { L"en-US", -1, CSTR_EQUAL, 0, L"\xfe71", L"\x0640\x064b" }, /* Japanese locale */ { L"ja-JP", -1, CSTR_LESS_THAN, 0, L"\x6df8", L"\x654b\x29e9" }, { L"ja-JP", -1, CSTR_LESS_THAN, 0, L"\x685d\x1239\x1b61", L"\x59b6\x6542\x2a62\x04a7" }, @@ -3593,6 +3615,16 @@ static const struct sorting_test_entry unicode_sorting_tests[] = { L"en-US", 1, CSTR_GREATER_THAN, 0, L"\xa042\x09bc", L"\xa042" }, { L"en-US", 1, CSTR_GREATER_THAN, 0, L"\xa063\x302b", L"\xa063" }, { L"en-US", 1, CSTR_GREATER_THAN, 0, L"\xa07e\x0c56", L"\xa07e" }, + /* Reversed diacritics */ + { L"en-US", -1, CSTR_LESS_THAN, 0, L"\x00e9\x00e8", L"\x00e8\x00e9" }, + { L"fr-FR", 1, CSTR_GREATER_THAN, 0, L"\x00e9\x00e8", L"\x00e8\x00e9" }, + /* Digit sort */ + { L"en-US", -1, CSTR_LESS_THAN, 0, L"1230", L"321" }, + { L"en-US", 1, CSTR_GREATER_THAN, SORT_DIGITSASNUMBERS, L"1230", L"321" }, + { L"en-US", 1, CSTR_GREATER_THAN, SORT_DIGITSASNUMBERS, L"\xc6f\xc6c\xc6a", L"\xc6f\xc6e" }, + /* Compressions */ + { L"en-US", -1, CSTR_LESS_THAN, 0, L"E\x0300", L"F" }, + { L"rm-CH", 1, CSTR_GREATER_THAN, 0, L"E\x0300", L"F" }, }; static void test_unicode_sorting(void)

1 0

Fabian Maurer : kernel32/tests: Add a number of Unicode collation tests.
by Alexandre Julliard 30 May '22

30 May '22

Module: wine Branch: master Commit: 64ba1b3245c14f74e5359a3595d8623bb974b1aa URL: https://source.winehq.org/git/wine.git/?a=commit;h=64ba1b3245c14f74e5359a35… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Mon May 30 15:54:24 2022 +0200 kernel32/tests: Add a number of Unicode collation tests. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/locale.c | 369 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 369 insertions(+) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=64ba1b3245c14f74e535…

1 0

← Newer
1
...
6
7
8
9
10
11
12
...
91
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits May 2022