Change in osmo-hlr[master]: hlr_ussd.c: fix: reject 'structured' SS requests
Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/11341 ) Change subject: hlr_ussd.c: fix: reject 'structured' SS requests .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/11341 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia1317c5d372a42473cce65c0c985103e43be77fd Gerrit-Change-Number: 11341 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102) Gerrit-Comment-Date: Fri, 12 Oct 2018 20:01:47 + Gerrit-HasComments: No Gerrit-HasLabels: Yes
Change in osmo-hlr[master]: hlr_ussd.c: fix: reject 'structured' SS requests
Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/11341 ) Change subject: hlr_ussd.c: fix: reject 'structured' SS requests .. Patch Set 1: would be great to have a related test case in the test suite. Thanks! -- To view, visit https://gerrit.osmocom.org/11341 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia1317c5d372a42473cce65c0c985103e43be77fd Gerrit-Change-Number: 11341 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102) Gerrit-Comment-Date: Fri, 12 Oct 2018 20:02:07 + Gerrit-HasComments: No Gerrit-HasLabels: No
Change in osmo-hlr[master]: hlr_ussd.c: fix: reject 'structured' SS requests
Harald Welte has submitted this change and it was merged. ( https://gerrit.osmocom.org/11341 ) Change subject: hlr_ussd.c: fix: reject 'structured' SS requests .. hlr_ussd.c: fix: reject 'structured' SS requests As we don't store any SS related information (e.g. call forwarding preferences) in the database, we don't handle 'structured' SS requests at all. Let's reject them by sending error message with FACILITY_NOT_SUPPORTED code. Change-Id: Ia1317c5d372a42473cce65c0c985103e43be77fd Related: OS#3651 --- M src/hlr_ussd.c 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified Harald Welte: Looks good to me, approved diff --git a/src/hlr_ussd.c b/src/hlr_ussd.c index c96f47c..ba373f3 100644 --- a/src/hlr_ussd.c +++ b/src/hlr_ussd.c @@ -399,8 +399,15 @@ LOGPSS(ss, LOGL_INFO, "SS CompType=%s, OpCode=%s\n", gsm0480_comp_type_name(comp_type), gsm0480_op_code_name(req->opcode)); - /* FIXME */ - return 0; + + /** +* FIXME: As we don't store any SS related information +* (e.g. call forwarding preferences) in the database, +* we don't handle "structured" SS requests at all. +*/ + LOGPSS(ss, LOGL_NOTICE, "Structured SS requests are not supported, rejecting...\n"); + ss_tx_error(ss, req->invoke_id, GSM0480_ERR_CODE_FACILITY_NOT_SUPPORTED); + return -ENOTSUP; } /* Handle a USSD GSUP message for a given SS Session received from VLR or EUSE */ -- To view, visit https://gerrit.osmocom.org/11341 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: Ia1317c5d372a42473cce65c0c985103e43be77fd Gerrit-Change-Number: 11341 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder (102)
Change in osmo-hlr[master]: hlr_ussd.c: fix: reject 'structured' SS requests
Vadim Yanitskiy has uploaded this change for review. ( https://gerrit.osmocom.org/11341 Change subject: hlr_ussd.c: fix: reject 'structured' SS requests .. hlr_ussd.c: fix: reject 'structured' SS requests As we don't store any SS related information (e.g. call forwarding preferences) in the database, we don't handle 'structured' SS requests at all. Let's reject them by sending error message with FACILITY_NOT_SUPPORTED code. Change-Id: Ia1317c5d372a42473cce65c0c985103e43be77fd Related: OS#3651 --- M src/hlr_ussd.c 1 file changed, 9 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/41/11341/1 diff --git a/src/hlr_ussd.c b/src/hlr_ussd.c index c96f47c..ba373f3 100644 --- a/src/hlr_ussd.c +++ b/src/hlr_ussd.c @@ -399,8 +399,15 @@ LOGPSS(ss, LOGL_INFO, "SS CompType=%s, OpCode=%s\n", gsm0480_comp_type_name(comp_type), gsm0480_op_code_name(req->opcode)); - /* FIXME */ - return 0; + + /** +* FIXME: As we don't store any SS related information +* (e.g. call forwarding preferences) in the database, +* we don't handle "structured" SS requests at all. +*/ + LOGPSS(ss, LOGL_NOTICE, "Structured SS requests are not supported, rejecting...\n"); + ss_tx_error(ss, req->invoke_id, GSM0480_ERR_CODE_FACILITY_NOT_SUPPORTED); + return -ENOTSUP; } /* Handle a USSD GSUP message for a given SS Session received from VLR or EUSE */ -- To view, visit https://gerrit.osmocom.org/11341 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ia1317c5d372a42473cce65c0c985103e43be77fd Gerrit-Change-Number: 11341 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy