1
0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-01-19 05:49:09 +02:00

avcodec/interplayvideo: cosmetics

This commit is contained in:
Paul B Mahol 2019-10-24 15:30:08 +02:00
parent f166951d6e
commit 9bf5e7992f

View File

@ -1231,83 +1231,83 @@ static int ipvideo_decode_frame(AVCodecContext *avctx,
s->decoding_map_size = AV_RL16(buf + 4); s->decoding_map_size = AV_RL16(buf + 4);
s->skip_map_size = AV_RL16(buf + 6); s->skip_map_size = AV_RL16(buf + 6);
switch(frame_format) { switch (frame_format) {
case 0x06: case 0x06:
if (s->decoding_map_size) { if (s->decoding_map_size) {
av_log(avctx, AV_LOG_ERROR, "Decoding map for format 0x06\n"); av_log(avctx, AV_LOG_ERROR, "Decoding map for format 0x06\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (s->skip_map_size) { if (s->skip_map_size) {
av_log(avctx, AV_LOG_ERROR, "Skip map for format 0x06\n"); av_log(avctx, AV_LOG_ERROR, "Skip map for format 0x06\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (s->is_16bpp) { if (s->is_16bpp) {
av_log(avctx, AV_LOG_ERROR, "Video format 0x06 does not support 16bpp movies\n"); av_log(avctx, AV_LOG_ERROR, "Video format 0x06 does not support 16bpp movies\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
/* Decoding map for 0x06 frame format is at the top of pixeldata */ /* Decoding map for 0x06 frame format is at the top of pixeldata */
s->decoding_map_size = ((s->avctx->width / 8) * (s->avctx->height / 8)) * 2; s->decoding_map_size = ((s->avctx->width / 8) * (s->avctx->height / 8)) * 2;
s->decoding_map = buf + 8 + 14; /* 14 bits of op data */ s->decoding_map = buf + 8 + 14; /* 14 bits of op data */
video_data_size -= s->decoding_map_size + 14; video_data_size -= s->decoding_map_size + 14;
if (video_data_size <= 0 || s->decoding_map_size == 0) if (video_data_size <= 0 || s->decoding_map_size == 0)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
if (buf_size < 8 + s->decoding_map_size + 14 + video_data_size) if (buf_size < 8 + s->decoding_map_size + 14 + video_data_size)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
bytestream2_init(&s->stream_ptr, buf + 8 + s->decoding_map_size + 14, video_data_size); bytestream2_init(&s->stream_ptr, buf + 8 + s->decoding_map_size + 14, video_data_size);
break; break;
case 0x10: case 0x10:
if (! s->decoding_map_size) { if (! s->decoding_map_size) {
av_log(avctx, AV_LOG_ERROR, "Empty decoding map for format 0x10\n"); av_log(avctx, AV_LOG_ERROR, "Empty decoding map for format 0x10\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (! s->skip_map_size) { if (! s->skip_map_size) {
av_log(avctx, AV_LOG_ERROR, "Empty skip map for format 0x10\n"); av_log(avctx, AV_LOG_ERROR, "Empty skip map for format 0x10\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (s->is_16bpp) { if (s->is_16bpp) {
av_log(avctx, AV_LOG_ERROR, "Video format 0x10 does not support 16bpp movies\n"); av_log(avctx, AV_LOG_ERROR, "Video format 0x10 does not support 16bpp movies\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (buf_size < 8 + video_data_size + s->decoding_map_size + s->skip_map_size) if (buf_size < 8 + video_data_size + s->decoding_map_size + s->skip_map_size)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
bytestream2_init(&s->stream_ptr, buf + 8, video_data_size); bytestream2_init(&s->stream_ptr, buf + 8, video_data_size);
s->decoding_map = buf + 8 + video_data_size; s->decoding_map = buf + 8 + video_data_size;
s->skip_map = buf + 8 + video_data_size + s->decoding_map_size; s->skip_map = buf + 8 + video_data_size + s->decoding_map_size;
break; break;
case 0x11: case 0x11:
if (! s->decoding_map_size) { if (! s->decoding_map_size) {
av_log(avctx, AV_LOG_ERROR, "Empty decoding map for format 0x11\n"); av_log(avctx, AV_LOG_ERROR, "Empty decoding map for format 0x11\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (s->skip_map_size) { if (s->skip_map_size) {
av_log(avctx, AV_LOG_ERROR, "Skip map for format 0x11\n"); av_log(avctx, AV_LOG_ERROR, "Skip map for format 0x11\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (buf_size < 8 + video_data_size + s->decoding_map_size) if (buf_size < 8 + video_data_size + s->decoding_map_size)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
bytestream2_init(&s->stream_ptr, buf + 8, video_data_size); bytestream2_init(&s->stream_ptr, buf + 8, video_data_size);
s->decoding_map = buf + 8 + video_data_size; s->decoding_map = buf + 8 + video_data_size;
break; break;
default: default:
av_log(avctx, AV_LOG_ERROR, "Frame type 0x%02X unsupported\n", frame_format); av_log(avctx, AV_LOG_ERROR, "Frame type 0x%02X unsupported\n", frame_format);
} }
/* ensure we can't overread the packet */ /* ensure we can't overread the packet */
@ -1330,16 +1330,16 @@ static int ipvideo_decode_frame(AVCodecContext *avctx,
} }
} }
switch(frame_format) { switch (frame_format) {
case 0x06: case 0x06:
ipvideo_decode_format_06_opcodes(s, frame); ipvideo_decode_format_06_opcodes(s, frame);
break; break;
case 0x10: case 0x10:
ipvideo_decode_format_10_opcodes(s, frame); ipvideo_decode_format_10_opcodes(s, frame);
break; break;
case 0x11: case 0x11:
ipvideo_decode_format_11_opcodes(s, frame); ipvideo_decode_format_11_opcodes(s, frame);
break; break;
} }
*got_frame = send_buffer; *got_frame = send_buffer;