1 files changed, 233 insertions, 143 deletions

diff --git a/plugins/check_fping.c b/plugins/check_fping.c
index c1d03ece..6160c2cb 100644
--- a/plugins/check_fping.c
+++ b/plugins/check_fping.c
@@ -38,52 +38,30 @@ const char *email = "devel@monitoring-plugins.org";
 #include "netutils.h"
 #include "utils.h"
 #include <stdbool.h>
+#include "check_fping.d/config.h"
+#include "states.h"
 
 enum {
-        PACKET_COUNT = 1,
-        PACKET_SIZE = 56,
         PL = 0,
         RTA = 1
 };
 
-static int textscan(char *buf);
-static int process_arguments(int /*argc*/, char ** /*argv*/);
+static mp_state_enum textscan(char *buf, const char * /*server_name*/, bool /*crta_p*/,
+                                                          double /*crta*/, bool /*wrta_p*/, double /*wrta*/, bool /*cpl_p*/,
+                                                          int /*cpl*/, bool /*wpl_p*/, int /*wpl*/, bool /*alive_p*/);
+
+typedef struct {
+        int errorcode;
+        check_fping_config config;
+} check_fping_config_wrapper;
+static check_fping_config_wrapper process_arguments(int /*argc*/, char ** /*argv*/);
 static int get_threshold(char *arg, char *rv[2]);
 static void print_help(void);
 void print_usage(void);
 
-static char *server_name = NULL;
-static char *sourceip = NULL;
-static char *sourceif = NULL;
-static int packet_size = PACKET_SIZE;
-static int packet_count = PACKET_COUNT;
-static int target_timeout = 0;
-static int packet_interval = 0;
 static bool verbose = false;
-static bool dontfrag = false;
-static bool randomize_packet_data = false;
-static int cpl;
-static int wpl;
-static double crta;
-static double wrta;
-static bool cpl_p = false;
-static bool wpl_p = false;
-static bool alive_p = false;
-static bool crta_p = false;
-static bool wrta_p = false;
 
 int main(int argc, char **argv) {
-        /* normally should be  int result = STATE_UNKNOWN; */
-
-        int status = STATE_UNKNOWN;
-        int result = 0;
-        char *fping_prog = NULL;
-        char *server = NULL;
-        char *command_line = NULL;
-        char *input_buffer = NULL;
-        char *option_string = "";
-        input_buffer = malloc(MAX_INPUT_BUFFER);
-
         setlocale(LC_ALL, "");
         bindtextdomain(PACKAGE, LOCALEDIR);
         textdomain(PACKAGE);
@@ -91,39 +69,81 @@ int main(int argc, char **argv) {
         /* Parse extra opts if any */
         argv = np_extra_opts(&argc, argv, progname);
 
-        if (process_arguments(argc, argv) == ERROR)
+        check_fping_config_wrapper tmp_config = process_arguments(argc, argv);
+        if (tmp_config.errorcode == ERROR) {
                 usage4(_("Could not parse arguments"));
+        }
+
+        const check_fping_config config = tmp_config.config;
+
+        char *server = NULL;
+        server = strscpy(server, config.server_name);
 
-        server = strscpy(server, server_name);
+        char *option_string = "";
+        char *fping_prog = NULL;
+
+        /* First determine if the target is dualstack or ipv6 only. */
+        bool server_is_inet6_addr = is_inet6_addr(server);
+
+        /*
+         * If the user requested -6 OR the user made no assertion and the address is v6 or dualstack
+         *   -> we use ipv6
+         * If the user requested -4 OR the user made no assertion and the address is v4 ONLY
+         *   -> we use ipv4
+         */
+        if (address_family == AF_INET6 || (address_family == AF_UNSPEC && server_is_inet6_addr)) {
+                xasprintf(&option_string, "%s-6 ", option_string);
+        } else {
+                xasprintf(&option_string, "%s-4 ", option_string);
+        }
+        fping_prog = strdup(PATH_TO_FPING);
 
         /* compose the command */
-        if (target_timeout)
-                xasprintf(&option_string, "%s-t %d ", option_string, target_timeout);
-        if (packet_interval)
-                xasprintf(&option_string, "%s-p %d ", option_string, packet_interval);
-        if (sourceip)
-                xasprintf(&option_string, "%s-S %s ", option_string, sourceip);
-        if (sourceif)
-                xasprintf(&option_string, "%s-I %s ", option_string, sourceif);
-        if (dontfrag)
+        if (config.target_timeout) {
+                xasprintf(&option_string, "%s-t %d ", option_string, config.target_timeout);
+        }
+        if (config.packet_interval) {
+                xasprintf(&option_string, "%s-p %d ", option_string, config.packet_interval);
+        }
+        if (config.sourceip) {
+                xasprintf(&option_string, "%s-S %s ", option_string, config.sourceip);
+        }
+        if (config.sourceif) {
+                xasprintf(&option_string, "%s-I %s ", option_string, config.sourceif);
+        }
+        if (config.dontfrag) {
                 xasprintf(&option_string, "%s-M ", option_string);
-        if (randomize_packet_data)
+        }
+        if (config.randomize_packet_data) {
                 xasprintf(&option_string, "%s-R ", option_string);
+        }
 
+        if (config.fwmark_set) {
+                xasprintf(&option_string, "%s--fwmark %u ", option_string, config.fwmark);
+        }
 
-#ifdef PATH_TO_FPING6
-        if (address_family != AF_INET && is_inet6_addr(server))
-                fping_prog = strdup(PATH_TO_FPING6);
-        else
-                fping_prog = strdup(PATH_TO_FPING);
-#else
-        fping_prog = strdup(PATH_TO_FPING);
-#endif
+        if (config.icmp_timestamp) {
+                xasprintf(&option_string, "%s--icmp-timestamp ", option_string);
+        }
 
-        xasprintf(&command_line, "%s %s-b %d -c %d %s", fping_prog, option_string, packet_size, packet_count, server);
+        if (config.check_source) {
+                xasprintf(&option_string, "%s--check-source ", option_string);
+        }
+
+        char *command_line = NULL;
 
-        if (verbose)
+        if (config.icmp_timestamp) {
+                // no packet size settable for ICMP timestamp
+                xasprintf(&command_line, "%s %s -c %d %s", fping_prog, option_string, config.packet_count,
+                                  server);
+        } else {
+                xasprintf(&command_line, "%s %s-b %d -c %d %s", fping_prog, option_string,
+                                  config.packet_size, config.packet_count, server);
+        }
+
+        if (verbose) {
                 printf("%s\n", command_line);
+        }
 
         /* run the command */
         child_process = spopen(command_line);
@@ -137,23 +157,31 @@ int main(int argc, char **argv) {
                 printf(_("Could not open stderr for %s\n"), command_line);
         }
 
+        char *input_buffer = malloc(MAX_INPUT_BUFFER);
+        mp_state_enum status = STATE_UNKNOWN;
         while (fgets(input_buffer, MAX_INPUT_BUFFER - 1, child_process)) {
-                if (verbose)
+                if (verbose) {
                         printf("%s", input_buffer);
-                status = max_state(status, textscan(input_buffer));
+                }
+                status = max_state(status, textscan(input_buffer, config.server_name, config.crta_p,
+                                                                                        config.crta, config.wrta_p, config.wrta, config.cpl_p,
+                                                                                        config.cpl, config.wpl_p, config.wpl, config.alive_p));
         }
 
         /* If we get anything on STDERR, at least set warning */
         while (fgets(input_buffer, MAX_INPUT_BUFFER - 1, child_stderr)) {
                 status = max_state(status, STATE_WARNING);
-                if (verbose)
+                if (verbose) {
                         printf("%s", input_buffer);
-                status = max_state(status, textscan(input_buffer));
+                }
+                status = max_state(status, textscan(input_buffer, config.server_name, config.crta_p,
+                                                                                        config.crta, config.wrta_p, config.wrta, config.cpl_p,
+                                                                                        config.cpl, config.wpl_p, config.wpl, config.alive_p));
         }
         (void)fclose(child_stderr);
 
         /* close the pipe */
-        result = spclose(child_process);
+        int result = spclose(child_process);
         if (result) {
                 /* need to use max_state not max */
                 status = max_state(status, STATE_WARNING);
@@ -172,21 +200,17 @@ int main(int argc, char **argv) {
                 }
         }
 
-        printf("FPING %s - %s\n", state_text(status), server_name);
+        printf("FPING %s - %s\n", state_text(status), config.server_name);
 
         return status;
 }
 
-int textscan(char *buf) {
-        char *rtastr = NULL;
-        char *losstr = NULL;
-        char *xmtstr = NULL;
-        double loss;
-        double rta;
-        double xmt;
-        int status = STATE_UNKNOWN;
-
+mp_state_enum textscan(char *buf, const char *server_name, bool crta_p, double crta, bool wrta_p,
+                                           double wrta, bool cpl_p, int cpl, bool wpl_p, int wpl, bool alive_p) {
         /* stops testing after the first successful reply. */
+        double rta;
+        double loss;
+        char *rtastr = NULL;
         if (alive_p && strstr(buf, "avg, 0% loss)")) {
                 rtastr = strstr(buf, "ms (");
                 rtastr = 1 + index(rtastr, '(');
@@ -195,9 +219,14 @@ int textscan(char *buf) {
                 die(STATE_OK, _("FPING %s - %s (rta=%f ms)|%s\n"), state_text(STATE_OK), server_name, rta,
                         /* No loss since we only waited for the first reply
                         perfdata ("loss", (long int)loss, "%", wpl_p, wpl, cpl_p, cpl, true, 0, true, 100), */
-                        fperfdata("rta", rta / 1.0e3, "s", wrta_p, wrta / 1.0e3, crta_p, crta / 1.0e3, true, 0, false, 0));
+                        fperfdata("rta", rta / 1.0e3, "s", wrta_p, wrta / 1.0e3, crta_p, crta / 1.0e3, true, 0,
+                                          false, 0));
         }
 
+        mp_state_enum status = STATE_UNKNOWN;
+        char *xmtstr = NULL;
+        double xmt;
+        char *losstr = NULL;
         if (strstr(buf, "not found")) {
                 die(STATE_CRITICAL, _("FPING UNKNOWN - %s not found\n"), server_name);
 
@@ -221,19 +250,22 @@ int textscan(char *buf) {
                 rtastr = 1 + index(rtastr, '/');
                 loss = strtod(losstr, NULL);
                 rta = strtod(rtastr, NULL);
-                if (cpl_p && loss > cpl)
+                if (cpl_p && loss > cpl) {
                         status = STATE_CRITICAL;
-                else if (crta_p && rta > crta)
+                } else if (crta_p && rta > crta) {
                         status = STATE_CRITICAL;
-                else if (wpl_p && loss > wpl)
+                } else if (wpl_p && loss > wpl) {
                         status = STATE_WARNING;
-                else if (wrta_p && rta > wrta)
+                } else if (wrta_p && rta > wrta) {
                         status = STATE_WARNING;
-                else
+                } else {
                         status = STATE_OK;
-                die(status, _("FPING %s - %s (loss=%.0f%%, rta=%f ms)|%s %s\n"), state_text(status), server_name, loss, rta,
-                        perfdata("loss", (long int)loss, "%", wpl_p, wpl, cpl_p, cpl, true, 0, true, 100),
-                        fperfdata("rta", rta / 1.0e3, "s", wrta_p, wrta / 1.0e3, crta_p, crta / 1.0e3, true, 0, false, 0));
+                }
+                die(status, _("FPING %s - %s (loss=%.0f%%, rta=%f ms)|%s %s\n"), state_text(status),
+                        server_name, loss, rta,
+                        perfdata("loss", (long int)loss, "%", wpl_p, wpl, cpl_p, cpl, false, 0, false, 0),
+                        fperfdata("rta", rta / 1.0e3, "s", wrta_p, wrta / 1.0e3, crta_p, crta / 1.0e3, true, 0,
+                                          false, 0));
 
         } else if (strstr(buf, "xmt/rcv/%loss")) {
                 /* no min/max/avg if host was unreachable in fping v2.2.b1 */
@@ -241,22 +273,24 @@ int textscan(char *buf) {
                 losstr = strstr(buf, "=");
                 xmtstr = 1 + losstr;
                 xmt = strtod(xmtstr, NULL);
-                if (xmt == 0)
+                if (xmt == 0) {
                         die(STATE_CRITICAL, _("FPING CRITICAL - %s is down\n"), server_name);
+                }
                 losstr = 1 + strstr(losstr, "/");
                 losstr = 1 + strstr(losstr, "/");
                 loss = strtod(losstr, NULL);
-                if (atoi(losstr) == 100)
+                if (atoi(losstr) == 100) {
                         status = STATE_CRITICAL;
-                else if (cpl_p && loss > cpl)
+                } else if (cpl_p && loss > cpl) {
                         status = STATE_CRITICAL;
-                else if (wpl_p && loss > wpl)
+                } else if (wpl_p && loss > wpl) {
                         status = STATE_WARNING;
-                else
+                } else {
                         status = STATE_OK;
+                }
                 /* loss=%.0f%%;%d;%d;0;100 */
                 die(status, _("FPING %s - %s (loss=%.0f%% )|%s\n"), state_text(status), server_name, loss,
-                        perfdata("loss", (long int)loss, "%", wpl_p, wpl, cpl_p, cpl, true, 0, true, 100));
+                        perfdata("loss", (long int)loss, "%", wpl_p, wpl, cpl_p, cpl, false, 0, false, 0));
 
         } else {
                 status = max_state(status, STATE_WARNING);
@@ -266,11 +300,12 @@ int textscan(char *buf) {
 }
 
 /* process command-line arguments */
-int process_arguments(int argc, char **argv) {
-        int c;
-        char *rv[2];
-
-        int option = 0;
+check_fping_config_wrapper process_arguments(int argc, char **argv) {
+        enum {
+                FWMARK_OPT = CHAR_MAX + 1,
+                ICMP_TIMESTAMP_OPT,
+                CHECK_SOURCE_OPT,
+        };
         static struct option longopts[] = {{"hostname", required_argument, 0, 'H'},
                                                                            {"sourceip", required_argument, 0, 'S'},
                                                                            {"sourceif", required_argument, 0, 'I'},
@@ -288,32 +323,53 @@ int process_arguments(int argc, char **argv) {
                                                                            {"use-ipv6", no_argument, 0, '6'},
                                                                            {"dontfrag", no_argument, 0, 'M'},
                                                                            {"random", no_argument, 0, 'R'},
+#ifdef FPING_VERSION_5_2_OR_HIGHER
+                                                                           // only available with fping version >= 5.2
+                                                                           {"fwmark", required_argument, NULL, FWMARK_OPT},
+#       ifdef FPING_VERSION_5_3_OR_HIGHER
+                                                                           // only available with fping version >= 5.3
+                                                                           {"icmp-timestamp", no_argument, NULL, ICMP_TIMESTAMP_OPT},
+                                                                           {"check-source", no_argument, NULL, CHECK_SOURCE_OPT},
+#       endif
+#endif
                                                                            {0, 0, 0, 0}};
 
+        char *rv[2];
         rv[PL] = NULL;
         rv[RTA] = NULL;
 
-        if (argc < 2)
-                return ERROR;
+        int option = 0;
+
+        check_fping_config_wrapper result = {
+                .errorcode = OK,
+                .config = check_fping_config_init(),
+        };
+
+        if (argc < 2) {
+                result.errorcode = ERROR;
+                return result;
+        }
 
         if (!is_option(argv[1])) {
-                server_name = argv[1];
+                result.config.server_name = argv[1];
                 argv[1] = argv[0];
                 argv = &argv[1];
                 argc--;
         }
 
-        while (1) {
-                c = getopt_long(argc, argv, "+hVvaH:S:c:w:b:n:T:i:I:M:R:46", longopts, &option);
+        while (true) {
+                int option_index =
+                        getopt_long(argc, argv, "+hVvaH:S:c:w:b:n:T:i:I:M:R:46", longopts, &option);
 
-                if (c == -1 || c == EOF || c == 1)
+                if (option_index == -1 || option_index == EOF || option_index == 1) {
                         break;
+                }
 
-                switch (c) {
+                switch (option_index) {
                 case '?': /* print short usage statement if args not parsable */
                         usage5();
                 case 'a': /* host alive mode */
-                        alive_p = true;
+                        result.config.alive_p = true;
                         break;
                 case 'h': /* help */
                         print_help();
@@ -325,109 +381,128 @@ int process_arguments(int argc, char **argv) {
                         verbose = true;
                         break;
                 case 'H': /* hostname */
-                        if (is_host(optarg) == false) {
+                        if (!is_host(optarg)) {
                                 usage2(_("Invalid hostname/address"), optarg);
                         }
-                        server_name = strscpy(server_name, optarg);
+                        result.config.server_name = optarg;
                         break;
                 case 'S': /* sourceip */
-                        if (is_host(optarg) == false) {
+                        if (!is_host(optarg)) {
                                 usage2(_("Invalid hostname/address"), optarg);
                         }
-                        sourceip = strscpy(sourceip, optarg);
+                        result.config.sourceip = optarg;
                         break;
                 case 'I': /* sourceip */
-                        sourceif = strscpy(sourceif, optarg);
+                        result.config.sourceif = optarg;
                         break;
                 case '4': /* IPv4 only */
                         address_family = AF_INET;
                         break;
                 case '6': /* IPv6 only */
-#ifdef USE_IPV6
                         address_family = AF_INET6;
-#else
-                        usage(_("IPv6 support not available\n"));
-#endif
                         break;
                 case 'c':
                         get_threshold(optarg, rv);
                         if (rv[RTA]) {
-                                crta = strtod(rv[RTA], NULL);
-                                crta_p = true;
+                                result.config.crta = strtod(rv[RTA], NULL);
+                                result.config.crta_p = true;
                                 rv[RTA] = NULL;
                         }
                         if (rv[PL]) {
-                                cpl = atoi(rv[PL]);
-                                cpl_p = true;
+                                result.config.cpl = atoi(rv[PL]);
+                                result.config.cpl_p = true;
                                 rv[PL] = NULL;
                         }
                         break;
                 case 'w':
                         get_threshold(optarg, rv);
                         if (rv[RTA]) {
-                                wrta = strtod(rv[RTA], NULL);
-                                wrta_p = true;
+                                result.config.wrta = strtod(rv[RTA], NULL);
+                                result.config.wrta_p = true;
                                 rv[RTA] = NULL;
                         }
                         if (rv[PL]) {
-                                wpl = atoi(rv[PL]);
-                                wpl_p = true;
+                                result.config.wpl = atoi(rv[PL]);
+                                result.config.wpl_p = true;
                                 rv[PL] = NULL;
                         }
                         break;
                 case 'b': /* bytes per packet */
-                        if (is_intpos(optarg))
-                                packet_size = atoi(optarg);
-                        else
+                        if (is_intpos(optarg)) {
+                                result.config.packet_size = atoi(optarg);
+                        } else {
                                 usage(_("Packet size must be a positive integer"));
+                        }
                         break;
                 case 'n': /* number of packets */
-                        if (is_intpos(optarg))
-                                packet_count = atoi(optarg);
-                        else
+                        if (is_intpos(optarg)) {
+                                result.config.packet_count = atoi(optarg);
+                        } else {
                                 usage(_("Packet count must be a positive integer"));
+                        }
                         break;
                 case 'T': /* timeout in msec */
-                        if (is_intpos(optarg))
-                                target_timeout = atoi(optarg);
-                        else
+                        if (is_intpos(optarg)) {
+                                result.config.target_timeout = atoi(optarg);
+                        } else {
                                 usage(_("Target timeout must be a positive integer"));
+                        }
                         break;
                 case 'i': /* interval in msec */
-                        if (is_intpos(optarg))
-                                packet_interval = atoi(optarg);
-                        else
+                        if (is_intpos(optarg)) {
+                                result.config.packet_interval = atoi(optarg);
+                        } else {
                                 usage(_("Interval must be a positive integer"));
+                        }
                         break;
                 case 'R':
-                        randomize_packet_data = true;
+                        result.config.randomize_packet_data = true;
                         break;
                 case 'M':
-                        dontfrag = true;
+                        result.config.dontfrag = true;
+                        break;
+                case FWMARK_OPT:
+                        if (is_intpos(optarg)) {
+                                result.config.fwmark = (unsigned int)atol(optarg);
+                                result.config.fwmark_set = true;
+                        } else {
+                                usage(_("fwmark must be a positive integer"));
+                        }
+                        break;
+                case ICMP_TIMESTAMP_OPT:
+                        result.config.icmp_timestamp = true;
+                        break;
+                case CHECK_SOURCE_OPT:
+                        result.config.check_source = true;
                         break;
                 }
         }
 
-        if (server_name == NULL)
+        if (result.config.server_name == NULL) {
                 usage4(_("Hostname was not supplied"));
+        }
 
-        return OK;
+        return result;
 }
 
 int get_threshold(char *arg, char *rv[2]) {
-        char *arg1 = NULL;
         char *arg2 = NULL;
 
-        arg1 = strscpy(arg1, arg);
-        if (strpbrk(arg1, ",:"))
+        char *arg1 = strdup(arg);
+        if (strpbrk(arg1, ",:")) {
                 arg2 = 1 + strpbrk(arg1, ",:");
+        }
 
         if (arg2) {
                 arg1[strcspn(arg1, ",:")] = 0;
-                if (strstr(arg1, "%") && strstr(arg2, "%"))
-                        die(STATE_UNKNOWN, _("%s: Only one threshold may be packet loss (%s)\n"), progname, arg);
-                if (!strstr(arg1, "%") && !strstr(arg2, "%"))
-                        die(STATE_UNKNOWN, _("%s: Only one threshold must be packet loss (%s)\n"), progname, arg);
+                if (strstr(arg1, "%") && strstr(arg2, "%")) {
+                        die(STATE_UNKNOWN, _("%s: Only one threshold may be packet loss (%s)\n"), progname,
+                                arg);
+                }
+                if (!strstr(arg1, "%") && !strstr(arg2, "%")) {
+                        die(STATE_UNKNOWN, _("%s: Only one threshold must be packet loss (%s)\n"), progname,
+                                arg);
+                }
         }
 
         if (arg2 && strstr(arg2, "%")) {
@@ -452,7 +527,8 @@ void print_help(void) {
         printf("Copyright (c) 1999 Didi Rieder <adrieder@sbox.tu-graz.ac.at>\n");
         printf(COPYRIGHT, copyright, email);
 
-        printf("%s\n", _("This plugin will use the fping command to ping the specified host for a fast check"));
+        printf("%s\n",
+                   _("This plugin will use the fping command to ping the specified host for a fast check"));
 
         printf("%s\n", _("Note that it is necessary to set the suid flag on fping."));
 
@@ -466,7 +542,8 @@ void print_help(void) {
         printf(UT_IPv46);
 
         printf(" %s\n", "-H, --hostname=HOST");
-        printf("    %s\n", _("name or IP Address of host to ping (IP Address bypasses name lookup, reducing system load)"));
+        printf("    %s\n", _("name or IP Address of host to ping (IP Address bypasses name lookup, "
+                                                 "reducing system load)"));
         printf(" %s\n", "-w, --warning=THRESHOLD");
         printf("    %s\n", _("warning threshold pair"));
         printf(" %s\n", "-c, --critical=THRESHOLD");
@@ -480,7 +557,8 @@ void print_help(void) {
         printf(" %s\n", "-T, --target-timeout=INTEGER");
         printf("    %s (default: fping's default for -t)\n", _("Target timeout (ms)"));
         printf(" %s\n", "-i, --interval=INTEGER");
-        printf("    %s (default: fping's default for -p)\n", _("Interval (ms) between sending packets"));
+        printf("    %s (default: fping's default for -p)\n",
+                   _("Interval (ms) between sending packets"));
         printf(" %s\n", "-S, --sourceip=HOST");
         printf("    %s\n", _("name or IP Address of sourceip"));
         printf(" %s\n", "-I, --sourceif=IF");
@@ -489,9 +567,20 @@ void print_help(void) {
         printf("    %s\n", _("set the Don't Fragment flag"));
         printf(" %s\n", "-R, --random");
         printf("    %s\n", _("random packet data (to foil link data compression)"));
+#ifdef FPING_VERSION_5_2_OR_HIGHER
+        printf(" %s\n", "--fwmark=INTEGER");
+        printf("    %s\n", _("set the routing mark to INTEGER (fping option)"));
+#       ifdef FPING_VERSION_5_3_OR_HIGHER
+        printf(" %s\n", "--icmp-timestamp");
+        printf("    %s\n", _("use ICMP Timestamp instead of ICMP Echo (fping option)"));
+        printf(" %s\n", "--check-source");
+        printf("    %s\n", _("discard replies not from target address (fping option)"));
+#       endif
+#endif
         printf(UT_VERBOSE);
         printf("\n");
-        printf(" %s\n", _("THRESHOLD is <rta>,<pl>%% where <rta> is the round trip average travel time (ms)"));
+        printf(" %s\n",
+                   _("THRESHOLD is <rta>,<pl>%% where <rta> is the round trip average travel time (ms)"));
         printf(" %s\n", _("which triggers a WARNING or CRITICAL state, and <pl> is the percentage of"));
         printf(" %s\n", _("packet loss to trigger an alarm state."));
 
@@ -503,5 +592,6 @@ void print_help(void) {
 
 void print_usage(void) {
         printf("%s\n", _("Usage:"));
-        printf(" %s <host_address> -w limit -c limit [-b size] [-n number] [-T number] [-i number]\n", progname);
+        printf(" %s <host_address> -w limit -c limit [-b size] [-n number] [-T number] [-i number]\n",
+                   progname);
 }