further unification work

diff --git a/configure.ac b/configure.ac
index 9e60f88e2e5d69e44d0ca6625517a47640197ccc..fdf2808f6311899cd1ceef71b1b982de26b37431 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -29,7 +29,7 @@ if test x$ipv4 = xfalse && test x$ipv6 = xfalse; then
 fi
 
 AC_CANONICAL_TARGET
-AM_INIT_AUTOMAKE([foreign])
+AM_INIT_AUTOMAKE([-Wall -Werror foreign])
 AM_MAINTAINER_MODE
 
 AC_CONFIG_HEADERS([config.h])

diff --git a/src/fping.c b/src/fping.c
index 7f4131415cc8cb2d978c06ee50f33bf688195bc3..f3bd4658bed778ae97c1a675ab0207a4080b1ff1 100644 (file)
--- a/src/fping.c
+++ b/src/fping.c
@@ -365,7 +365,7 @@ int main( int argc, char **argv )
     int tos = 0; 
     HOST_ENTRY *cursor;
 
-    s = open_ping_socket();
+    s = open_ping_socket(ping_data_size);
 
     if((uid = getuid())) {
         /* drop privileges */
@@ -1405,7 +1405,7 @@ int send_ping( int s, HOST_ENTRY *h )
         printf( "sending [%d] to %s\n", h->num_sent, h->host );
 #endif /* DEBUG || _DEBUG */
 
-    n = socket_sendto_ping(s, (struct sockaddr *) &h->saddr, h->saddr_len, ping_pkt_size, myseq, ident);
+    n = socket_sendto_ping(s, (struct sockaddr *) &h->saddr, h->saddr_len, myseq, ident);
 
     if(
         (n < 0 || n != ping_pkt_size)

diff --git a/src/fping.h b/src/fping.h
index d42691da3fc861466da2646450014893702b709f..29c183caaf328a9ab791916d0477cc0d5a3ee8ca 100644 (file)
--- a/src/fping.h
+++ b/src/fping.h
@@ -21,8 +21,8 @@ void errno_crash_and_burn( char *message );
 int in_cksum( unsigned short *p, int n );
 
 /* socket.c */
-int  open_ping_socket();
+int  open_ping_socket(size_t ping_data_size);
 void socket_set_src_addr(int s, FPING_INADDR src_addr);
-int  socket_sendto_ping(int s, struct sockaddr *saddr, socklen_t saddr_len, size_t ping_pkg_size, uint16_t icmp_seq, uint16_t icmp_id);
+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 54ee1ec8df9b8ca969a9322b8bec421eafa34559..988afdb5f4a8edfbf7a2b80c1c19f92b53df443c 100644 (file)
--- a/src/socket.c
+++ b/src/socket.c
@@ -37,15 +37,15 @@ int open_ping_socket_ipv4();
 int open_ping_socket_ipv6();
 void socket_set_src_addr_ipv4(int s, FPING_INADDR src_addr);
 void socket_set_src_addr_ipv6(int s, FPING_INADDR src_addr);
-int socket_sendto_ping_ipv4(int s, struct sockaddr *saddr, socklen_t saddr_len, size_t ping_pkg_size, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
-int socket_sendto_ping_ipv6(int s, struct sockaddr *saddr, socklen_t saddr_len, size_t ping_pkg_size, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
+int socket_sendto_ping_ipv4(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
+int socket_sendto_ping_ipv6(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr);
 
-int open_ping_socket()
+int open_ping_socket(size_t ping_data_size)
 {
 #ifndef IPV6
-    return open_ping_socket_ipv4();
+    return open_ping_socket_ipv4(ping_data_size);
 #else
-    return open_ping_socket_ipv6();
+    return open_ping_socket_ipv6(ping_data_size);
 #endif
 }
 
@@ -58,11 +58,11 @@ void socket_set_src_addr(int s, FPING_INADDR src_addr)
 #endif
 }
 
-int socket_sendto_ping(int s, struct sockaddr *saddr, socklen_t saddr_len, size_t ping_pkt_size, uint16_t icmp_seq_nr, uint16_t icmp_id_nr)
+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
-    return socket_sendto_ping_ipv4(s, saddr, saddr_len, ping_pkt_size, icmp_seq_nr, icmp_id_nr);
+    return socket_sendto_ping_ipv4(s, saddr, saddr_len, icmp_seq_nr, icmp_id_nr);
 #else
-    return socket_sendto_ping_ipv6(s, saddr, saddr_len, ping_pkt_size, icmp_seq_nr, icmp_id_nr);
+    return socket_sendto_ping_ipv6(s, saddr, saddr_len, icmp_seq_nr, icmp_id_nr);
 #endif
 }

diff --git a/src/socket4.c b/src/socket4.c
index 51ba9bdf7ebd471d73e20645ff10d0c9484f061f..318f7feb218a9309097cbeed7503b5f6d46ed12f 100644 (file)
--- a/src/socket4.c
+++ b/src/socket4.c
@@ -84,7 +84,7 @@ void socket_set_src_addr_ipv4(int s, FPING_INADDR src_addr)
         errno_crash_and_burn( "cannot bind source address" );
 }
 
-int socket_sendto_ping_ipv4(int s, struct sockaddr *saddr, socklen_t saddr_len, size_t ping_pkt_size, uint16_t icmp_seq_nr, uint16_t icmp_id_nr)
+int socket_sendto_ping_ipv4(int s, struct sockaddr *saddr, socklen_t saddr_len, uint16_t icmp_seq_nr, uint16_t icmp_id_nr)
 {
     struct icmp *icp;
     int n;

diff --git a/src/socket6.c b/src/socket6.c
index 361612a5c1751a6234c333713740af3bb8d6697c..c2147edf6757a838388f1ad0e978cf8ec0ea6a7d 100644 (file)
--- a/src/socket6.c
+++ b/src/socket6.c
@@ -92,10 +92,6 @@ int socket_sendto_ping_ipv6(int s, struct sockaddr *saddr, socklen_t saddr_len,
 
     // FIXME: randomization
     icp = (struct icmp6_hdr *) ping_buffer;
-    if(!icp) {
-        crash_and_burn("can't malloc ping packet");
-    }
-
     icp->icmp6_type  = ICMP6_ECHO_REQUEST;
     icp->icmp6_code  = 0;
     icp->icmp6_seq   = htons(icmp_seq_nr);