[monitoring-plugins] check_http: Handle chunked encoding without ...

RincewindsHat git at monitoring-plugins.org
Mon Feb 6 13:10:12 CET 2023


 Module: monitoring-plugins
 Branch: master
 Commit: 6d3e44d2d8395076060e9c741e9b173dc5d57b76
 Author: RincewindsHat <12514511+RincewindsHat at users.noreply.github.com>
   Date: Mon Feb  6 11:39:44 2023 +0100
    URL: https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=6d3e44d

check_http: Handle chunked encoding without actual content correctly

---

 plugins/check_http.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/plugins/check_http.c b/plugins/check_http.c
index 5fa310f..8dda046 100644
--- a/plugins/check_http.c
+++ b/plugins/check_http.c
@@ -1462,7 +1462,13 @@ char *unchunk_content(const char *content) {
     memcpy(result + (overall_size - size_of_chunk), start_of_chunk, size_of_chunk);
   }
 
-  result[overall_size] = '\0';
+  if (overall_size == 0 && result == NULL) {
+    // We might just have received the end chunk without previous content, so result is never allocated
+    result = calloc(1, sizeof(char));
+    // No error handling here, we can only return NULL anyway
+  } else {
+    result[overall_size] = '\0';
+  }
   return result;
 }
 



More information about the Commits mailing list