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")); |
