[ros-diffs] [gedmurphy] 20390: Fix some MSVC warnings

[gedmurphy at svn.reactos.com]

Fix some MSVC warnings
I'm feeling very lazy today, today's commits prove that point nicely :)
Modified: trunk/reactos/apps/utils/net/arp/arp.c
  _____  

Modified: trunk/reactos/apps/utils/net/arp/arp.c
--- trunk/reactos/apps/utils/net/arp/arp.c	2005-12-28 01:29:42 UTC
(rev 20389)
+++ trunk/reactos/apps/utils/net/arp/arp.c	2005-12-28 02:20:27 UTC
(rev 20390)
@@ -66,26 +66,30 @@

 {
     LPVOID lpMsgBuf;
     DWORD RetVal;
-    /* double brackets to silence the assignment warning message */
-    if ((RetVal = FormatMessage(
-            FORMAT_MESSAGE_ALLOCATE_BUFFER |
-            FORMAT_MESSAGE_FROM_SYSTEM |
-            FORMAT_MESSAGE_IGNORE_INSERTS,
-            NULL,
-            ErrorCode,
-            MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), /* Default
language */
-            (LPTSTR) &lpMsgBuf,
-            0,
-            NULL ))) {
-        _tprintf(_T("%s"), (LPTSTR)lpMsgBuf);
 
-        LocalFree(lpMsgBuf);
-        /* return number of TCHAR's stored in output buffer
-         * excluding '\0' - as FormatMessage does*/
-        return RetVal;
+    if (ErrorCode != ERROR_SUCCESS) 
+    {
+        RetVal = FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER |
+                               FORMAT_MESSAGE_FROM_SYSTEM |
+                               FORMAT_MESSAGE_IGNORE_INSERTS,
+                               NULL,
+                               ErrorCode,
+                               MAKELANGID(LANG_NEUTRAL,
SUBLANG_DEFAULT), /* Default language */
+                               (LPTSTR) &lpMsgBuf,
+                               0,
+                               NULL );
+
+        if (RetVal != 0)
+        {
+            _tprintf(_T("%s"), (LPTSTR)lpMsgBuf);
+
+            LocalFree(lpMsgBuf);
+            /* return number of TCHAR's stored in output buffer
+             * excluding '\0' - as FormatMessage does*/
+            return RetVal;
+        }
     }
-    else
-        return 0;
+    return 0;
 }
 
 
@@ -256,8 +260,7 @@
     DWORD dwIpAddr = 0;
     DWORD dwSize = 0;
     ULONG ulSize = 0;
-    INT iRet, i, val;
-    TCHAR c;
+    INT iRet, i, val, c;
 
     /* error checking */
 
@@ -318,7 +321,14 @@
     /* set dwIndex field to the index of a local IP address to
      * indicate the network on which the ARP entry applies */
     if (pszIfAddr)
-        sscanf(pszIfAddr, "%lx", &pAddHost->dwIndex);
+    {
+        if (sscanf(pszIfAddr, "%lx", &pAddHost->dwIndex) == EOF)
+        {
+            free(pIpNetTable);
+            free(pAddHost);
+            return EXIT_FAILURE;
+        }
+    }
     else
     {
         /* map the IP to the index */
@@ -355,7 +365,7 @@
         c = toupper(pszEthAddr[i*3 + 1]);
         c = c - (isdigit(c) ? '0' : ('A' - 10));
         val += c;
-        pAddHost->bPhysAddr[i] = val;
+        pAddHost->bPhysAddr[i] = (BYTE)val;
     }
 
 
@@ -444,7 +454,15 @@
     /* set dwIndex field to the index of a local IP address to
      * indicate the network on which the ARP entry applies */
     if (pszIfAddr)
-        sscanf(pszIfAddr, "%lx", &pDelHost->dwIndex);
+    {
+        if (sscanf(pszIfAddr, "%lx", &pDelHost->dwIndex) == EOF)
+        {
+            free(pIpNetTable);
+            free(pIpAddrTable);
+            free(pDelHost);
+            return EXIT_FAILURE;
+        }
+    }
     else
     {
         /* map the IP to the index */
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.reactos.org/pipermail/ros-diffs/attachments/20051228/495afc24/attachment.html