[Nagios-devel] [PATCH] fix out of bounds access in
Posted: Fri Sep 08, 2006 1:41 am
Do not write outside the hostname array.
snprintf will likely terminate the string anyway.
---
cgi/notifications.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: nagios-2.5/cgi/notifications.c
===================================================================
--- nagios-2.5.orig/cgi/notifications.c
+++ nagios-2.5/cgi/notifications.c
@@ -553,7 +553,7 @@ void display_notifications(void){
/* get the host name */
temp_buffer=(char *)strtok(NULL,";");
snprintf(host_name,sizeof(host_name),"%s",(temp_buffer==NULL)?"":temp_buffer);
- host_name[sizeof(host_name)]='\x0';
+ host_name[sizeof(host_name) - 1]='\x0';
/* get the service name */
if(notification_type==SERVICE_NOTIFICATION){
This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]