[monitoring-plugins] check_curl: abort redir if location is not found

Lorenz Kästle git at monitoring-plugins.org
Thu Nov 27 14:50:11 CET 2025


 Module: monitoring-plugins
 Branch: master
 Commit: 2917b8735f1d56211eac0ad1bf7a051a842abd76
 Author: Lorenz Kästle <12514511+RincewindsHat at users.noreply.github.com>
   Date: Thu Nov 27 13:24:30 2025 +0100
    URL: https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=2917b873

check_curl: abort redir if location is not found

This commit changes the behaviour of check_curl slightly.
Previously when the redirection method was set to the old 'check_http'
style redirection and there was no "location" header in the original
answer 'check_curl' segfaulted.
Now, at least it dies properly with a message.

---

 plugins/check_curl.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/plugins/check_curl.c b/plugins/check_curl.c
index fc704171..4eaa5f1d 100644
--- a/plugins/check_curl.c
+++ b/plugins/check_curl.c
@@ -672,6 +672,11 @@ redir_wrapper redir(curlhelp_write_curlbuf *header_buf, const check_curl_config
 
 	char *location = get_header_value(headers, nof_headers, "location");
 
+	if (location == NULL) {
+		// location header not found
+		die(STATE_UNKNOWN, "HTTP UNKNOWN - could not find \"location\" header\n");
+	}
+
 	if (verbose >= 2) {
 		printf(_("* Seen redirect location %s\n"), location);
 	}



More information about the Commits mailing list