diff options
| author | Anders Kaseorg <andersk@mit.edu> | 2012-06-29 00:57:48 -0400 |
|---|---|---|
| committer | Holger Weiss <holger@zedat.fu-berlin.de> | 2012-06-29 13:39:11 +0200 |
| commit | 028d50d6f99e647a325a0a68303016382c4bbdc9 (patch) | |
| tree | 1d9a14635602169d137409becfa108cd6bdb371c /plugins/check_ping.c | |
| parent | 9976876584e5a1df6e1c9315212c3d274df7a12e (diff) | |
| download | monitoring-plugins-028d50d6f99e647a325a0a68303016382c4bbdc9.tar.gz | |
Die when asprintf fails
Fixes many instances of
warning: ignoring return value of 'asprintf', declared with attribute warn_unused_result [-Wunused-result]
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
Diffstat (limited to 'plugins/check_ping.c')
| -rw-r--r-- | plugins/check_ping.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/plugins/check_ping.c b/plugins/check_ping.c index 473f9f24..eef2195b 100644 --- a/plugins/check_ping.c +++ b/plugins/check_ping.c | |||
| @@ -121,12 +121,12 @@ main (int argc, char **argv) | |||
| 121 | /* does the host address of number of packets argument come first? */ | 121 | /* does the host address of number of packets argument come first? */ |
| 122 | #ifdef PING_PACKETS_FIRST | 122 | #ifdef PING_PACKETS_FIRST |
| 123 | # ifdef PING_HAS_TIMEOUT | 123 | # ifdef PING_HAS_TIMEOUT |
| 124 | asprintf (&cmd, rawcmd, timeout_interval, max_packets, addresses[i]); | 124 | xasprintf (&cmd, rawcmd, timeout_interval, max_packets, addresses[i]); |
| 125 | # else | 125 | # else |
| 126 | asprintf (&cmd, rawcmd, max_packets, addresses[i]); | 126 | xasprintf (&cmd, rawcmd, max_packets, addresses[i]); |
| 127 | # endif | 127 | # endif |
| 128 | #else | 128 | #else |
| 129 | asprintf (&cmd, rawcmd, addresses[i], max_packets); | 129 | xasprintf (&cmd, rawcmd, addresses[i], max_packets); |
| 130 | #endif | 130 | #endif |
| 131 | 131 | ||
| 132 | if (verbose >= 2) | 132 | if (verbose >= 2) |
| @@ -491,7 +491,7 @@ run_ping (const char *cmd, const char *addr) | |||
| 491 | if (warn_text == NULL) { | 491 | if (warn_text == NULL) { |
| 492 | warn_text = strdup(_("System call sent warnings to stderr ")); | 492 | warn_text = strdup(_("System call sent warnings to stderr ")); |
| 493 | } else { | 493 | } else { |
| 494 | asprintf(&warn_text, "%s %s", warn_text, _("System call sent warnings to stderr ")); | 494 | xasprintf(&warn_text, "%s %s", warn_text, _("System call sent warnings to stderr ")); |
| 495 | } | 495 | } |
| 496 | } | 496 | } |
| 497 | } | 497 | } |
| @@ -540,7 +540,7 @@ error_scan (char buf[MAX_INPUT_BUFFER], const char *addr) | |||
| 540 | if (warn_text == NULL) | 540 | if (warn_text == NULL) |
| 541 | warn_text = strdup (_(WARN_DUPLICATES)); | 541 | warn_text = strdup (_(WARN_DUPLICATES)); |
| 542 | else if (! strstr (warn_text, _(WARN_DUPLICATES)) && | 542 | else if (! strstr (warn_text, _(WARN_DUPLICATES)) && |
| 543 | asprintf (&warn_text, "%s %s", warn_text, _(WARN_DUPLICATES)) == -1) | 543 | xasprintf (&warn_text, "%s %s", warn_text, _(WARN_DUPLICATES)) == -1) |
| 544 | die (STATE_UNKNOWN, _("Unable to realloc warn_text\n")); | 544 | die (STATE_UNKNOWN, _("Unable to realloc warn_text\n")); |
| 545 | return (STATE_WARNING); | 545 | return (STATE_WARNING); |
| 546 | } | 546 | } |
