pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/30754 )

Change subject: amr: constify input buffer in osmo_amr_is_oa()
......................................................................

amr: constify input buffer in osmo_amr_is_oa()

Change-Id: I72ef6fc700f628ea158ea888a5f1e9dc8776ab7e
---
M include/osmocom/netif/amr.h
M src/amr.c
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved
  laforge: Looks good to me, approved



diff --git a/include/osmocom/netif/amr.h b/include/osmocom/netif/amr.h
index 47f7f3d..e8f08d4 100644
--- a/include/osmocom/netif/amr.h
+++ b/include/osmocom/netif/amr.h
@@ -159,7 +159,7 @@
 size_t osmo_amr_bytes(uint8_t amr_cmr);
 size_t osmo_amr_bits(uint8_t amr_ft);

-bool osmo_amr_is_oa(uint8_t *payload, unsigned int payload_len);
+bool osmo_amr_is_oa(const uint8_t *payload, unsigned int payload_len);
 int osmo_amr_oa_to_bwe(uint8_t *payload, unsigned int payload_len);
 int osmo_amr_bwe_to_oa(uint8_t *payload, unsigned int payload_len,
                       unsigned int payload_maxlen);
diff --git a/src/amr.c b/src/amr.c
index 66f41c6..b648d54 100644
--- a/src/amr.c
+++ b/src/amr.c
@@ -113,7 +113,7 @@
  *  \param[inout] payload user provided memory containing the AMR payload.
  *  \param[in] payload_len overall length of the AMR payload.
  *  \returns true when the payload is octet aligned. */
-bool osmo_amr_is_oa(uint8_t *payload, unsigned int payload_len)
+bool osmo_amr_is_oa(const uint8_t *payload, unsigned int payload_len)
 {
        /* NOTE: The distinction between octet-aligned and bandwith-efficient
         * mode normally relys on out of band methods that explicitly select

--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30754
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I72ef6fc700f628ea158ea888a5f1e9dc8776ab7e
Gerrit-Change-Number: 30754
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to