unification work

diff --git a/src/fping.c b/src/fping.c
index cbc7cd5ab4bcc2872a44666f80bb37c8f3f355d2..131280fc974269a05fdce17d2ddf3a0327d716bb 100644 (file)
--- a/src/fping.c
+++ b/src/fping.c
@@ -108,7 +108,7 @@ extern int h_errno;
 #define SIZE_ICMP_HDR   8     /* from ip_icmp.h */
 #define MAX_PING_DATA   ( MAX_IP_PACKET - SIZE_IP_HDR - SIZE_ICMP_HDR )
 
-#define MAX_LOOP       100000
+#define MAX_GENERATE    100000  /* maximum number of hosts that -g can generate */
 
 /* sized so as to be like traditional ping */
 #define DEFAULT_PING_DATA_SIZE  56
@@ -234,7 +234,14 @@ HOST_ENTRY *ev_last;
 
 char *prog;
 int ident;                  /* our pid */
-int s;                      /* socket */
+#ifndef IPV6
+int socket4 = 0;
+int *allsocket[2] = { &socket4, NULL };
+#else
+int socket6 = 0;
+/* FIXME UNIFY */
+int *allsocket[2] = { &socket6, NULL };
+#endif
 unsigned int debugging = 0;
 
 /* times get *100 because all times are calculated in 10 usec units, not ms */
@@ -248,11 +255,11 @@ unsigned int count = 1;
 unsigned int trials;
 unsigned int report_interval = 0;
 unsigned int ttl = 0;
-int src_addr_present = 0;
-#ifndef IPV6
+int src_addr_set = 0;
 struct in_addr src_addr;
-#else
-struct in6_addr src_addr;
+#ifdef IPV6
+int src_addr6_set = 0;
+struct in6_addr src_addr6;
 #endif
 
 /* global stats */
@@ -349,7 +356,11 @@ int main( int argc, char **argv )
 
     prog = argv[0];
 
-    s = open_ping_socket(ping_data_size);
+#ifndef IPV6
+    socket4 = open_ping_socket_ipv4(ping_data_size);
+#else
+    socket6 = open_ping_socket_ipv6(ping_data_size);
+#endif
 
     if((uid = getuid())) {
         /* drop privileges */
@@ -372,8 +383,11 @@ int main( int argc, char **argv )
 #ifdef IP_MTU_DISCOVER
             {
                 int val = IP_PMTUDISC_DO;
-                if (setsockopt(s, IPPROTO_IP, IP_MTU_DISCOVER, &val, sizeof(val))) {
-                    perror("setsockopt IP_MTU_DISCOVER");
+                int *sp;
+                for(sp=allsocket[0]; *sp; sp++) {
+                    if (setsockopt(*sp, IPPROTO_IP, IP_MTU_DISCOVER, &val, sizeof(val))) {
+                        perror("setsockopt IP_MTU_DISCOVER");
+                    }
                 }
             }
 #else
@@ -524,19 +538,30 @@ int main( int argc, char **argv )
             break;
 
         case 'S':
-#ifndef IPV6
-            if( ! inet_pton( AF_INET, optarg, &src_addr ) )
-#else
-            if( ! inet_pton( AF_INET6, optarg, &src_addr ) )
+            if(inet_pton(AF_INET, optarg, &src_addr))
+            {
+                src_addr_set = 1;
+                break;
+            }
+#ifdef IPV6
+            if(inet_pton(AF_INET6, optarg, &src_addr6))
+            {
+                src_addr6_set = 1;
+                break;
+            }
 #endif
-                usage(1);
-            src_addr_present = 1;
+            usage(1);
             break;
 
         case 'I':
 #ifdef SO_BINDTODEVICE
-            if (setsockopt(s, SOL_SOCKET, SO_BINDTODEVICE, optarg, strlen(optarg))) {
-                perror("binding to specific interface (SO_BINTODEVICE)");
+            {
+                int *sp;
+                for(sp=allsocket[0]; *sp; sp++) {
+                    if (setsockopt(*sp, SOL_SOCKET, SO_BINDTODEVICE, optarg, strlen(optarg))) {
+                        perror("binding to specific interface (SO_BINTODEVICE)");
+                    }
+                }
             }
 #else
             printf( "%s: cant bind to a particular net interface since SO_BINDTODEVICE is not supported on your os.\n", argv[0] );
@@ -550,8 +575,11 @@ int main( int argc, char **argv )
 
         case 'O':
             if (sscanf(optarg,"%i",&tos)){
-                if ( setsockopt(s, IPPROTO_IP, IP_TOS, &tos, sizeof(tos))) {
-                    perror("setting type of service octet IP_TOS");
+                int *sp;
+                for(sp=allsocket[0]; *sp; sp++) {
+                    if ( setsockopt(*sp, IPPROTO_IP, IP_TOS, &tos, sizeof(tos))) {
+                        perror("setting type of service octet IP_TOS");
+                    }
                 }
             }
             else {
@@ -703,16 +731,22 @@ int main( int argc, char **argv )
 
     /* set the TTL, if the -H option was set (otherwise ttl will be = 0) */
     if(ttl > 0) {
-        if (setsockopt(s, IPPROTO_IP, IP_TTL,  &ttl, sizeof(ttl))) {
-            perror("setting time to live");
+        int *sp;
+        for(sp=allsocket[0]; *sp; sp++) {
+            if (setsockopt(*sp, IPPROTO_IP, IP_TTL,  &ttl, sizeof(ttl))) {
+                perror("setting time to live");
+            }
         }
     }
 
 #if HAVE_SO_TIMESTAMP
     {
         int opt = 1;
-        if (setsockopt(s, SOL_SOCKET, SO_TIMESTAMP,  &opt, sizeof(opt))) {
-            perror("setting SO_TIMESTAMP option");
+        int *sp;
+        for(sp=allsocket[0]; *sp; sp++) {
+            if (setsockopt(*sp, SOL_SOCKET, SO_TIMESTAMP,  &opt, sizeof(opt))) {
+                perror("setting SO_TIMESTAMP option");
+            }
         }
     }
 #endif
@@ -792,9 +826,15 @@ int main( int argc, char **argv )
         exit(num_noaddress ? 2 : 1);
     }
 
-    if(src_addr_present) {
-        socket_set_src_addr(s, src_addr);
+#ifndef IPV6
+    if(src_addr_set) {
+        socket_set_src_addr_ipv4(socket4, &src_addr);
     }
+#else
+    if(src_addr6_set) {
+        socket_set_src_addr_ipv6(socket6, &src_addr6);
+    }
+#endif
 
     /* allocate array to hold outstanding ping requests */
 
@@ -967,7 +1007,7 @@ void add_range(char *start, char *end)
     end_long = ntohl(((struct sockaddr_in *) addr_res->ai_addr)->sin_addr.s_addr);
     freeaddrinfo(addr_res);
 
-    if(end_long > start_long + MAX_LOOP) {
+    if(end_long > start_long + MAX_GENERATE) {
             fprintf(stderr, "Error: -g parameter generates too many addresses\n");
             exit(1);
     }
@@ -1007,7 +1047,11 @@ void main_loop()
 
                 /* Send the ping */
                 /*printf("Sending ping after %d ms\n", lt/100); */
-                send_ping(s, h);
+#ifndef IPV6
+                send_ping(socket4, h);
+#else
+                send_ping(socket6, h);
+#endif
 
                 /* Check what needs to be done next */
                 if(!loop_flag && !count_flag) {
@@ -1560,17 +1604,25 @@ int send_ping( int s, HOST_ENTRY *h )
     return(ret);
 }
 
-int socket_can_read(int socket, struct timeval *timeout)
+int socket_can_read(struct timeval *timeout)
 {
     int nfound;
     fd_set readset, writeset;
+    int s;
+
+/* FIXME UNIFY */
+#ifndef IPV6
+    s = socket4;
+#else
+    s = socket6;
+#endif
 
 select_again:
     FD_ZERO( &readset );
     FD_ZERO( &writeset );
     FD_SET( s, &readset );
 
-    nfound = select(socket + 1, &readset, &writeset, NULL, timeout);
+    nfound = select(s + 1, &readset, &writeset, NULL, timeout);
     if(nfound < 0) {
         if(errno == EINTR) {
             /* interrupted system call: redo the select */
@@ -1584,7 +1636,7 @@ select_again:
     if(nfound == 0)
         return 0;
     else
-        return 1;
+        return s;
 }
 
 int receive_packet(int socket,
@@ -1867,10 +1919,11 @@ int wait_for_reply(long wait_time)
     SEQMAP_VALUE *seqmap_value;
     unsigned short id;
     unsigned short seq;
+    struct timeval to;
+    int s = 0;
 
-    /* Wait for the socket to become ready */
+    /* Wait for a socket to become ready */
     if(wait_time) {
-        struct timeval to;
         if(wait_time < 100000) {
             to.tv_sec = 0;
             to.tv_usec = wait_time * 10;
@@ -1879,10 +1932,14 @@ int wait_for_reply(long wait_time)
             to.tv_sec = wait_time / 100000 ;
             to.tv_usec = (wait_time % 100000) * 10 ;
         }
-
-        if(!socket_can_read(s, &to)) {
-            return 0;   /* timeout */
-        }
+    }
+    else {
+        to.tv_sec = 0;
+        to.tv_usec = 0;
+    }
+    s = socket_can_read(&to);
+    if(s == 0) {
+        return 0;   /* timeout */
     }
 
     /* Receive packet */

diff --git a/src/fping.h b/src/fping.h
index e85c64d48ab922cb46250d4cc656f7a50f477607..0dedc61ee595dce30ebc50609139e36ef4f47c1f 100644 (file)
--- a/src/fping.h
+++ b/src/fping.h
@@ -7,12 +7,6 @@
 #include <sys/types.h>
 #include <netinet/in.h>
 
-#ifndef IPV6
-#define FPING_INADDR   struct in_addr
-#else
-#define FPING_INADDR   struct in6_addr
-#endif
-
 /* fping.c */
 void crash_and_burn( char *message );
 void errno_crash_and_burn( char *message );
@@ -20,9 +14,14 @@ int in_cksum( unsigned short *p, int n );
 int random_data_flag;
 
 /* socket.c */
+int  open_ping_socket_ipv4();
 int  open_ping_socket();
 void init_ping_buffer(size_t ping_data_size);
-void socket_set_src_addr(int s, FPING_INADDR src_addr);
+void socket_set_src_addr_ipv4(int s, struct in_addr *src_addr);
+#ifdef IPV6
+int  open_ping_socket_ipv6();
+void socket_set_src_addr_ipv6(int s, struct in6_addr *src_addr);
+#endif
 int  socket_sendto_ping(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq, uint16_t icmp_id);
 
 #endif

diff --git a/src/socket.c b/src/socket.c
index 782f0415eb32e484d6d4cce112a485222c40379c..dd692dcf5c41f1b7f747702027f17777bb9b8e7a 100644 (file)
--- a/src/socket.c
+++ b/src/socket.c
@@ -37,10 +37,12 @@ int open_ping_socket_ipv4();
 int open_ping_socket_ipv6();
 void init_ping_buffer_ipv4(size_t ping_data_size);
 void init_ping_buffer_ipv6(size_t ping_data_size);
-void socket_set_src_addr_ipv4(int s, FPING_INADDR src_addr);
-void socket_set_src_addr_ipv6(int s, FPING_INADDR src_addr);
+void socket_set_src_addr_ipv4(int s, struct in_addr *src_addr);
 int socket_sendto_ping_ipv4(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
+#ifdef IPV6
+void socket_set_src_addr_ipv6(int s, struct in6_addr *src_addr);
 int socket_sendto_ping_ipv6(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
+#endif
 
 int open_ping_socket()
 {
@@ -60,15 +62,6 @@ void init_ping_buffer(size_t ping_data_size)
 #endif
 }
 
-void socket_set_src_addr(int s, FPING_INADDR src_addr)
-{
-#ifndef IPV6
-    socket_set_src_addr_ipv4(s, src_addr);
-#else
-    socket_set_src_addr_ipv6(s, src_addr);
-#endif
-}
-
 int socket_sendto_ping(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr)
 {
 #ifndef IPV6

diff --git a/src/socket4.c b/src/socket4.c
index e298656f94f491c14c89bbad4ad76a7fb3fe867c..e97193fff16dd5b589b2ea47fa3841b7e27a3bf2 100644 (file)
--- a/src/socket4.c
+++ b/src/socket4.c
@@ -89,12 +89,12 @@ void init_ping_buffer_ipv4(size_t ping_data_size)
         crash_and_burn( "can't malloc ping packet" );
 }
 
-void socket_set_src_addr_ipv4(int s, FPING_INADDR src_addr)
+void socket_set_src_addr_ipv4(int s, struct in_addr *src_addr)
 {
     struct sockaddr_in sa;
     memset( &sa, 0, sizeof( sa ) );
     sa.sin_family = AF_INET;
-    sa.sin_addr = src_addr;
+    sa.sin_addr = *src_addr;
 
     if ( bind( s, (struct sockaddr *)&sa, sizeof( sa ) ) < 0 )
         errno_crash_and_burn( "cannot bind source address" );

diff --git a/src/socket6.c b/src/socket6.c
index 881021c96dc0956ea87e000a2736ecdbcd70929e..83738f85eb3a1ca45317ba237efb80b622fbc6f6 100644 (file)
--- a/src/socket6.c
+++ b/src/socket6.c
@@ -88,12 +88,12 @@ void init_ping_buffer_ipv6(size_t ping_data_size)
         crash_and_burn( "can't malloc ping packet" );
 }
 
-void socket_set_src_addr_ipv6(int s, FPING_INADDR src_addr)
+void socket_set_src_addr_ipv6(int s, struct in6_addr *src_addr)
 {
     struct sockaddr_in6 sa;
     memset( &sa, 0, sizeof( sa ) );
     sa.sin6_family = AF_INET6;
-    sa.sin6_addr = src_addr;
+    sa.sin6_addr = *src_addr;
 
     if ( bind( s, (struct sockaddr *)&sa, sizeof( sa ) ) < 0 )
         errno_crash_and_burn( "cannot bind source address" );