The default value of HEAP_MEM_POOL_ADD_SIZE_SOCKETPAIR (255) seems to be insufficient to create a socket_pair.

To create a socket pair using z_impl_zsock_socketpair from socketpair.c, spair_new() is executed twice. spair_new() does a malloc() of sizeof(struct spair) (148) bytes. This results in a total of 296 bytes of heap usage.

If there is only one entity like socketpair wanting heap, then this is indeed problem. If there are other entities adding their requirements to the heap size, then the issue is probably not seen so often.

Since the heap is only 256 bytes, this will fail.

Is this derised behaviour? Or should the default size be increased? Also, sizeof(struct spair) depends on CONFIG_NET_SOCKETPAIR_BUFFER_SIZE. Should this somehow be taken into account?

Yes, we should allocate the enough space for the two spair struct.

Would you be able to send a fix for this, so increase the default value from 255 to what ever is needed?
Also it might make sense to add build asserts in the code so that if there is not enough space allocated, then the build will fail so we can detect this issue earlier. With the build assert, we do not need to specifically check the value of CONFIG_NET_SOCKETPAIR_BUFFER_SIZE.

@IVandeVeire will you open a Pull Request to fix this?

@IVandeVeire will you open a Pull Request to fix this?

For the coap_server sample specifically this has been fixed in #87544, but one could argue that this is a broader issue.

@IVandeVeire will you open a Pull Request to fix this?

I created a PR #88962