[XS] Change in libosmo-netif[master]: rtp: osmo_rtp_get_payload(): Fix return ptr if RTP pkt contains padding

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/38063?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: rtp: osmo_rtp_get_payload(): Fix return ptr if RTP pkt contains padding ...................................................................... rtp: osmo_rtp_get_payload(): Fix return ptr if RTP pkt contains padding Padding is added at the end of the packet, hence the return pointer calculation was wrong in the case where RTP packet contained padding at the end. Related: OS#6562 Change-Id: I581fa1ca5c656ecddddba0f1c85c63beece2a624 --- M src/rtp.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/rtp.c b/src/rtp.c index 1da409c..9b1d7cb 100644 --- a/src/rtp.c +++ b/src/rtp.c @@ -162,7 +162,7 @@ } *plen = payload_len; - return (uint8_t *)msg->data + msg->len - payload_len; + return payload; } struct msgb * -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/38063?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-netif Gerrit-Branch: master Gerrit-Change-Id: I581fa1ca5c656ecddddba0f1c85c63beece2a624 Gerrit-Change-Number: 38063 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>