From bb4a310bb85f43e62240145a656b1e5285b14239 Mon Sep 17 00:00:00 2001 From: Martin Storsjö Date: Tue, 24 Feb 2015 13:07:57 +0200 Subject: rtpdec: Don't free the payload context in the .free function MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This makes it more consistent with depacketizers that don't have any .free function at all, where the payload context is freed by the surrounding framework. Always free the context in the surrounding framework, having the individual depacketizers only free any data they've specifically allocated themselves. This is similar to how this works for demuxer/muxers/codecs - a component shouldn't free the priv_data that the framework has allocated for it. Signed-off-by: Martin Storsjö --- libavformat/rtpdec_h261.c | 3 --- 1 file changed, 3 deletions(-) (limited to 'libavformat/rtpdec_h261.c') diff --git a/libavformat/rtpdec_h261.c b/libavformat/rtpdec_h261.c index dac76c8ad7..b2d2a53f1c 100644 --- a/libavformat/rtpdec_h261.c +++ b/libavformat/rtpdec_h261.c @@ -50,9 +50,6 @@ static av_cold void h261_free_context(PayloadContext *pl_ctx) if (pl_ctx->buf) { h261_free_dyn_buffer(&pl_ctx->buf); } - - /* free context */ - av_free(pl_ctx); } static int h261_handle_packet(AVFormatContext *ctx, PayloadContext *rtp_h261_ctx, -- cgit v1.2.3