Attention is currently required from: dexter.

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

Change subject: mgcp_client_fsm: fix inconsistent API (param_present, param). ......................................................................

Patch Set 2:

(3 comments)

File include/osmocom/mgcp_client/mgcp_client.h:

https://gerrit.osmocom.org/c/osmo-mgw/+/34351/comment/79964b57_02605264 PS2, Line 121: struct mgcp_msg { If this struct is being allocated outside of the library, now it's for sure the moment to add an alloc function! :)

File include/osmocom/mgcp_client/mgcp_client_fsm.h:

https://gerrit.osmocom.org/c/osmo-mgw/+/34351/comment/09ee5baf_57243b68 PS2, Line 70: struct mgcp_codec_param2 codecs_param[MGCP_MAX_CODECS]; as soon as you add a field to "struct mgcp_codec_param2", the size of the struct will and change any program using the old size will really be fucked up doing weird stuff on this array. This should be given a second thought in order to improve the API so that we don't go breaking stuff every time we want to support something new.

We should start by having an API to allocate this struct from within the library...

File src/libosmo-mgcp-client/mgcp_client.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/34351/comment/e000e601_6828c6b9 PS2, Line 1376: LOGPMGW(mgcp, LOGL_DEBUG, "using depreacted struct member mgcp_msg->param, please use mgcp_msg->codecs_param!\n"); only debug?