On Wed, May 14, 2014 at 02:44:26PM +0200, Jacob Erlbeck wrote:
> >> + /* cleanup first */
> >> + if (state) {
> >> + talloc_free(state);
> >> + dst_end->rtp_process_data = NULL;
> >
> > state = NULL;
> >> + }
> >
> >
> > Or just avoid assigning state that early?
>
> This might called multiply during a single call and this way does a full
> transcoding reset every time including cleaning up and disabling it when
> called with src_end == NULL.
Yes, my question was maybe use if (dst_end->rtp_process_data) and assign
state only after it. What we should avoid is having state be a dangling
pointer.
