trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zw...@apache.org
Subject [trafficserver] branch 7.1.x updated: Fix a TM crash on alarms
Date Tue, 09 Oct 2018 16:21:32 GMT
This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new ced4d18  Fix a TM crash on alarms
ced4d18 is described below

commit ced4d18031e5f00476431963f98e460955a2cb1c
Author: Masakazu Kitajo <maskit@apache.org>
AuthorDate: Fri Sep 28 13:44:23 2018 -0600

    Fix a TM crash on alarms
    
    An alarm receiver expects null terminated message but a sender calculated message size
incorrectly, and it caused a buffer overflow.
    
    (cherry picked from commit 58bd441976cb5c9e4422e1de26c07572af40271b)
---
 lib/records/I_RecCore.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/records/I_RecCore.h b/lib/records/I_RecCore.h
index 0245140..294db16 100644
--- a/lib/records/I_RecCore.h
+++ b/lib/records/I_RecCore.h
@@ -194,7 +194,7 @@ void RecSignalManager(int id, const char *, size_t);
 static inline void
 RecSignalManager(int id, const char *str)
 {
-  RecSignalManager(id, str, strlen(str + 1));
+  RecSignalManager(id, str, strlen(str) + 1);
 }
 
 // Format a message, and send it to the manager and to the Warning diagnostic.


Mime
View raw message