Changeset 72 in pacpussensors for trunk/Sick/SickLDMRSSensor.cpp

Timestamp:

Dec 4, 2014, 1:02:02 PM (10 years ago)

Author:

DHERBOMEZ Gérald

Message:

• Correction of a little bug in gps component (property recording bad)
• Improvement of sick lms151 component
• Correction of bug in sick LDRMS : buffer overflow in a memcpy

File:

• trunk/Sick/SickLDMRSSensor.cpp (modified) (1 diff)

trunk/Sick/SickLDMRSSensor.cpp

@@ -322 +322 @@
         // we have a complete message available that we can add to the list
         MessageLDMRS msg;
-        memcpy(msg.body, pendingBytes.data.c_str() + index, msgSize);
-
+                if (msgSize <= sizeof(msg.body))
+                        memcpy(msg.body, pendingBytes.data.c_str() + index, msgSize);
+                else 
+                        LOG_ERROR("Impossible to copy the pending bytes in the MessageLDMRS, checked the size of the body in the code! Data might be corrupted...");
 //        msg->body = messageData;