From 2917b8735f1d56211eac0ad1bf7a051a842abd76 Mon Sep 17 00:00:00 2001 From: Lorenz Kästle <12514511+RincewindsHat@users.noreply.github.com> Date: Thu, 27 Nov 2025 13:24:30 +0100 Subject: 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(+) (limited to 'plugins/check_curl.c') 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); } -- cgit v1.2.3-74-g34f1