summaryrefslogtreecommitdiffstats
path: root/plugins/check_ups.c
diff options
context:
space:
mode:
authorPhiBo <phibo@dinotools.org>2021-11-24 10:23:47 (GMT)
committerPhiBo <phibo@dinotools.org>2021-11-24 10:23:47 (GMT)
commit09bfb1b482f428b1c11f68b5be7d134ba5e3b457 (patch)
tree51fba2105e268dca60959d3cb404de84fefcb1d5 /plugins/check_ups.c
parentdf835cfde00334de219efc741e2ba7bf5f64f635 (diff)
downloadmonitoring-plugins-09bfb1b482f428b1c11f68b5be7d134ba5e3b457.tar.gz
Fix possible overflow in check_ups (Fixes: #1030)refs/pull/1727/head
Diffstat (limited to 'plugins/check_ups.c')
-rw-r--r--plugins/check_ups.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/plugins/check_ups.c b/plugins/check_ups.c
index e9e56a5..0de37a2 100644
--- a/plugins/check_ups.c
+++ b/plugins/check_ups.c
@@ -402,7 +402,10 @@ get_ups_variable (const char *varname, char *buf, size_t buflen)
402 402
403 /* create the command string to send to the UPS daemon */ 403 /* create the command string to send to the UPS daemon */
404 /* Add LOGOUT to avoid read failure logs */ 404 /* Add LOGOUT to avoid read failure logs */
405 sprintf (send_buffer, "GET VAR %s %s\nLOGOUT\n", ups_name, varname); 405 if (snprintf (send_buffer, sizeof(send_buffer), "GET VAR %s %s\nLOGOUT\n", ups_name, varname) >= sizeof(send_buffer)) {
406 printf("%s\n", _("UPS name to long for buffer"));
407 return ERROR;
408 }
406 409
407 /* send the command to the daemon and get a response back */ 410 /* send the command to the daemon and get a response back */
408 if (process_tcp_request 411 if (process_tcp_request