SICKAG / sick_scan_xd

A versatile driver for a wide range of SICK LiDAR and RADAR devices, providing support for both Linux (native, ROS 1, ROS 2) and Windows (native, ROS 2) platforms.
Apache License 2.0
107 stars 87 forks source link

Access violation in SickScanFieldMonSingleton::parseBinaryDatagram #154

Closed PeterWimmerTmc closed 1 year ago

PeterWimmerTmc commented 1 year ago

Scanner: LMS511

Datagram size is 23 bytes, but SickScanFieldMonSingleton::parseBinaryDatagram doesn't check the length of the message and reads beyond the end.

Screenshot 2023-03-15 072414

rostest commented 1 year ago

Many thanks for reporting this issue! Please use the fix in the update branch https://github.com/SICKAG/sick_scan_xd/tree/feature/lms511_field_mon.