3 files changed, 19 insertions, 5 deletions

diff --git a/NEWS b/NEWS
index 2686cd85..f4e662aa 100644
--- a/NEWS
+++ b/NEWS
@@ -11,6 +11,7 @@ This file documents the major additions and syntax changes between releases.
         Fixed dependency issue on libtap when ./configure --enable-libtap used. Warning: will install libtap
         Fixed segfault in extra-opts under some circumstance when reading multiple sections
         Fix long options parsing in check_tcp
+        check_icmp now reports min and max round trip time perfdata (Steve Rader)
 
 1.4.13 25th Sept 2008
         Fix Debian bug #460097: check_http --max-age broken (Hilko Bengen)
diff --git a/THANKS.in b/THANKS.in
index 80fa47a3..b042f9e7 100644
--- a/THANKS.in
+++ b/THANKS.in
@@ -240,3 +240,4 @@ Sven Nierlein
 Erik Wasser
 Tilman Koschnick
 Olivier 'Babar' Raginel
+Steve Rader
diff --git a/plugins-root/check_icmp.c b/plugins-root/check_icmp.c
index dff3abb2..ff6c73b3 100644
--- a/plugins-root/check_icmp.c
+++ b/plugins-root/check_icmp.c
@@ -106,6 +106,9 @@ const char *email = "nagiosplug-devel@lists.sourceforge.net";
 # define ICMP_UNREACH_PRECEDENCE_CUTOFF 15
 #endif
 
+#ifndef DBL_MAX
+# define DBL_MAX 9.9999999999e999
+#endif
 
 typedef unsigned short range_t;  /* type for get_range() -- unimplemented */
 
@@ -120,6 +123,8 @@ typedef struct rta_host {
         unsigned char icmp_type, icmp_code; /* type and code from errors */
         unsigned short flags;        /* control/status flags */
         double rta;                  /* measured RTA */
+        double rtmax;                /* max rtt */
+        double rtmin;                /* min rtt */
         unsigned char pl;            /* measured packet loss */
         struct rta_host *next;       /* linked list */
 } rta_host;
@@ -782,11 +787,15 @@ wait_for_reply(int sock, u_int t)
                 host->time_waited += tdiff;
                 host->icmp_recv++;
                 icmp_recv++;
+                if (tdiff > host->rtmax)
+                        host->rtmax = tdiff;
+                if (tdiff < host->rtmin)
+                        host->rtmin = tdiff;
 
                 if(debug) {
-                        printf("%0.3f ms rtt from %s, outgoing ttl: %u, incoming ttl: %u\n",
+                        printf("%0.3f ms rtt from %s, outgoing ttl: %u, incoming ttl: %u, max: %0.3f, min: %0.3f\n",
                                    (float)tdiff / 1000, inet_ntoa(resp_addr.sin_addr),
-                                   ttl, ip->ip_ttl);
+                                   ttl, ip->ip_ttl, (float)host->rtmax / 1000, (float)host->rtmin / 1000);
                 }
 
                 /* if we're in hostcheck mode, exit with limited printouts */
@@ -993,11 +1002,12 @@ finish(int sig)
         host = list;
         while(host) {
                 if(debug) puts("");
-                printf("%srta=%0.3fms;%0.3f;%0.3f;0; %spl=%u%%;%u;%u;; ",
+                printf("%srta=%0.3fms;%0.3f;%0.3f;0; %spl=%u%%;%u;%u;; %srtmax=%0.3fms;;;; %srtmin=%0.3fms;;;; ",
                            (targets > 1) ? host->name : "",
                            host->rta / 1000, (float)warn.rta / 1000, (float)crit.rta / 1000,
-                           (targets > 1) ? host->name : "",
-                           host->pl, warn.pl, crit.pl);
+                           (targets > 1) ? host->name : "", host->pl, warn.pl, crit.pl,
+                           (targets > 1) ? host->name : "", (float)host->rtmax / 1000,
+                           (targets > 1) ? host->name : "", (host->rtmin < DBL_MAX) ? (float)host->rtmin / 1000 : (float)0);
 
                 host = host->next;
         }
@@ -1074,6 +1084,8 @@ add_target_ip(char *arg, struct in_addr *in)
         host->saddr_in.sin_family = AF_INET;
         host->saddr_in.sin_addr.s_addr = in->s_addr;
 
+        host->rtmin = DBL_MAX;
+
         if(!list) list = cursor = host;
         else cursor->next = host;