Attention is currently required from: dexter, pespin.

laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/osmo-msc/+/43237?usp=email )

Change subject: sgs_iface: clean up code in function decode_mme_name ......................................................................

Patch Set 4:

(1 comment)

File src/libmsc/sgs_iface.c:

https://gerrit.osmocom.org/c/osmo-msc/+/43237/comment/e192a2db_b2a81dff?usp=... : PS2, Line 178:

I have a problem with alternate path: At the beginning of this function we require the contents of S […]

the questions is what such proprietary formats are, how they look like, used by whom, and why there was any attempt to support them at all.

This code was merged as part of ``` commit e2bd9eb37d8160b436a5a1bffc14690321f40ae6 Author: Omar Ramadan omar.ramadan93@gmail.com Date: Fri Apr 12 09:03:39 2019 -0700 ```

So someone had a need at some point. I would be fine with not enforcing a minimum length of 55 digits at all, and possibly printing just a warning about it (if at all). To the MSC, AFAICT it doesn't really matter. We just treat the MME name as an opaque identifier and don't ever try to parse it, right?