[QUESTION] Why in erpc_setup_mbf_XXX failure is fatal?

[amgross]

If you didn't find answer in existing open/closed issues you may ask here

In static mbf setup, in case that there is no more buffers assert is called:

erpc/erpc_c/setup/erpc_setup_mbf_static.cpp

Line 70 in 5c2c6b7

erpc_assert(idx < ERPC_DEFAULT_BUFFERS_COUNT);

In dynamic, if allocation failed then the NULL pointer will be set as buffer:

erpc/erpc_c/setup/erpc_setup_mbf_dynamic.cpp

Line 33 in 5c2c6b7

return MessageBuffer(buf, ERPC_DEFAULT_BUFFER_SIZE);

I would expect in both cases to return error. Why is current implementation as is? (or it is a bug?)

Steps you didn't forgot to do

• I checked if there is no related issue opened/closed.
• I checked that there doesn't exist opened/closed PR which is solving this issue.
• I looked in documentation if there is related information.

[amgross]

Maybe it is just because this function does not return pointer but class? hence should be fixed with #416