Bug #2872
open
OsmoMSC doesn't check if CIPHER MODE COMPLETE contains cipher that matches REQUEST
Added by laforge over 6 years ago.
Updated over 4 years ago.
Category:
A interface (general)
Description
When the MSC requests a cipher from a set of ciphers as stated in the BSSMAP CIPHER MODE REQUEST, we should check if the (bsc-)chosen cipher actually is within that set.
Care must be taken as the 'chosen algorithm' IE is optional.
A corresponding TTCN-3 test case should be developed, trying to COMPLETE with a cipher that's not in the set of those REQUESTed
- Assignee changed from 4368 to stsp
"BSSMAP CIPHER MODE REQUEST" doesn't seem to exist.
You probably meant BSSMAP CIPHER MODE CMD instead?
- Status changed from New to In Progress
This proposed patch adds a TTCN3 test for this issue:
https://gerrit.osmocom.org/#/c/osmo-ttcn3-hacks/+/12332
At present the MSC responds with a LU reject after receiving a CIPHER MODE COMPLETE with an invalid cipher.
The test looks for this LU reject and passes when it is received.
Is this the correct behaviour? Should the MSC respond in some other way?
re "command" vs "request": the things can be named differently on the layers, e.g. there's the Cipher Mode Command on BSSMAP, but Cipher Mode Request on 04.08...
The above test has been merged. neels says the current osmo-msc behaviour which the test checks for (expect a LU reject) is correct as it is.
Can this issue be closed?
this issue says that osmo-msc should check that the cipher matches the request,
and the issue implies that osmo-msc doesn't check that.
If it turns out that osmo-msc does indeed reject a mismatch already, that's nice.
How about the side thing there, if the chosen algorithm is not provided in the response?
I guess osmo-msc should accept then. Does it? (If they mismatch then, the ciphered data will not be decipherable anyway.)
Maybe duplicate the test for that situation.
- Status changed from In Progress to Feedback
- Assignee changed from stsp to neels
the patch back then was not merged. neels: Is this still an issue in the "new msc" ?
IIRC there were some problems in those patches.
Current OsmoMSC doesn't verify the chosen cipher AFAIR, only stores it. So it should still be an open issue.
Also available in: Atom
PDF