[mcip-list] patch: data types
Management and Control Information Protocol
Brought to you by:
fjutscha
|
From: Jean-Christophe Z. <jc...@gm...> - 2016-01-11 14:46:34
|
Hello everyone, here is a patch for libmcip. I tried to use the library for sending larger messages (over 8000 bytes) and failed miserably. After much trial and error, I figured that libmcip file-pointer arithmetic must be flawed: mcip_remove_telegram() with "SEARCH_NEXT_STARTBYTE" returned a position that was bogus: it was somewhere in the middle of the telegram, not _after_ the telegram according to the length specified. To fix this, I did the following: all pointers to the payload were changed from char to uint8_t. This fixed the issue. However, to hide this change in the API, I opted for the following approach: I changed all char pointers to void pointers, which are then cast to uint8_t in the function body. That way, users can continue to use the API as they did before. Best regards Jean-Christophe Zeus |