[PATCH v2 0/2] MR8075: windows.networking.connectivity: Implement NetworkInformation IActivationFactory::GetRuntimeClassName().
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58239 -- v2: windows.networking.connectivity: Implement NetworkInformation IActivationFactory::GetRuntimeClassName(). windows.networking.connectivity/tests: Add NetworkInformation IActivationFactory::GetRuntimeClassName() test. https://gitlab.winehq.org/wine/wine/-/merge_requests/8075
From: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> --- dlls/windows.networking.connectivity/tests/connectivity.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/dlls/windows.networking.connectivity/tests/connectivity.c b/dlls/windows.networking.connectivity/tests/connectivity.c index 88634cbea95..80af03b4b67 100644 --- a/dlls/windows.networking.connectivity/tests/connectivity.c +++ b/dlls/windows.networking.connectivity/tests/connectivity.c @@ -80,6 +80,13 @@ static void test_NetworkInformationStatics(void) hr = IActivationFactory_QueryInterface( factory, &IID_INetworkInformationStatics, (void **)&network_information_statics ); ok( hr == S_OK, "got hr %#lx.\n", hr ); + str = (void *)0xdeadbeef; + hr = IActivationFactory_GetRuntimeClassName( factory, &str ); + todo_wine + ok( hr == E_ILLEGAL_METHOD_CALL, "got hr %#lx.\n", hr ); + todo_wine + ok( str == NULL, "got str %s.\n", wine_dbgstr_hstring( str ) ); + hr = CoCreateInstance( &CLSID_NetworkListManager, NULL, CLSCTX_INPROC_SERVER, &IID_INetworkListManager, (void **)&network_list_manager ); ok( hr == S_OK, "got hr %#lx.\n", hr ); if (FAILED(hr)) -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/8075
From: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58239 --- dlls/windows.networking.connectivity/network_information.c | 6 ++++-- dlls/windows.networking.connectivity/tests/connectivity.c | 2 -- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/windows.networking.connectivity/network_information.c b/dlls/windows.networking.connectivity/network_information.c index fa041fce754..029053d106c 100644 --- a/dlls/windows.networking.connectivity/network_information.c +++ b/dlls/windows.networking.connectivity/network_information.c @@ -89,8 +89,10 @@ static HRESULT WINAPI factory_GetIids( IActivationFactory *iface, ULONG *iid_cou static HRESULT WINAPI factory_GetRuntimeClassName( IActivationFactory *iface, HSTRING *class_name ) { - FIXME( "iface %p, class_name %p stub!\n", iface, class_name ); - return E_NOTIMPL; + TRACE( "iface %p, class_name %p\n", iface, class_name ); + + *class_name = NULL; + return E_ILLEGAL_METHOD_CALL; } static HRESULT WINAPI factory_GetTrustLevel( IActivationFactory *iface, TrustLevel *trust_level ) diff --git a/dlls/windows.networking.connectivity/tests/connectivity.c b/dlls/windows.networking.connectivity/tests/connectivity.c index 80af03b4b67..dc963d8bb9a 100644 --- a/dlls/windows.networking.connectivity/tests/connectivity.c +++ b/dlls/windows.networking.connectivity/tests/connectivity.c @@ -82,9 +82,7 @@ static void test_NetworkInformationStatics(void) str = (void *)0xdeadbeef; hr = IActivationFactory_GetRuntimeClassName( factory, &str ); - todo_wine ok( hr == E_ILLEGAL_METHOD_CALL, "got hr %#lx.\n", hr ); - todo_wine ok( str == NULL, "got str %s.\n", wine_dbgstr_hstring( str ) ); hr = CoCreateInstance( &CLSID_NetworkListManager, NULL, CLSCTX_INPROC_SERVER, &IID_INetworkListManager, (void **)&network_list_manager ); -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/8075
On Tue May 20 08:43:27 2025 +0000, Rémi Bernon wrote:
This seems okay but I would be surprised it makes any difference for the application, can we confirm that it does? Yeah, I found it strange that it calls this function. Maybe they're using mods? I'd give it a week for the reporter to confirm, as bugs are often abandoned.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8075#note_104064
participants (2)
-
Mohamad Al-Jaf -
Mohamad Al-Jaf (＠maljaf)