From: Christian Grothoff <christian@grothoff.org>
Date: Fri, 20 Aug 2010 14:03:17 +0000 (+0000)
Subject: more code cleanup
X-Git-Tag: initial-import-from-subversion-38251~20569
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=4981e720cb71b9bf5e51dd90b5a9abacf21665ca;p=oweals%2Fgnunet.git

more code cleanup
---

diff --git a/src/transport/gnunet-nat-client-windows.c b/src/transport/gnunet-nat-client-windows.c
index 26cf72f6a..febcfc717 100644
--- a/src/transport/gnunet-nat-client-windows.c
+++ b/src/transport/gnunet-nat-client-windows.c
@@ -249,7 +249,6 @@ send_icmp_udp (const struct in_addr *my_ip,
 
   /* ip header: send to (known) ip address */
   off = 0;
-  memset(&ip_pkt, 0, sizeof(ip_pkt));
   ip_pkt.vers_ihl = 0x45;
   ip_pkt.tos = 0;
   ip_pkt.pkt_len = htons(sizeof (packet));
diff --git a/src/transport/gnunet-nat-client.c b/src/transport/gnunet-nat-client.c
index ddb59bc84..2fc2a51a0 100644
--- a/src/transport/gnunet-nat-client.c
+++ b/src/transport/gnunet-nat-client.c
@@ -40,10 +40,11 @@
  * - Christian Grothoff
  * - Nathan Evans
  */
-#define _GNU_SOURCE
 #if HAVE_CONFIG_H
 /* Just needed for HAVE_SOCKADDR_IN_SIN_LEN test macro! */
 #include "gnunet_config.h"
+#else
+#define _GNU_SOURCE
 #endif
 #include <sys/types.h> 
 #include <sys/socket.h>
@@ -375,7 +376,7 @@ send_icmp (const struct in_addr *my_ip,
 
   /* no go back to calculate ICMP packet checksum */
   off = sizeof (ip_pkt);
-  icmp_pkt.checksum = htons(calc_checksum(&packet[off],
+  icmp_pkt.checksum = htons(calc_checksum((uint16_t*) &packet[off],
 					  sizeof (struct icmp_packet) + sizeof(struct ip_packet) + sizeof(struct icmp_echo_packet)));
   memcpy (&packet[off],
 	  &icmp_pkt,
diff --git a/src/transport/gnunet-nat-server-windows.c b/src/transport/gnunet-nat-server-windows.c
index ceff3fb8d..12aa5b138 100644
--- a/src/transport/gnunet-nat-server-windows.c
+++ b/src/transport/gnunet-nat-server-windows.c
@@ -255,7 +255,7 @@ send_icmp_echo (const struct in_addr *my_ip)
   ip_pkt.src_ip = my_ip->s_addr;
   ip_pkt.dst_ip = dummy.s_addr;
   ip_pkt.checksum = htons(calc_checksum((uint16_t*)&ip_pkt, sizeof (ip_pkt)));
-  memcpy (packet, &ip_pkt, sizeof (ip_pkt));
+  memcpy (&packet[off], &ip_pkt, sizeof (ip_pkt));
   off += sizeof (ip_pkt);
 
   icmp_echo.type = ICMP_ECHO;
diff --git a/src/transport/gnunet-nat-server.c b/src/transport/gnunet-nat-server.c
index fb88874d3..e9cbee47c 100644
--- a/src/transport/gnunet-nat-server.c
+++ b/src/transport/gnunet-nat-server.c
@@ -40,10 +40,11 @@
  * - Christian Grothoff
  * - Nathan Evans
  */
-#define _GNU_SOURCE
 #if HAVE_CONFIG_H
 /* Just needed for HAVE_SOCKADDR_IN_SIN_LEN test macro! */
 #include "gnunet_config.h"
+#else
+#define _GNU_SOURCE
 #endif
 #include <sys/types.h> 
 #include <sys/socket.h>
@@ -237,9 +238,6 @@ send_icmp_echo (const struct in_addr *my_ip)
   icmp_echo.checksum = 0;
   icmp_echo.checksum = htons(calc_checksum((uint16_t*)&icmp_echo, 
 					   sizeof (struct icmp_packet)));
-
-  make_echo (my_ip, &icmp_echo);
-
   memcpy (&packet[off], &icmp_echo, sizeof (icmp_echo));
   off += sizeof (icmp_echo);