From 0e8dbe92f0b4cd5570addf8ee5e6841f5236e466 Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Tue, 29 Dec 2015 21:33:38 +1100 Subject: [PATCH] GCS_MAVLink: fixed string overrun found by asan --- libraries/GCS_MAVLink/GCS_Common.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libraries/GCS_MAVLink/GCS_Common.cpp b/libraries/GCS_MAVLink/GCS_Common.cpp index a2b60f0274..0cfade2121 100644 --- a/libraries/GCS_MAVLink/GCS_Common.cpp +++ b/libraries/GCS_MAVLink/GCS_Common.cpp @@ -589,7 +589,9 @@ GCS_MAVLINK::send_text(MAV_SEVERITY severity, const char *str) comm_get_txspace(chan) >= MAVLINK_NUM_NON_PAYLOAD_BYTES+MAVLINK_MSG_ID_STATUSTEXT_LEN) { // send immediately - mavlink_msg_statustext_send(chan, severity, str); + char msg[50] {}; + strncpy(msg, str, sizeof(msg)); + mavlink_msg_statustext_send(chan, severity, msg); } else { // send via the deferred queuing system mavlink_statustext_t *s = &pending_status;