mgcp_protocol: assert endp when it becomes mandatory

The logic when an endp pointer is guranteed and when we are able to
process the request without the endp pointer populated is qute complex.
This shows up as a bug to coverity.

Change-Id: I1d4221f2df13c43321d5466534485cf21f0d9010
Fixes: CID#237088
This commit is contained in:
Philipp Maier 2021-07-23 10:58:19 +02:00 committed by dexter
parent 41ab87f67c
commit ce18705875
1 changed files with 4 additions and 0 deletions

View File

@ -1467,6 +1467,10 @@ static struct msgb *handle_delete_con(struct mgcp_request_data *rq)
return create_ok_response(NULL, 200, "DLCX", pdata->trans);
}
/* The logic does not permit to go past this point without having the
* the endp pointer populated. */
OSMO_ASSERT(endp);
/* When no connection id is supplied, we will interpret this as a
* wildcarded DLCX that refers to the selected endpoint. This means
* that we drop all connections on that specific endpoint at once.