From 880e7e8dbe7f63c261093902a2d1af27bcb22773 Mon Sep 17 00:00:00 2001
From: Ged Murphy <gedmurphy@reactos.org>
Date: Tue, 27 Dec 2005 18:59:13 +0000
Subject: [PATCH] Fix a few msvc warnings

svn path=/trunk/; revision=20376
---
 reactos/apps/utils/net/tracert/tracert.c | 24 +++++-------------------
 reactos/apps/utils/net/tracert/tracert.h | 13 +++++++++++++
 2 files changed, 18 insertions(+), 19 deletions(-)

diff --git a/reactos/apps/utils/net/tracert/tracert.c b/reactos/apps/utils/net/tracert/tracert.c
index 94a14adeb6d..e41269eebbd 100644
--- a/reactos/apps/utils/net/tracert/tracert.c
+++ b/reactos/apps/utils/net/tracert/tracert.c
@@ -69,20 +69,6 @@ INT iMaxHops = 30;              // -h  Max number of hops before trace ends
 INT iHostList;                  // -j  @UNIMPLEMENTED@
 INT iTimeOut = 2000;            // -w  time before packet times out
 
-/* function definitions */
-static BOOL ParseCmdline(int argc, char* argv[]);
-static INT Driver(void);
-static INT Setup(INT ttl);
-static VOID SetupTimingMethod(void);
-static VOID ResolveHostname(void);
-static VOID PreparePacket(INT packetSize, INT seqNum);
-static INT SendPacket(INT datasize);
-static INT ReceivePacket(INT datasize);
-static INT DecodeResponse(INT packetSize, INT seqNum);
-static LONG GetTime(void);
-static WORD CheckSum(PUSHORT data, UINT size);
-static VOID Usage(void);
-
 
 /*
  *
@@ -137,7 +123,7 @@ static INT Driver(VOID)
 {
 
     INT iHopCount = 1;              // hop counter. default max is 30
-    INT iSeqNum = 0;                // initialise packet sequence number
+    USHORT iSeqNum = 0;                // initialise packet sequence number
     INT iTTL = 1;                   // set initial packet TTL to 1
     BOOL bFoundTarget = FALSE;      // Have we reached our destination yet
     BOOL bAwaitPacket;              // indicates whether we have recieved a good packet
@@ -433,7 +419,7 @@ static INT Setup(INT iTTL)
  * Calculate the packet checksum
  *
  */
-static VOID PreparePacket(INT iPacketSize, INT iSeqNum)
+static VOID PreparePacket(INT iPacketSize, USHORT iSeqNum)
 {
     /* assemble ICMP echo request packet */
     sendpacket.icmpheader.type      = ECHO_REQUEST;
@@ -480,8 +466,8 @@ static INT SendPacket(INT datasize)
         if (WSAGetLastError() == WSAEACCES)
         {
             _tprintf(_T("\n\nYou must be an administrator to run this program!\n\n"));
-            exit(1);
             WSACleanup();
+            exit(1);
         }
         else
         {
@@ -584,7 +570,7 @@ static INT ReceivePacket(INT datasize)
  * It all is well, print the time taken for the round trip.
  *
  */
-static INT DecodeResponse(INT iPacketSize, INT iSeqNum)
+static INT DecodeResponse(INT iPacketSize, USHORT iSeqNum)
 {
     unsigned short header_len = recvpacket.h_len * 4;
     /* cast the recieved packet into an ECHO reply and a TTL Exceed so we can check the ID*/
@@ -645,7 +631,7 @@ static INT DecodeResponse(INT iPacketSize, INT iSeqNum)
  *
  */
 
-static LONG GetTime(VOID)
+static LONGLONG GetTime(VOID)
 {
     LARGE_INTEGER Time;
 
diff --git a/reactos/apps/utils/net/tracert/tracert.h b/reactos/apps/utils/net/tracert/tracert.h
index eee6e7c6ce4..67801f0f9b5 100644
--- a/reactos/apps/utils/net/tracert/tracert.h
+++ b/reactos/apps/utils/net/tracert/tracert.h
@@ -68,3 +68,16 @@ typedef struct TTLExceedHeader
 /* return to normal */
 #include <poppack.h>
 
+/* function definitions */
+static BOOL ParseCmdline(int argc, char* argv[]);
+static INT Driver(void);
+static INT Setup(INT ttl);
+static VOID SetupTimingMethod(void);
+static VOID ResolveHostname(void);
+static VOID PreparePacket(INT packetSize, USHORT seqNum);
+static INT SendPacket(INT datasize);
+static INT ReceivePacket(INT datasize);
+static INT DecodeResponse(INT packetSize, USHORT seqNum);
+static LONGLONG GetTime(void);
+static WORD CheckSum(PUSHORT data, UINT size);
+static VOID Usage(void);
-- 
2.17.1