[monitoring-plugins] - delay set_source_ip() until address_family is ...

[Sven Nierlein]

    Module: monitoring-plugins
    Branch: master
    Commit: 986b2479465648c49a7eefc3fbf4df8860e3e4b7
    Author: ghciv6 <ghciv6@localhost>
 Committer: Sven Nierlein <s...@nierlein.org>
      Date: Mon Dec 20 22:39:57 2021 +0000
       URL: 
https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=986b247

- delay set_source_ip() until address_family is detected
- add a test to check '-s'

---

 plugins-root/check_icmp.c   | 5 ++++-
 plugins-root/t/check_icmp.t | 8 +++++++-
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/plugins-root/check_icmp.c b/plugins-root/check_icmp.c
index 01ae174..f97b0ed 100644
--- a/plugins-root/check_icmp.c
+++ b/plugins-root/check_icmp.c
@@ -410,6 +410,7 @@ main(int argc, char **argv)
 #ifdef SO_TIMESTAMP
        int on = 1;
 #endif
+       char *source_ip = NULL;
        char * opts_str = "vhVw:c:n:p:t:H:s:i:b:I:l:m:64";
 
        setlocale (LC_ALL, "");
@@ -542,7 +543,7 @@ main(int argc, char **argv)
                                }
                                break;
                        case 's': /* specify source IP address */
-                               set_source_ip(optarg);
+                               source_ip = optarg;
                                break;
                        case 'V': /* version */
                                print_revision (progname, NP_VERSION);
@@ -597,6 +598,8 @@ main(int argc, char **argv)
                sockets |= HAVE_ICMP;
        else icmp_sockerrno = errno;
 
+       if( source_ip )
+               set_source_ip(source_ip);
 
 #ifdef SO_TIMESTAMP
        if(setsockopt(icmp_sock, SOL_SOCKET, SO_TIMESTAMP, &on, sizeof(on)))
diff --git a/plugins-root/t/check_icmp.t b/plugins-root/t/check_icmp.t
index e043d4e..55edc31 100644
--- a/plugins-root/t/check_icmp.t
+++ b/plugins-root/t/check_icmp.t
@@ -12,7 +12,7 @@ my $allow_sudo = getTestParameter( "NP_ALLOW_SUDO",
        "no" );
 
 if ($allow_sudo eq "yes" or $> == 0) {
-       plan tests => 16;
+       plan tests => 18;
 } else {
        plan skip_all => "Need sudo to test check_icmp";
 }
@@ -83,3 +83,9 @@ $res = NPTest->testCmd(
 is( $res->return_code, 2, "One of two host nonresponsive - two required" );
 like( $res->output, $failureOutput, "Output OK" );
 
+$res = NPTest->testCmd(
+       "$sudo ./check_icmp -H $host_responsive -s 127.0.15.15 -w 10000ms,100% 
-c 10000ms,100% -n 1 -m 2"
+       );
+is( $res->return_code, 0, "IPv4 source_ip accepted" );
+like( $res->output, $successOutput, "Output OK" );
+