pespin submitted this change.

Approvals:
  daniel: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  laforge: Looks good to me, approved
  Jenkins Builder: Verified

stream: Log error on short send

This is clearly a problem on TCP streams which needs to be addressed in
the future.

Related: OS#5836
Change-Id: I9bd257b80a378b779df84e204673f8e394eca5b6
---
M src/stream.c
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/src/stream.c b/src/stream.c
index e9860c3..1ca4e9a 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -384,6 +384,8 @@
 			osmo_stream_cli_reconnect(cli);
 		}
 		LOGSCLI(cli, LOGL_ERROR, "error %d to send\n", ret);
+	} else if (ret < msgb_length(msg)) {
+		LOGP(DLINP, LOGL_ERROR, "short send: %d < exp %u\n", ret, msgb_length(msg));
 	}
 	msgb_free(msg);
 	return 0;
@@ -1344,6 +1346,8 @@
 	}
 	if (ret == -1) /* send(): On error -1 is returned, and errno is set appropriately */
 		LOGP(DLINP, LOGL_ERROR, "error to send: %s\n", strerror(errno));
+	else if (ret < msgb_length(msg))
+		LOGP(DLINP, LOGL_ERROR, "short send: %d < exp %u\n", ret, msgb_length(msg));
 
 	msgb_free(msg);

To view, visit change 30720. To unsubscribe, or for help writing mail filters, visit settings.