[Feature request]: Add buffer length checks

[stijnb1234]

Background and motivation

I've had the situation that I got weird errors because I tried to write text longer than the buffer can hold. There is no check for buffer length < write length.

Describe the solution expected

Add a check if CONFIG_NEX_UART_TRANS_COMMAND_FORMAT_BUFFER_SIZE is smaller then the amount of bytes to write, then return specific error.

Additional info

No response

[gfurtadoalmeida]

Hi @stijnb1234 ,
Thinking about the implementation, as the buffer cannot be resized at runtime, having an error code of a condition that cannot be recovered might not help.
What is your opinion about using abort?

[stijnb1234]

An error code at least gives me insight about what happend. Most of the time, text is fixed.

[gfurtadoalmeida]

Hi @stijnb1234, would you be interested in testing the code in the buffer-length-check branch?
Functions will return NEX_FAIL if an instruction cannot be formatted, with the current buffer size and required size being logged.

Be aware that this branch might have braking changes depending on what functions you are using.