pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/35631?usp=email )


Change subject: vlr: Use new libosmogsm struct osmo_gsup_pdp_info fields
......................................................................

vlr: Use new libosmogsm struct osmo_gsup_pdp_info fields

This also makes sure it doesn't compile against older libosmogsm gsup
versions which would break ABI.

Related: OS#6091
Depends: libosmocore.git Change-Id 70be3560659c58f24b8db529c4fc85da4bb0ec04
Change-Id: Ia002fd6e0334d56de34d352a0bf1a8604e2e9fd3
---
M TODO-RELEASE
M src/libvlr/vlr.c
2 files changed, 22 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/31/35631/1

diff --git a/TODO-RELEASE b/TODO-RELEASE
index d0852fc..8b07972 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -7,3 +7,4 @@
 # If any interfaces have been added since the last public release: c:r:a + 1.
 # If any interfaces have been removed or changed since the last public 
release: c:r:0.
 #library       what            description / commit summary line
+libosmogsm  >1.9.0  ABI breakage in struct osmo_gsup_pdp_info, use new fields 
pdp_type_* and pdp_address.
\ No newline at end of file
diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c
index fb034cf..e2e02bf 100644
--- a/src/libvlr/vlr.c
+++ b/src/libvlr/vlr.c
@@ -731,7 +731,9 @@
        struct llist_head       list;

        unsigned int            context_id;
-       uint16_t                pdp_type;
+       enum gsm48_pdp_type_org pdp_type_org;
+       enum gsm48_pdp_type_nr  pdp_type_nr;
+       struct osmo_sockaddr    pdp_address[2];
        char                    apn_str[GSM_APN_LENGTH];
        uint8_t                 qos_subscribed[20];
        size_t                  qos_subscribed_len;
@@ -1024,7 +1026,10 @@
                }

                OSMO_ASSERT(pdp_data != NULL);
-               pdp_data->pdp_type = pdp_info->pdp_type;
+               pdp_data->pdp_type_org = pdp_info->pdp_type_org;
+               pdp_data->pdp_type_nr = pdp_info->pdp_type_nr;
+               memcpy(&pdp_data->pdp_address[0], &pdp_info->pdp_address[0], 
sizeof(pdp_data->pdp_address[0]));
+               memcpy(&pdp_data->pdp_address[1], &pdp_info->pdp_address[1], 
sizeof(pdp_data->pdp_address[1]));
                osmo_apn_to_str(pdp_data->apn_str,
                                pdp_info->apn_enc, pdp_info->apn_enc_len);
                memcpy(pdp_data->qos_subscribed, pdp_info->qos_enc, 
pdp_info->qos_enc_len);

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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ia002fd6e0334d56de34d352a0bf1a8604e2e9fd3
Gerrit-Change-Number: 35631
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to