[PATCH 0/2] MR4926: Adapter test

[John Freeman (＠tingling0072)]

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4926

[PATCH 1/2] added test_GetAdaptersAddressesSizeZeroParam

[John Freeman]

From: John Freeman <john(a)freemania.pro>

---
 dlls/iphlpapi/tests/iphlpapi.c | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/dlls/iphlpapi/tests/iphlpapi.c b/dlls/iphlpapi/tests/iphlpapi.c
index cd192003345..5bb5ab13ee3 100644
--- a/dlls/iphlpapi/tests/iphlpapi.c
+++ b/dlls/iphlpapi/tests/iphlpapi.c
@@ -67,7 +67,7 @@ static DWORD (WINAPI *pParseNetworkString)(const WCHAR*,DWORD,NET_ADDRESS_INFO*,
 static DWORD (WINAPI *pNotifyUnicastIpAddressChange)(ADDRESS_FAMILY, PUNICAST_IPADDRESS_CHANGE_CALLBACK,
                                                 PVOID, BOOLEAN, HANDLE *);
 static DWORD (WINAPI *pCancelMibChangeNotify2)(HANDLE);
-
+static DWORD (WINAPI *pGetAdaptersAddresses)(ULONG, ULONG, void*, IP_ADAPTER_ADDRESSES*, ULONG*)
 DWORD WINAPI ConvertGuidToStringA( const GUID *, char *, DWORD );
 DWORD WINAPI ConvertGuidToStringW( const GUID *, WCHAR *, DWORD );
 DWORD WINAPI ConvertStringToGuidW( const WCHAR *, GUID * );
@@ -88,6 +88,7 @@ static void loadIPHlpApi(void)
     pParseNetworkString = (void *)GetProcAddress(hLibrary, "ParseNetworkString");
     pNotifyUnicastIpAddressChange = (void *)GetProcAddress(hLibrary, "NotifyUnicastIpAddressChange");
     pCancelMibChangeNotify2 = (void *)GetProcAddress(hLibrary, "CancelMibChangeNotify2");
+    pGetAdaptersAddresses = (void *)GetProcAddress(hLibrary, "GetAdaptersAddresses");
   }
 }
 
@@ -2913,6 +2914,18 @@ static void test_compartments(void)
     id = GetCurrentThreadCompartmentId();
     ok(id == NET_IF_COMPARTMENT_ID_PRIMARY, "got %u\n", id);
 }
+//Test passes no parameters of importance but returns error if size passed into
+//GetAdaptersAddresses is zero and is Ok if the returned value is ERROR_INVALID_PARAMETER
+static void test_GetAdaptersAddressesSizeZeroParamReturnInvalidParameterError(void)
+{
+    ULONG family = 0;
+    ULONG flags = 0;
+    void *reserved;
+    IP_ADAPTER_ADDRESSES *aa;
+    ULONG *size;
+    ULONG err = GetAdaptersAddresses(family, flags, reserved, aa, size);
+    ok(err == ERROR_INVALID_PARAMETER, "got %ld\n", err);
+}
 
 START_TEST(iphlpapi)
 {
@@ -2949,6 +2962,7 @@ START_TEST(iphlpapi)
     test_NotifyUnicastIpAddressChange();
     test_ConvertGuidToString();
     test_compartments();
+    test_GetAdaptersAddressesSizeZeroParamReturnInvalidParameterError();
     freeIPHlpApi();
   }
 }
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/4926

[PATCH 2/2] missing semi-colon

[John H. Freeman]

From: "John H. Freeman" <john(a)freemania.pro>

---
 dlls/iphlpapi/tests/iphlpapi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/iphlpapi/tests/iphlpapi.c b/dlls/iphlpapi/tests/iphlpapi.c
index 5bb5ab13ee3..013cb4de7e9 100644
--- a/dlls/iphlpapi/tests/iphlpapi.c
+++ b/dlls/iphlpapi/tests/iphlpapi.c
@@ -67,7 +67,7 @@ static DWORD (WINAPI *pParseNetworkString)(const WCHAR*,DWORD,NET_ADDRESS_INFO*,
 static DWORD (WINAPI *pNotifyUnicastIpAddressChange)(ADDRESS_FAMILY, PUNICAST_IPADDRESS_CHANGE_CALLBACK,
                                                 PVOID, BOOLEAN, HANDLE *);
 static DWORD (WINAPI *pCancelMibChangeNotify2)(HANDLE);
-static DWORD (WINAPI *pGetAdaptersAddresses)(ULONG, ULONG, void*, IP_ADAPTER_ADDRESSES*, ULONG*)
+static DWORD (WINAPI *pGetAdaptersAddresses)(ULONG, ULONG, void*, IP_ADAPTER_ADDRESSES*, ULONG*);
 DWORD WINAPI ConvertGuidToStringA( const GUID *, char *, DWORD );
 DWORD WINAPI ConvertGuidToStringW( const GUID *, WCHAR *, DWORD );
 DWORD WINAPI ConvertStringToGuidW( const WCHAR *, GUID * );
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/4926

[John H. Freeman (＠tingling0072)]

Test to detect failure condition of size null pointer [#51642](https://bugs.winehq.org/show_bug.cgi?id=51642)

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4926#note_58758