h264: move freeing the escaped RBSP buffer to free_context()

It does not logically belong in free_tables(), since it's not allocated
in alloc_tables() and its size has nothing to do with the frame size.
This commit is contained in:
Anton Khirnov 2015-04-06 21:00:42 +02:00
parent ee62b364be
commit 11f024ef0a
3 changed files with 8 additions and 11 deletions

View File

@ -332,7 +332,7 @@ static int decode_rbsp_trailing(H264Context *h, const uint8_t *src)
return 0;
}
void ff_h264_free_tables(H264Context *h, int free_rbsp)
void ff_h264_free_tables(H264Context *h)
{
int i;
@ -372,11 +372,6 @@ void ff_h264_free_tables(H264Context *h, int free_rbsp)
sl->edge_emu_buffer_allocated = 0;
sl->top_borders_allocated[0] = 0;
sl->top_borders_allocated[1] = 0;
if (free_rbsp) {
av_freep(&sl->rbsp_buffer);
sl->rbsp_buffer_size = 0;
}
}
}
@ -433,7 +428,7 @@ int ff_h264_alloc_tables(H264Context *h)
return 0;
fail:
ff_h264_free_tables(h, 1);
ff_h264_free_tables(h);
return AVERROR(ENOMEM);
}
@ -1065,7 +1060,7 @@ static void flush_dpb(AVCodecContext *avctx)
h->mb_y = 0;
ff_h264_free_tables(h, 1);
ff_h264_free_tables(h);
h->context_initialized = 0;
}
@ -1701,7 +1696,7 @@ av_cold void ff_h264_free_context(H264Context *h)
{
int i;
ff_h264_free_tables(h, 1); // FIXME cleanup init stuff perhaps
ff_h264_free_tables(h); // FIXME cleanup init stuff perhaps
if (h->DPB) {
for (i = 0; i < H264_MAX_PICTURE_COUNT; i++)
@ -1711,6 +1706,8 @@ av_cold void ff_h264_free_context(H264Context *h)
h->cur_pic_ptr = NULL;
for (i = 0; i < h->nb_slice_ctx; i++)
av_freep(&h->slice_ctx[i].rbsp_buffer);
av_freep(&h->slice_ctx);
h->nb_slice_ctx = 0;

View File

@ -1084,6 +1084,6 @@ int ff_h264_update_thread_context(AVCodecContext *dst,
void ff_h264_flush_change(H264Context *h);
void ff_h264_free_tables(H264Context *h, int free_rbsp);
void ff_h264_free_tables(H264Context *h);
#endif /* AVCODEC_H264_H */

View File

@ -994,7 +994,7 @@ static int h264_slice_header_init(H264Context *h, int reinit)
}
if (reinit)
ff_h264_free_tables(h, 0);
ff_h264_free_tables(h);
h->first_field = 0;
h->prev_interlaced_frame = 1;