diff options
author | Anders Kaseorg <andersk@mit.edu> | 2012-06-29 04:57:48 (GMT) |
---|---|---|
committer | Holger Weiss <holger@zedat.fu-berlin.de> | 2012-06-29 11:39:11 (GMT) |
commit | 028d50d6f99e647a325a0a68303016382c4bbdc9 (patch) | |
tree | 1d9a14635602169d137409becfa108cd6bdb371c /plugins/check_dns.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_dns.c')
-rw-r--r-- | plugins/check_dns.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugins/check_dns.c b/plugins/check_dns.c index 73b560c..91af730 100644 --- a/plugins/check_dns.c +++ b/plugins/check_dns.c | |||
@@ -103,7 +103,7 @@ main (int argc, char **argv) | |||
103 | } | 103 | } |
104 | 104 | ||
105 | /* get the command to run */ | 105 | /* get the command to run */ |
106 | asprintf (&command_line, "%s %s %s", NSLOOKUP_COMMAND, query_address, dns_server); | 106 | xasprintf (&command_line, "%s %s %s", NSLOOKUP_COMMAND, query_address, dns_server); |
107 | 107 | ||
108 | alarm (timeout_interval); | 108 | alarm (timeout_interval); |
109 | gettimeofday (&tv, NULL); | 109 | gettimeofday (&tv, NULL); |
@@ -208,19 +208,19 @@ main (int argc, char **argv) | |||
208 | for (i=0; i<expected_address_cnt; i++) { | 208 | for (i=0; i<expected_address_cnt; i++) { |
209 | /* check if we get a match and prepare an error string */ | 209 | /* check if we get a match and prepare an error string */ |
210 | if (strcmp(address, expected_address[i]) == 0) result = STATE_OK; | 210 | if (strcmp(address, expected_address[i]) == 0) result = STATE_OK; |
211 | asprintf(&temp_buffer, "%s%s; ", temp_buffer, expected_address[i]); | 211 | xasprintf(&temp_buffer, "%s%s; ", temp_buffer, expected_address[i]); |
212 | } | 212 | } |
213 | if (result == STATE_CRITICAL) { | 213 | if (result == STATE_CRITICAL) { |
214 | /* Strip off last semicolon... */ | 214 | /* Strip off last semicolon... */ |
215 | temp_buffer[strlen(temp_buffer)-2] = '\0'; | 215 | temp_buffer[strlen(temp_buffer)-2] = '\0'; |
216 | asprintf(&msg, _("expected '%s' but got '%s'"), temp_buffer, address); | 216 | xasprintf(&msg, _("expected '%s' but got '%s'"), temp_buffer, address); |
217 | } | 217 | } |
218 | } | 218 | } |
219 | 219 | ||
220 | /* check if authoritative */ | 220 | /* check if authoritative */ |
221 | if (result == STATE_OK && expect_authority && non_authoritative) { | 221 | if (result == STATE_OK && expect_authority && non_authoritative) { |
222 | result = STATE_CRITICAL; | 222 | result = STATE_CRITICAL; |
223 | asprintf(&msg, _("server %s is not authoritative for %s"), dns_server, query_address); | 223 | xasprintf(&msg, _("server %s is not authoritative for %s"), dns_server, query_address); |
224 | } | 224 | } |
225 | 225 | ||
226 | microsec = deltime (tv); | 226 | microsec = deltime (tv); |