padding bytes in mcp2518fd_TransmitChannelLoad method is written into wrong locations

[AntDeM]

Padding bytes added to write a multiple of 4 bytes into RAM is written to wrong location because the offset of 8 is added two times:

• one in row

  Seeed_Arduino_CAN/src/mcp2518fd_can.cpp

  Line 1370 in 6acaf5a

  i = txdNumBytes + 8;

• and the second time in for loop:
  

  Seeed_Arduino_CAN/src/mcp2518fd_can.cpp

  Line 1373 in 6acaf5a

  txBuffer[i + 8 + j] = 0;

The worst case is when txdNumBytes is greater than 56. In this case txBuffer is written out of bounds.

[Pillar1989]

@thePapero welcome to send PR to us. we hope your name appears in the code commit history.

[Lesords]

Hello,

I'm going to close this issue, feel free to re-open it if you have any other questions.

[Autocommentary]