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_by_ssh.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_by_ssh.c')
-rw-r--r-- | plugins/check_by_ssh.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/plugins/check_by_ssh.c b/plugins/check_by_ssh.c index 87520166..c5f0d529 100644 --- a/plugins/check_by_ssh.c +++ b/plugins/check_by_ssh.c | |||
@@ -281,8 +281,8 @@ process_arguments (int argc, char **argv) | |||
281 | case 'C': /* Command for remote machine */ | 281 | case 'C': /* Command for remote machine */ |
282 | commands++; | 282 | commands++; |
283 | if (commands > 1) | 283 | if (commands > 1) |
284 | asprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd); | 284 | xasprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd); |
285 | asprintf (&remotecmd, "%s%s", remotecmd, optarg); | 285 | xasprintf (&remotecmd, "%s%s", remotecmd, optarg); |
286 | break; | 286 | break; |
287 | case 'S': /* skip n (or all) lines on stdout */ | 287 | case 'S': /* skip n (or all) lines on stdout */ |
288 | if (optarg == NULL) | 288 | if (optarg == NULL) |
@@ -328,13 +328,13 @@ process_arguments (int argc, char **argv) | |||
328 | if (strlen(remotecmd) == 0) { | 328 | if (strlen(remotecmd) == 0) { |
329 | for (; c < argc; c++) | 329 | for (; c < argc; c++) |
330 | if (strlen(remotecmd) > 0) | 330 | if (strlen(remotecmd) > 0) |
331 | asprintf (&remotecmd, "%s %s", remotecmd, argv[c]); | 331 | xasprintf (&remotecmd, "%s %s", remotecmd, argv[c]); |
332 | else | 332 | else |
333 | asprintf (&remotecmd, "%s", argv[c]); | 333 | xasprintf (&remotecmd, "%s", argv[c]); |
334 | } | 334 | } |
335 | 335 | ||
336 | if (commands > 1 || passive) | 336 | if (commands > 1 || passive) |
337 | asprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd); | 337 | xasprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd); |
338 | 338 | ||
339 | if (remotecmd == NULL || strlen (remotecmd) <= 1) | 339 | if (remotecmd == NULL || strlen (remotecmd) <= 1) |
340 | usage_va(_("No remotecmd")); | 340 | usage_va(_("No remotecmd")); |