Fixed angle_compensate_nodes array overflow issue

Slamtec / rplidar_ros

BSD 2-Clause "Simplified" License

504 stars 532 forks source link

Fixed angle_compensate_nodes array overflow issue #10

Open salihmarangoz opened 5 years ago

salihmarangoz commented 5 years ago

On A2, angle_compensate_nodes size is 720 but sometimes angle_value-angle_compensate_offset+j becomes equal to 720. This causes overflow at line 313

salihmarangoz commented 5 years ago

Related to: #1

deb0ch commented 5 years ago

I faced a similar issue today and it was fixed by a similar patch.

I was running the node with a RPLidar A3 in Stability mode on a Raspi using the official usb module with the highest baudrate and it crashed consistently after a few seconds of running.