pespin has uploaded this change for review.

View Change

mgcp-client: Remove impossible code path

read(4096 - 128) cannot return > 4096 - 128.

Change-Id: I352c3b28c8de6482277cb84cd0f51893494dc929
---
M src/libosmo-mgcp-client/mgcp_client.c
1 file changed, 0 insertions(+), 4 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/82/28482/1
diff --git a/src/libosmo-mgcp-client/mgcp_client.c b/src/libosmo-mgcp-client/mgcp_client.c
index 3711f24..77de23f 100644
--- a/src/libosmo-mgcp-client/mgcp_client.c
+++ b/src/libosmo-mgcp-client/mgcp_client.c
@@ -726,10 +726,6 @@

msgb_free(msg);
return -1;
- } else if (ret > 4096 - 128) {
- LOGPMGW(mgcp, LOGL_ERROR, "Too much data: %s: %d\n", osmo_sock_get_name2(fd->fd), ret);
- msgb_free(msg);
- return -1;
}

msg->l2h = msgb_put(msg, ret);

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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I352c3b28c8de6482277cb84cd0f51893494dc929
Gerrit-Change-Number: 28482
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-MessageType: newchange