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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
