Attention is currently required from: laforge.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-netif/+/42191?usp=email )
Change subject: stream: Improve error handling and logging in write_cb ......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
alternatively, if we submitted a write of zero bytes length, then that's also an error in the applic […]
Actually a write(count=0) is an expected correct scenario, see man 2 write:
``` If count is zero and fd refers to a regular file, then write() may return a failure status if one of the errors below is detected. If no errors are detected, or error detection is not performed, 0 is returned without causing any other effect. If count is zero and fd refers to a file other than a regular file, the results are not specified. ```
Also, currently stream_cli and stream_srv are handling that part different, so for sure we want to unify the behavior there...