[riot-notifications] [RIOT-OS/RIOT] net/nanocoap: don't abort server on recv error (#11071)
notifications at github.com
Tue Feb 26 18:54:34 CET 2019
### Contribution description
As reported in #10972, the nanocoap_server example becomes unresponsive after receiving a payload greater than 256 bytes long. This issue is caused by the nanocoap_server() function exiting on an any error to sock_udp_recv().
This PR fixes the issue by dropping the packet and resuming listening, rather than exiting the server. After review of the function documentation of sock_udp_recv(), I don't see a situation in which exiting the server is useful in this context. However, I did add debug logging of the error value.
### Testing procedure
See #10972. Thanks @jcarrano for the well written report!
### Issues/PRs references
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* net/nanocoap: don't abort server on recv error
-- File Changes --
M sys/net/application_layer/nanocoap/sock.c (3)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications