Beckhoff / ADS

Beckhoff protocol to communicate with TwinCAT devices.
MIT License
515 stars 197 forks source link

ADSERR_DEVICE_SYMBOLVERSIONINVALID creating AdsNotification by symbolName on big endian CPU #139

Closed heppth closed 3 years ago

heppth commented 3 years ago

I have found a bug that only occurs with big endian CPUs: If you create an AdsNotification with a symbolName, you will receive error code 0x711 which is equal to ADSERR_DEVICE_SYMBOLVERSIONINVALID. Same code on little endian CPUs is working fine. Creating AdsNotification with indexGroup and indexOffset works on big endian CPUs. The error is due to a missing byte-order conversion in AdsNotificationOOI.h I will make some tests an create a pull request.