Attention is currently required from: pespin. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/27916 )
Change subject: CM Serv Rej: do not crash on use count mismatch ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > does it make sense to send a CM Serv Rej despite no CM Serv Request was sent > in first place? Probabl […] IIUC the sequence of events is: CM Serv Req received, MSC does not allow CM Serv Req for some reason (hence no use count), still wants to send CM Serv Rej message, there is no use count to decrease, entire MSC shuts down. I think some time could be spent on reviewing the CM Service use tokens, but that's likely nontrivial (IOW i'm lazy). What I'm sure of is that we don't want to shut down the entire MSC just because some minor use count got confused. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/27916 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I2009df42d1f27ec0d011e22bfc46dbc17afe7239 Gerrit-Change-Number: 27916 Gerrit-PatchSet: 1 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Mon, 25 Apr 2022 21:31:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Gerrit-MessageType: comment
