dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/32752 )
Change subject: l1sap: cosmetic: rename payload_len to rtp_pl_len ......................................................................
l1sap: cosmetic: rename payload_len to rtp_pl_len
The function signature of rtppayload_is_octet_aligned has a parameter rtp_pl for the payload and a parameter payload_len for the length of the payload, while other functions use rtp_pl and rtp_pl_len.
Change-Id: I8a0e0357aab2a78e25811f66b1b870e8c6ebffe9 --- M src/common/l1sap.c 1 file changed, 15 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/52/32752/1
diff --git a/src/common/l1sap.c b/src/common/l1sap.c index 3939294..cff82cf 100644 --- a/src/common/l1sap.c +++ b/src/common/l1sap.c @@ -1216,7 +1216,7 @@ return 1; }
-static bool rtppayload_is_octet_aligned(const uint8_t *rtp_pl, uint8_t payload_len) +static bool rtppayload_is_octet_aligned(const uint8_t *rtp_pl, uint8_t rtp_pl_len) { /* * Logic: If 1st bit padding is not zero, packet is either: @@ -1237,7 +1237,7 @@ #define AMR_PADDING1(rtp_pl) (rtp_pl[0] & 0x0f) #define AMR_PADDING2(rtp_pl) (rtp_pl[1] & 0x03)
- if (payload_len < 2 || AMR_PADDING1(rtp_pl) || AMR_PADDING2(rtp_pl)) + if (rtp_pl_len < 2 || AMR_PADDING1(rtp_pl) || AMR_PADDING2(rtp_pl)) return false;
return true;