pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmo-sccp/+/34464?usp=email )


Change subject: sccp2sua: remove unused msg param from 
sccp_{addr,data}_to_sua_ptr
......................................................................

sccp2sua: remove unused msg param from sccp_{addr,data}_to_sua_ptr

Change-Id: If4a8283614639d456d2bdf2b31873e28c68f59bc
---
M src/sccp2sua.c
1 file changed, 25 insertions(+), 16 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/64/34464/1

diff --git a/src/sccp2sua.c b/src/sccp2sua.c
index 59cb64b..ed224cf 100644
--- a/src/sccp2sua.c
+++ b/src/sccp2sua.c
@@ -338,7 +338,7 @@
 }

 /*! \brief convenience wrapper around sccp_addr_to_sua() for variable 
mandatory addresses */
-static int sccp_addr_to_sua_ptr(struct xua_msg *xua, uint16_t iei, struct msgb 
*msg, uint8_t *ptr_addr)
+static int sccp_addr_to_sua_ptr(struct xua_msg *xua, uint16_t iei, uint8_t 
*ptr_addr)
 {
        uint8_t *addr = ptr_addr + *ptr_addr + 1;
        unsigned int addrlen = *(ptr_addr + *ptr_addr);
@@ -456,7 +456,7 @@
 }

 /*! \brief convenience wrapper around xua_msg_add_data() for variable 
mandatory data */
-static int sccp_data_to_sua_ptr(struct xua_msg *xua, uint16_t iei, struct msgb 
*msg, uint8_t *ptr_addr)
+static int sccp_data_to_sua_ptr(struct xua_msg *xua, uint16_t iei, uint8_t 
*ptr_addr)
 {
        uint8_t *addr = ptr_addr + *ptr_addr + 1;
        unsigned int addrlen = *(ptr_addr + *ptr_addr);
@@ -1028,7 +1028,7 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &req->variable_called))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, msg, 
&req->variable_called);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, &req->variable_called);
        /* Optional Part */
        return sccp_to_xua_opt(msg, &req->optional_start, xua);
 }
@@ -1188,7 +1188,7 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &dt1->variable_start))
                return NULL;
-       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, msg, &dt1->variable_start);
+       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, &dt1->variable_start);
        return xua;
 }

@@ -1219,13 +1219,13 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &udt->variable_called))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, msg, 
&udt->variable_called);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, &udt->variable_called);
        if (!sccp_ptr_part_consistent(msg, &udt->variable_calling))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, msg, 
&udt->variable_calling);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, &udt->variable_calling);
        if (!sccp_ptr_part_consistent(msg, &udt->variable_data))
                return NULL;
-       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, msg, &udt->variable_data);
+       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, &udt->variable_data);
        return xua;

 }
@@ -1263,13 +1263,13 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &xudt->variable_called))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, msg, 
&xudt->variable_called);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, &xudt->variable_called);
        if (!sccp_ptr_part_consistent(msg, &xudt->variable_calling))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, msg, 
&xudt->variable_calling);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, &xudt->variable_calling);
        if (!sccp_ptr_part_consistent(msg, &xudt->variable_data))
                return NULL;
-       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, msg, &xudt->variable_data);
+       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, &xudt->variable_data);
        /* Optional Part */
        return sccp_to_xua_opt(msg, &xudt->optional_start, xua);

@@ -1303,13 +1303,13 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &udts->variable_called))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, msg, 
&udts->variable_called);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, &udts->variable_called);
        if (!sccp_ptr_part_consistent(msg, &udts->variable_calling))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, msg, 
&udts->variable_calling);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, &udts->variable_calling);
        if (!sccp_ptr_part_consistent(msg, &udts->variable_data))
                return NULL;
-       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, msg, &udts->variable_data);
+       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, &udts->variable_data);
        return xua;

 }
@@ -1348,13 +1348,13 @@
        /* Variable Part */
        if (!sccp_ptr_part_consistent(msg, &xudts->variable_called))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, msg, 
&xudts->variable_called);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_DEST_ADDR, &xudts->variable_called);
        if (!sccp_ptr_part_consistent(msg, &xudts->variable_calling))
                return NULL;
-       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, msg, 
&xudts->variable_calling);
+       sccp_addr_to_sua_ptr(xua, SUA_IEI_SRC_ADDR, &xudts->variable_calling);
        if (!sccp_ptr_part_consistent(msg, &xudts->variable_data))
                return NULL;
-       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, msg, &xudts->variable_data);
+       sccp_data_to_sua_ptr(xua, SUA_IEI_DATA, &xudts->variable_data);
        /* Optional Part */
        return sccp_to_xua_opt(msg, &xudts->optional_start, xua);
 }

-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/34464?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: If4a8283614639d456d2bdf2b31873e28c68f59bc
Gerrit-Change-Number: 34464
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to