This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Vadim Yanitskiy gerrit-no-reply at lists.osmocom.orgVadim Yanitskiy has submitted this change and it was merged. ( https://gerrit.osmocom.org/11989 ) Change subject: gsupclient: introduce osmo_gsup_client_enc_send() ...................................................................... gsupclient: introduce osmo_gsup_client_enc_send() Several parts of OsmoMSC (e.g. GSM 04.11, 09.11, etc.) are dealing with GSUP message encoding and sending towards OsmoHLR. In order to avoid code duplication, let's have a shared function here. Change-Id: I0589ff27933e9bca2bcf93b8259004935778db8f --- M include/osmocom/gsupclient/gsup_client.h M src/gsupclient/gsup_client.c 2 files changed, 39 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified Stefan Sperling: Looks good to me, but someone else must approve Vadim Yanitskiy: Looks good to me, approved diff --git a/include/osmocom/gsupclient/gsup_client.h b/include/osmocom/gsupclient/gsup_client.h index 95163cd..154e3e0 100644 --- a/include/osmocom/gsupclient/gsup_client.h +++ b/include/osmocom/gsupclient/gsup_client.h @@ -24,6 +24,7 @@ #include <osmocom/core/timer.h> #include <osmocom/gsm/oap_client.h> #include <osmocom/gsm/ipa.h> +#include <osmocom/gsm/gsup.h> /* a loss of GSUP between MSC and HLR is considered quite serious, let's try to recover as quickly as * possible. Even one new connection attempt per second should be quite acceptable until the link is @@ -70,5 +71,7 @@ void osmo_gsup_client_destroy(struct osmo_gsup_client *gsupc); int osmo_gsup_client_send(struct osmo_gsup_client *gsupc, struct msgb *msg); +int osmo_gsup_client_enc_send(struct osmo_gsup_client *gsupc, + const struct osmo_gsup_message *gsup_msg); struct msgb *osmo_gsup_client_msgb_alloc(void); diff --git a/src/gsupclient/gsup_client.c b/src/gsupclient/gsup_client.c index f259bdc..c8408fd 100644 --- a/src/gsupclient/gsup_client.c +++ b/src/gsupclient/gsup_client.c @@ -360,6 +360,42 @@ return 0; } +/*! Encode and send a GSUP message. + * \param[in] gsupc GSUP client. + * \param[in] gsup_msg GSUP message to be sent. + * \returns 0 in case of success, negative on error. + */ +int osmo_gsup_client_enc_send(struct osmo_gsup_client *gsupc, + const struct osmo_gsup_message *gsup_msg) +{ + struct msgb *gsup_msgb; + int rc; + + gsup_msgb = osmo_gsup_client_msgb_alloc(); + if (!gsup_msgb) { + LOGP(DLGSUP, LOGL_ERROR, "Couldn't allocate GSUP message\n"); + return -ENOMEM; + } + + rc = osmo_gsup_encode(gsup_msgb, gsup_msg); + if (rc) { + LOGP(DLGSUP, LOGL_ERROR, "Couldn't encode GSUP message\n"); + goto error; + } + + rc = osmo_gsup_client_send(gsupc, gsup_msgb); + if (rc) { + LOGP(DLGSUP, LOGL_ERROR, "Couldn't send GSUP message\n"); + goto error; + } + + return 0; + +error: + talloc_free(gsup_msgb); + return rc; +} + struct msgb *osmo_gsup_client_msgb_alloc(void) { return msgb_alloc_headroom(4000, 64, __func__); -- To view, visit https://gerrit.osmocom.org/11989 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0589ff27933e9bca2bcf93b8259004935778db8f Gerrit-Change-Number: 11989 Gerrit-PatchSet: 9 Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Stefan Sperling <stsp at stsp.name> Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181219/0e753d71/attachment.htm>