arehbein submitted this change.

View Change


Approvals: laforge: Looks good to me, but someone else must approve daniel: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
core/osmo_io: Fix reception of partial packets

Always append to 'msg->tail' instead of to 'msg->data'.

Change-Id: I0ab8028c807b4181fddd3c00ea2e037c40cf0813
---
M src/core/osmo_io_poll.c
1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/src/core/osmo_io_poll.c b/src/core/osmo_io_poll.c
index dd86f29..c4819fa 100644
--- a/src/core/osmo_io_poll.c
+++ b/src/core/osmo_io_poll.c
@@ -56,7 +56,7 @@
}

hdr.msg = msg;
- hdr.iov[0].iov_base = msgb_data(msg);
+ hdr.iov[0].iov_base = msg->tail;
hdr.iov[0].iov_len = msgb_tailroom(msg);
hdr.hdr.msg_iov = &hdr.iov[0];
hdr.hdr.msg_iovlen = 1;

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I0ab8028c807b4181fddd3c00ea2e037c40cf0813
Gerrit-Change-Number: 33346
Gerrit-PatchSet: 3
Gerrit-Owner: arehbein <arehbein@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: arehbein <arehbein@sysmocom.de>
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged