This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.
Holger Hans Peter Freyther holger at freyther.deOn Thu, May 15, 2014 at 10:29:10AM +0200, Jacob Erlbeck wrote: Hi, > +static int set_audio_info(void *ctx, struct mgcp_rtp_end *rtp, > + int payload_type, const char *audio_name) > +{ ... > + talloc_free(rtp->subtype_name); > + rtp->subtype_name = NULL; > + talloc_free(rtp->audio_name); > + rtp->audio_name = NULL; ... > @@ -600,29 +646,18 @@ static int parse_sdp_data(struct mgcp_rtp_end *rtp, struct mgcp_parse_data *p) ... > + set_audio_info(p->cfg, rtp, payload, audio_name); > } else if (sscanf(line, "a=ptime:%d-%d", ... > + set_audio_info(p->cfg, rtp, audio_payload, NULL); > - endp->bts_end.payload_type = tcfg->audio_payload; > + set_audio_info(p->cfg, &endp->bts_end, tcfg->audio_payload, tcfg->audio_name); I raised this before as well. So for a CRCX MDCX DLCX procedure. We might have endp->bts_end->subtype_name still with an allocated string. Please make reset endpoint give up all the data.