Module: monitoring-plugins Branch: master Commit: a3a78795b2951777d2586abc2ea9ef40ca4f4399 Author: Peter (pir) Radcliffe <pir-gitweb....@pir.net> Committer: Sven Nierlein <s...@nierlein.de> Date: Tue Nov 24 03:03:41 2015 +0000 URL: https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=a3a7879
Use GMT timezone in SSL certs SSL certs are required to use times in GMT per https://www.ietf.org/rfc/rfc5280.txt but the mktime() here assumes the current timezone. Fix the time_t conversion to be done assuming GMT with timegm() and only do it once rather than twice. Display the expiry date and time with ISO format years and give an offset from GMT and a timezone to be very clear about exactly what time is being displayed. Time given is correct and now in the machine’s timezone. --- plugins/sslutils.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/plugins/sslutils.c b/plugins/sslutils.c index 4f9c793..6437289 100644 --- a/plugins/sslutils.c +++ b/plugins/sslutils.c @@ -264,10 +264,10 @@ int np_net_ssl_check_cert(int days_till_exp_warn, int days_till_exp_crit){ (tm->data[10 + offset] - '0') * 10 + (tm->data[11 + offset] - '0'); stamp.tm_isdst = -1; - time_left = difftime(timegm(&stamp), time(NULL)); + tm_t = timegm(&stamp); + time_left = difftime(tm_t, time(NULL)); days_left = time_left / 86400; - tm_t = mktime (&stamp); - strftime(timestamp, 50, "%c", localtime(&tm_t)); + strftime(timestamp, 50, "%F %R %z/%Z", localtime(&tm_t)); if (days_left > 0 && days_left <= days_till_exp_warn) { printf (_("%s - Certificate '%s' expires in %d day(s) (%s).\n"), (days_left>days_till_exp_crit)?"WARNING":"CRITICAL", cn, days_left, timestamp);