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.