rusoku/canal-dll

CANAL DLL for TouCAN USB to CAN bus converter

canal-dll's Issues

Not working with 10kbps

The device is not working with 10kbps.

See also macOS Rusoku.dylib issue 21.

Some calls to CanalOpen cause heap corruption

Hello there!

First off, big thanks for changing the license to MIT back in January!

We've recently integrated this DLL into our open source ISOBUS CAN stack at Open-Agriculture/AgIsoStack-plus-plus#220 and I noticed that it usually works, but sometimes when I call CanalOpen that something weird seems to be happening. More specifically, when CTouCANobj::Open is called, eventually it calls pDllList->RemoveAllNodes(&m_receiveList); which sometimes works, but other times it seems to cause heap corruption.

When this happens, if we inspect pdll it seems like nCount is suspiciously high, and the tail pointer of the CDllList seems suspicious also. The random nature of this happening makes it feel almost like a concurrency or initialization issue of some kind.

Anyways, I am not entirely sure what to make of it, but wanted to bring it to your attention. It is somewhat random but reproducible, so if you are interested in gathering some more information or troubleshooting I'd be happy to do what I can.

This was all tested using first the official released DLLs, but to actually debug in there and get the symbols needed, I compiled the DLL with Visual Studio 2022 17.5.3, specifically with compiler MSVC 19.35.32216.1 for 64 bit architecture to gather those screenshots.

Besides this, integrating this was very easy, so thank you again for your effort on that!

Regards,
Adrian