laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
sccp2sua: constify params of sccp_ptr_part_consistent()

Change-Id: I8011e5cf26d535120829c3925693a2bb3efdb864
---
M src/sccp2sua.c
1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/sccp2sua.c b/src/sccp2sua.c
index f18ebd7..59cb64b 100644
--- a/src/sccp2sua.c
+++ b/src/sccp2sua.c
@@ -431,9 +431,9 @@
* \param[in] msg Message containing SCCP address
* \param[in] ptr_addr pointer to byte with relative SCCP pointer
* \returns true if OK; false if message inconsistent */
-static bool sccp_ptr_part_consistent(struct msgb *msg, uint8_t *ptr_addr)
+static bool sccp_ptr_part_consistent(const struct msgb *msg, const uint8_t *ptr_addr)
{
- uint8_t *ptr;
+ const uint8_t *ptr;

/* check the address of the relative pointer is within msg */
if (ptr_addr < msg->data || ptr_addr > msg->tail) {

To view, visit change 34461. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I8011e5cf26d535120829c3925693a2bb3efdb864
Gerrit-Change-Number: 34461
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged