pespin submitted this change.

View Change

Approvals: fixeria: Looks good to me, but someone else must approve osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
cbsp_link: Log unexpected return error codes

This easies spotting the cause of close/destroy code paths.

Change-Id: Ibfd9a33798ba88e2332c00ed90709e38bb2222d7
---
M src/cbsp_link.c
1 file changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/cbsp_link.c b/src/cbsp_link.c
index 2c31455..07fbadd 100644
--- a/src/cbsp_link.c
+++ b/src/cbsp_link.c
@@ -121,6 +121,7 @@
/* message de-segmentation */
rc = osmo_cbsp_recv_buffered(conn, ofd->fd, &msg, &link->rx_msg);
if (rc <= 0) {
+ LOGPCC(link, LOGL_NOTICE, "osmo_cbsp_recv_buffered() ret %d\n", rc);
if (rc == -EAGAIN || rc == -EINTR) {
/* more data needs to be read */
return 0;
@@ -205,6 +206,7 @@
/* message de-segmentation */
rc = osmo_cbsp_recv_buffered(conn, ofd->fd, &msg, &link->rx_msg);
if (rc <= 0) {
+ LOGPCC(link, LOGL_NOTICE, "osmo_cbsp_recv_buffered() ret %d\n", rc);
if (rc == -EAGAIN || rc == -EINTR) {
/* more data needs to be read */
return 0;

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

Gerrit-MessageType: merged
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: Ibfd9a33798ba88e2332c00ed90709e38bb2222d7
Gerrit-Change-Number: 38977
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>