1
0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-06-30 22:24:04 +02:00

avcodec/mpeg4_unpack_bframes: cache input packets directly

Avoids unnecessary allocs+copies and makes the code slightly simpler.

Signed-off-by: James Almer <jamrial@gmail.com>
This commit is contained in:
James Almer
2018-03-20 21:59:10 -03:00
parent d8e92a89ed
commit bd90a2ec04

View File

@ -24,8 +24,7 @@
#include "mpeg4video.h" #include "mpeg4video.h"
typedef struct UnpackBFramesBSFContext { typedef struct UnpackBFramesBSFContext {
uint8_t *b_frame_buf; AVPacket *b_frame;
int b_frame_buf_size;
} UnpackBFramesBSFContext; } UnpackBFramesBSFContext;
/* search next start code */ /* search next start code */
@ -71,18 +70,6 @@ static void scan_buffer(const uint8_t *buf, int buf_size,
} }
} }
/* allocate new buffer and copy size bytes from src */
static uint8_t *create_new_buffer(const uint8_t *src, int size) {
uint8_t *dst = av_malloc(size + AV_INPUT_BUFFER_PADDING_SIZE);
if (dst) {
memcpy(dst, src, size);
memset(dst + size, 0, AV_INPUT_BUFFER_PADDING_SIZE);
}
return dst;
}
static int mpeg4_unpack_bframes_filter(AVBSFContext *ctx, AVPacket *out) static int mpeg4_unpack_bframes_filter(AVBSFContext *ctx, AVPacket *out)
{ {
UnpackBFramesBSFContext *s = ctx->priv_data; UnpackBFramesBSFContext *s = ctx->priv_data;
@ -97,20 +84,18 @@ static int mpeg4_unpack_bframes_filter(AVBSFContext *ctx, AVPacket *out)
av_log(ctx, AV_LOG_DEBUG, "Found %d VOP startcode(s) in this packet.\n", nb_vop); av_log(ctx, AV_LOG_DEBUG, "Found %d VOP startcode(s) in this packet.\n", nb_vop);
if (pos_vop2 >= 0) { if (pos_vop2 >= 0) {
if (s->b_frame_buf) { if (s->b_frame->data) {
av_log(ctx, AV_LOG_WARNING, av_log(ctx, AV_LOG_WARNING,
"Missing one N-VOP packet, discarding one B-frame.\n"); "Missing one N-VOP packet, discarding one B-frame.\n");
av_freep(&s->b_frame_buf); av_packet_unref(s->b_frame);
s->b_frame_buf_size = 0;
} }
/* store the packed B-frame in the BSFContext */ /* store the packed B-frame in the BSFContext */
s->b_frame_buf_size = in->size - pos_vop2; ret = av_packet_ref(s->b_frame, in);
s->b_frame_buf = create_new_buffer(in->data + pos_vop2, s->b_frame_buf_size); if (ret < 0) {
if (!s->b_frame_buf) {
s->b_frame_buf_size = 0;
ret = AVERROR(ENOMEM);
goto fail; goto fail;
} }
s->b_frame->size -= pos_vop2;
s->b_frame->data += pos_vop2;
} }
if (nb_vop > 2) { if (nb_vop > 2) {
@ -118,31 +103,22 @@ static int mpeg4_unpack_bframes_filter(AVBSFContext *ctx, AVPacket *out)
"Found %d VOP headers in one packet, only unpacking one.\n", nb_vop); "Found %d VOP headers in one packet, only unpacking one.\n", nb_vop);
} }
if (nb_vop == 1 && s->b_frame_buf) { if (nb_vop == 1 && s->b_frame->data) {
/* use frame from BSFContext */ /* use frame from BSFContext */
av_packet_move_ref(out, s->b_frame);
/* use properties from current input packet */
ret = av_packet_copy_props(out, in); ret = av_packet_copy_props(out, in);
if (ret < 0) { if (ret < 0) {
goto fail; goto fail;
} }
ret = av_packet_from_data(out, s->b_frame_buf, s->b_frame_buf_size);
if (ret < 0) {
goto fail;
}
if (in->size <= MAX_NVOP_SIZE) { if (in->size <= MAX_NVOP_SIZE) {
/* N-VOP */ /* N-VOP */
av_log(ctx, AV_LOG_DEBUG, "Skipping N-VOP.\n"); av_log(ctx, AV_LOG_DEBUG, "Skipping N-VOP.\n");
s->b_frame_buf = NULL;
s->b_frame_buf_size = 0;
} else { } else {
/* copy packet into BSFContext */ /* copy packet into BSFContext */
s->b_frame_buf_size = in->size; av_packet_move_ref(s->b_frame, in);
s->b_frame_buf = create_new_buffer(in->data, in->size);
if (!s->b_frame_buf) {
s->b_frame_buf_size = 0;
ret = AVERROR(ENOMEM);
goto fail;
}
} }
} else if (nb_vop >= 2) { } else if (nb_vop >= 2) {
/* use first frame of the packet */ /* use first frame of the packet */
@ -171,6 +147,12 @@ fail:
static int mpeg4_unpack_bframes_init(AVBSFContext *ctx) static int mpeg4_unpack_bframes_init(AVBSFContext *ctx)
{ {
UnpackBFramesBSFContext *s = ctx->priv_data;
s->b_frame = av_packet_alloc();
if (!s->b_frame)
return AVERROR(ENOMEM);
if (ctx->par_in->extradata) { if (ctx->par_in->extradata) {
int pos_p_ext = -1; int pos_p_ext = -1;
scan_buffer(ctx->par_in->extradata, ctx->par_in->extradata_size, &pos_p_ext, NULL, NULL); scan_buffer(ctx->par_in->extradata, ctx->par_in->extradata_size, &pos_p_ext, NULL, NULL);
@ -187,7 +169,7 @@ static int mpeg4_unpack_bframes_init(AVBSFContext *ctx)
static void mpeg4_unpack_bframes_close(AVBSFContext *bsfc) static void mpeg4_unpack_bframes_close(AVBSFContext *bsfc)
{ {
UnpackBFramesBSFContext *ctx = bsfc->priv_data; UnpackBFramesBSFContext *ctx = bsfc->priv_data;
av_freep(&ctx->b_frame_buf); av_packet_free(&ctx->b_frame);
} }
static const enum AVCodecID codec_ids[] = { static const enum AVCodecID codec_ids[] = {