neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27903 ) Change subject: emerg call deny: do not dispatch dtap ...................................................................... emerg call deny: do not dispatch dtap These should return, not break, to skip dtap upon error. Change-Id: I065953f0380e142824ba30a842c86f9c598f89fe --- M src/osmo-bsc/gsm_04_08_rr.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/03/27903/1 diff --git a/src/osmo-bsc/gsm_04_08_rr.c b/src/osmo-bsc/gsm_04_08_rr.c index 714ef90..be41b21 100644 --- a/src/osmo-bsc/gsm_04_08_rr.c +++ b/src/osmo-bsc/gsm_04_08_rr.c @@ -1064,14 +1064,14 @@ " are not allowed in sysinfo (cfg: network / bts / rach emergency call allowed 0)\n"); lchan_release(msg->lchan, true, true, GSM48_RR_CAUSE_PROT_ERROR_UNSPC, gscon_last_eutran_plmn(msg->lchan->conn)); - break; + return; } if (!conn->sccp.msc->allow_emerg) { LOG_LCHAN(msg->lchan, LOGL_NOTICE, "MS attempts EMERGENCY SETUP, but EMERGENCY CALLS are" " denied on this BSC (cfg: msc / allow-emergency deny)\n"); lchan_release(msg->lchan, true, true, GSM48_RR_CAUSE_PROT_ERROR_UNSPC, gscon_last_eutran_plmn(msg->lchan->conn)); - break; + return; } } /* fall through */ -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27903 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I065953f0380e142824ba30a842c86f9c598f89fe Gerrit-Change-Number: 27903 Gerrit-PatchSet: 1 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-MessageType: newchange