ping[6]: don't do htons(a++), it can be a macro

1 files changed, 2 insertions, 1 deletions

diff --git a/networking/ping.c b/networking/ping.c
index acbb8ec6f..8b49df1bd 100644
--- a/networking/ping.c
+++ b/networking/ping.c
@@ -218,9 +218,10 @@ static void sendping(int junk)
         pkt->icmp_type = ICMP_ECHO;
         pkt->icmp_code = 0;
         pkt->icmp_cksum = 0;
-        pkt->icmp_seq = htons(ntransmitted++);
+        pkt->icmp_seq = htons(ntransmitted); /* don't ++ here, it can be a macro */
         pkt->icmp_id = myid;
         CLR(ntohs(pkt->icmp_seq) % MAX_DUP_CHK);
+        ntransmitted++;
 
         gettimeofday((struct timeval *) &pkt->icmp_dun, NULL);
         pkt->icmp_cksum = in_cksum((unsigned short *) pkt, sizeof(packet));