improve coverage

diff --git a/ci/build-2-install.sh b/ci/build-2-install.sh
index 247eaefb8d131d0c69245423e140a38e5162f13f..bb94f32aac8e6389c164be63936536d5429d94a1 100755 (executable)
--- a/ci/build-2-install.sh
+++ b/ci/build-2-install.sh
@@ -10,7 +10,7 @@ fi
 
 autoreconf -i
 ./configure --enable-ipv4 --enable-ipv6 --prefix=/opt/fping
-make CFLAGS="-g -fprofile-arcs -ftest-coverage"
+make CFLAGS="-g -O0 -fprofile-arcs -ftest-coverage"
 ## setcap currently doesn't work anymore on travis-ci
 #sudo setcap cap_net_raw+ep src/fping
 ## setcap debugging:

diff --git a/ci/test-08-options-n-q.pl b/ci/test-08-options-n-q.pl
index c490c19e8b45cc81aeb51bbec12b88f910564832..d14173f2fa7b621722ec4389d455eb23f23664d3 100755 (executable)
--- a/ci/test-08-options-n-q.pl
+++ b/ci/test-08-options-n-q.pl
@@ -1,6 +1,6 @@
 #!/usr/bin/perl -w
 
-use Test::Command tests => 12;
+use Test::Command tests => 18;
 
 #  -n         show targets by name (-d is equivalent)
 #  -O n       set the type of service (tos) flag on the ICMP packets
@@ -11,6 +11,22 @@ use Test::Command tests => 12;
 
 # fping -n -> test-14-internet-hosts
 
+# fping -d -n
+{
+my $cmd = Test::Command->new(cmd => "fping -d -n 127.0.0.1");
+$cmd->exit_is_num(1);
+$cmd->stdout_is_eq("");
+$cmd->stderr_is_eq("fping: use either one of -d or -n\n");
+}
+
+# fping -n -d
+{
+my $cmd = Test::Command->new(cmd => "fping -n -d 127.0.0.1");
+$cmd->exit_is_num(1);
+$cmd->stdout_is_eq("");
+$cmd->stderr_is_eq("fping: use either one of -d or -n\n");
+}
+
 # fping -o
 {
 my $cmd = Test::Command->new(cmd => "fping -t100 -p 100 -o -c 5 8.8.8.7");

diff --git a/ci/test-09-option-r-t.pl b/ci/test-09-option-r-t.pl
index eae162dcab1e6af03fe167562a1c7b416f483835..2ea7a68d2a03b06307f52bfd53e545772e8b52b8 100755 (executable)
--- a/ci/test-09-option-r-t.pl
+++ b/ci/test-09-option-r-t.pl
@@ -1,6 +1,6 @@
 #!/usr/bin/perl -w
 
-use Test::Command tests => 18;
+use Test::Command tests => 21;
 use Test::More;
 
 #  -R         random bytes
@@ -95,4 +95,12 @@ SKIP: {
     $cmd->stderr_is_eq("");
 }
 
+# fping -S
+{
+my $cmd = Test::Command->new(cmd => "fping -S bla");
+$cmd->exit_is_num(1);
+$cmd->stdout_is_eq("");
+$cmd->stderr_is_eq("fping: can't parse source address: bla\n");
+}
+
 # fping -t tested in test-4-options-a-b.pl

diff --git a/src/fping.c b/src/fping.c
index 945eb09447a66721db091af57027a9e413e244db..ee2da5fd2f362e44be4b8c372c2e88e85cb4cae7 100644 (file)
--- a/src/fping.c
+++ b/src/fping.c
@@ -613,8 +613,8 @@ int main(int argc, char** argv)
                 break;
             }
 #endif
-            usage(1);
-            break;
+            fprintf(stderr, "%s: can't parse source address: %s\n", prog, optparse_state.optarg);
+            exit(1);
 
         case 'I':
 #ifdef SO_BINDTODEVICE