[PATCH v2 2/4] iphlpapi: Reimplement GetBestInterfaceEx() on top of GetBestRoute2().

[Paul Gofman]

From: Paul Gofman

---
 dlls/iphlpapi/iphlpapi_main.c | 30 ++++++++++++------------------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/dlls/iphlpapi/iphlpapi_main.c b/dlls/iphlpapi/iphlpapi_main.c
index 31c6c03eb8c..e860226b5bd 100644
--- a/dlls/iphlpapi/iphlpapi_main.c
+++ b/dlls/iphlpapi/iphlpapi_main.c
@@ -1389,27 +1389,21 @@ DWORD WINAPI GetBestInterface(IPAddr dwDestAddr, PDWORD pdwBestIfIndex)
  *  Success: NO_ERROR
  *  Failure: error code from winerror.h
  */
-DWORD WINAPI GetBestInterfaceEx(struct sockaddr *pDestAddr, PDWORD pdwBestIfIndex)
+DWORD WINAPI GetBestInterfaceEx( struct sockaddr *pDestAddr, PDWORD pdwBestIfIndex )
 {
-  DWORD ret;
+    SOCKADDR_INET best_address;
+    MIB_IPFORWARD_ROW2 row;
+    DWORD ret;

-  TRACE("pDestAddr %p, pdwBestIfIndex %p\n", pDestAddr, pdwBestIfIndex);
-  if (!pDestAddr || !pdwBestIfIndex)
-    ret = ERROR_INVALID_PARAMETER;
-  else {
-    MIB_IPFORWARDROW ipRow;
+    TRACE( "pDestAddr %p, pdwBestIfIndex %p\n", pDestAddr, pdwBestIfIndex );

-    if (pDestAddr->sa_family == AF_INET) {
-      ret = GetBestRoute(((struct sockaddr_in *)pDestAddr)->sin_addr.S_un.S_addr, 0, &ipRow);
-      if (ret == ERROR_SUCCESS)
-        *pdwBestIfIndex = ipRow.dwForwardIfIndex;
-    } else {
-      FIXME("address family %d not supported\n", pDestAddr->sa_family);
-      ret = ERROR_NOT_SUPPORTED;
-    }
-  }
-  TRACE("returning %ld\n", ret);
-  return ret;
+    if (!pDestAddr || !pdwBestIfIndex) return ERROR_INVALID_PARAMETER;
+
+    ret = GetBestRoute2( NULL, 0, NULL, (const SOCKADDR_INET *)pDestAddr, 0, &row, &best_address );
+    if (!ret) *pdwBestIfIndex = row.InterfaceIndex;
+
+    TRACE( "returning %ld\n", ret );
+    return ret;
 }