Use of sizeof() operator

[mpaland]

Use of sizeof() operator

[mpaland]

Hello @MaJerle,
thanks a lot for your suggestion about the sizeof() operator! Appreciate your feedback very much.

PRINTF_NTOA_BUFFER_SIZE is used as a (macro defined) module scoped constant here.
I would agree to use ARRAY_SIZE or COUNTOF macros if the buffer size is set by a magic number, is externally defined or depends on element initialization (buffer[] = { ... };) etc.
But in this case here using a well defined constant (even named XXX_BUFFER_SIZE) for setting the buffer size and checking its length in different expressions is perfectly fine (and automotive safe).
Using 'ARRAY_SIZE' has the advantage that one can define a buffer with an arbitrary size and the bounds check always works, that's right - but IMHO not mandatory necessary here.

I don't see., why and how this should lead to a bug here, except one would change char buf[PRINTF_NTOA_BUFFER_SIZE] to something like char buf[20] or char buf[PRINTF_NTOA_BUFFER_SIZE - 10U] - but that would be a serious violation (which coverity should find anyway).

Besides PRINTF_NTOA_BUFFER_SIZE is used in _ntoa_format() where buf is unknown as buffer (it's just a pointer there).