Attention is currently required from: daniel, laforge, neels.

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39224?usp=email )

Change subject: mgw: CRCX: Split mgcp header pars parsing into a previous step ......................................................................

Patch Set 5:

(1 comment)

File src/libosmo-mgcp/mgcp_protocol.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/39224/comment/3a9e3625_8ff6bd6b?usp=... : PS4, Line 710: if (osmux_init(trunk) < 0) {

if this is called due to whatever endp/conn triggerer and this function fails, then it's up to the c […]

BTW, in this case, it doesn't really matter at all which subscriber or endp triggered it, because it really has nothing to do with the fact of initiating the ports. It's just that we are lazy-initializing the shared socket.

This is actually something which should be done at startup or in an improved way anyway if we always get to implementer further osmux improvements I wrote in redmine.