From 79882007019a23d30f07e344477acae48d5193bd Mon Sep 17 00:00:00 2001 From: Adam Spiers Date: Sun, 20 Nov 2016 12:21:12 +0000 Subject: [PATCH] fix -Wunused-result warnings when calling asprintf() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix these warnings: papertrail_proc.hh: In constructor ‘papertrail_proc::papertrail_proc(const string&, time_t, time_t)’: papertrail_proc.hh:107:36: warning: ignoring return value of ‘int asprintf(char**, const char*, ...)’, declared with attribute warn_unused_result [-Wunused-result] this->ptp_api_key); ^ papertrail_proc.hh: In member function ‘void papertrail_proc::set_url()’: papertrail_proc.hh:146:47: warning: ignoring return value of ‘int asprintf(char**, const char*, ...)’, declared with attribute warn_unused_result [-Wunused-result] this->ptp_quoted_search.in()); --- src/papertrail_proc.hh | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/src/papertrail_proc.hh b/src/papertrail_proc.hh index ecaa383a..613383ea 100644 --- a/src/papertrail_proc.hh +++ b/src/papertrail_proc.hh @@ -102,9 +102,11 @@ public: this->ptp_search.c_str(), this->ptp_search.size()); - asprintf(this->ptp_token_header.out(), - "X-Papertrail-Token: %s", - this->ptp_api_key); + if (asprintf(this->ptp_token_header.out(), + "X-Papertrail-Token: %s", + this->ptp_api_key) == -1) { + perror("Failed to allocate X-Papertrail-Token string"); + } this->ptp_header_list = curl_slist_append(this->ptp_header_list, this->ptp_token_header.in()); @@ -140,10 +142,12 @@ public: "max_time=%ld&", this->ptp_max_time); } - asprintf(this->ptp_url.out(), - "%sq=%s", - base_url, - this->ptp_quoted_search.in()); + if (asprintf(this->ptp_url.out(), + "%sq=%s", + base_url, + this->ptp_quoted_search.in()) == -1) { + perror("Failed to allocate ptp_url"); + } curl_easy_setopt(this->cr_handle, CURLOPT_URL, this->ptp_url.in()); };