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 *