Changes in uspace/srv/net/inetsrv/icmp.c [9749e47:a1a101d] in mainline

File:

• uspace/srv/net/inetsrv/icmp.c (modified) (7 diffs)

uspace/srv/net/inetsrv/icmp.c

@@ -40 +40 @@
 #include <mem.h>
 #include <stdlib.h>
-#include <types/inetping.h>
+
 #include "icmp.h"
 #include "icmp_std.h"
@@ -105 +105 @@
         reply->checksum = host2uint16_t_be(checksum);
 
-        rdgram.iplink = 0;
         rdgram.src = dgram->dest;
         rdgram.dest = dgram->src;
@@ -121 +120 @@
 static int icmp_recv_echo_reply(inet_dgram_t *dgram)
 {
+        icmp_echo_t *reply;
+        inetping_sdu_t sdu;
+        uint16_t ident;
+
         log_msg(LOG_DEFAULT, LVL_DEBUG, "icmp_recv_echo_reply()");
 
@@ -126 +129 @@
                 return EINVAL;
 
-        icmp_echo_t *reply = (icmp_echo_t *) dgram->data;
-
-        inetping_sdu_t sdu;
+        reply = (icmp_echo_t *)dgram->data;
 
         sdu.src = dgram->src;
@@ -135 +136 @@
         sdu.data = reply + sizeof(icmp_echo_t);
         sdu.size = dgram->size - sizeof(icmp_echo_t);
-
-        uint16_t ident = uint16_t_be2host(reply->ident);
+        ident = uint16_t_be2host(reply->ident);
 
         return inetping_recv(ident, &sdu);
@@ -143 +143 @@
 int icmp_ping_send(uint16_t ident, inetping_sdu_t *sdu)
 {
-        size_t rsize = sizeof(icmp_echo_t) + sdu->size;
-        void *rdata = calloc(rsize, 1);
+        inet_dgram_t dgram;
+        icmp_echo_t *request;
+        void *rdata;
+        size_t rsize;
+        uint16_t checksum;
+        int rc;
+
+        rsize = sizeof(icmp_echo_t) + sdu->size;
+        rdata = calloc(rsize, 1);
         if (rdata == NULL)
                 return ENOMEM;
 
-        icmp_echo_t *request = (icmp_echo_t *) rdata;
+        request = (icmp_echo_t *)rdata;
 
         request->type = ICMP_ECHO_REQUEST;
@@ -158 +165 @@
         memcpy(rdata + sizeof(icmp_echo_t), sdu->data, sdu->size);
 
-        uint16_t checksum = inet_checksum_calc(INET_CHECKSUM_INIT, rdata, rsize);
+        checksum = inet_checksum_calc(INET_CHECKSUM_INIT, rdata, rsize);
         request->checksum = host2uint16_t_be(checksum);
-
-        inet_dgram_t dgram;
 
         dgram.src = sdu->src;
         dgram.dest = sdu->dest;
-        dgram.iplink = 0;
         dgram.tos = ICMP_TOS;
         dgram.data = rdata;
         dgram.size = rsize;
 
-        int rc = inet_route_packet(&dgram, IP_PROTO_ICMP, INET_TTL_MAX, 0);
+        rc = inet_route_packet(&dgram, IP_PROTO_ICMP, INET_TTL_MAX, 0);
 
         free(rdata);