mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-01-13 21:28:01 +02:00
xxan: return meaningful error codes.
This commit is contained in:
parent
aaf208329f
commit
4aebb8d999
@ -133,7 +133,7 @@ static int xan_unpack(XanContext *s,
|
|||||||
}
|
}
|
||||||
if (dest + size + size2 > dest_end ||
|
if (dest + size + size2 > dest_end ||
|
||||||
dest - orig_dest + size < back)
|
dest - orig_dest + size < back)
|
||||||
return -1;
|
return AVERROR_INVALIDDATA;
|
||||||
bytestream2_get_buffer(&s->gb, dest, size);
|
bytestream2_get_buffer(&s->gb, dest, size);
|
||||||
dest += size;
|
dest += size;
|
||||||
av_memcpy_backptr(dest, back, size2);
|
av_memcpy_backptr(dest, back, size2);
|
||||||
@ -143,7 +143,7 @@ static int xan_unpack(XanContext *s,
|
|||||||
|
|
||||||
size = finish ? opcode & 3 : ((opcode & 0x1f) << 2) + 4;
|
size = finish ? opcode & 3 : ((opcode & 0x1f) << 2) + 4;
|
||||||
if (dest_end - dest < size)
|
if (dest_end - dest < size)
|
||||||
return -1;
|
return AVERROR_INVALIDDATA;
|
||||||
bytestream2_get_buffer(&s->gb, dest, size);
|
bytestream2_get_buffer(&s->gb, dest, size);
|
||||||
dest += size;
|
dest += size;
|
||||||
if (finish)
|
if (finish)
|
||||||
@ -167,7 +167,7 @@ static int xan_decode_chroma(AVCodecContext *avctx, unsigned chroma_off)
|
|||||||
return 0;
|
return 0;
|
||||||
if (chroma_off + 4 >= bytestream2_get_bytes_left(&s->gb)) {
|
if (chroma_off + 4 >= bytestream2_get_bytes_left(&s->gb)) {
|
||||||
av_log(avctx, AV_LOG_ERROR, "Invalid chroma block position\n");
|
av_log(avctx, AV_LOG_ERROR, "Invalid chroma block position\n");
|
||||||
return -1;
|
return AVERROR_INVALIDDATA;
|
||||||
}
|
}
|
||||||
bytestream2_seek(&s->gb, chroma_off + 4, SEEK_SET);
|
bytestream2_seek(&s->gb, chroma_off + 4, SEEK_SET);
|
||||||
mode = bytestream2_get_le16(&s->gb);
|
mode = bytestream2_get_le16(&s->gb);
|
||||||
@ -178,7 +178,7 @@ static int xan_decode_chroma(AVCodecContext *avctx, unsigned chroma_off)
|
|||||||
|
|
||||||
if (offset >= bytestream2_get_bytes_left(&s->gb)) {
|
if (offset >= bytestream2_get_bytes_left(&s->gb)) {
|
||||||
av_log(avctx, AV_LOG_ERROR, "Invalid chroma block offset\n");
|
av_log(avctx, AV_LOG_ERROR, "Invalid chroma block offset\n");
|
||||||
return -1;
|
return AVERROR_INVALIDDATA;
|
||||||
}
|
}
|
||||||
|
|
||||||
bytestream2_skip(&s->gb, offset);
|
bytestream2_skip(&s->gb, offset);
|
||||||
@ -186,7 +186,7 @@ static int xan_decode_chroma(AVCodecContext *avctx, unsigned chroma_off)
|
|||||||
dec_size = xan_unpack(s, s->scratch_buffer, s->buffer_size);
|
dec_size = xan_unpack(s, s->scratch_buffer, s->buffer_size);
|
||||||
if (dec_size < 0) {
|
if (dec_size < 0) {
|
||||||
av_log(avctx, AV_LOG_ERROR, "Chroma unpacking failed\n");
|
av_log(avctx, AV_LOG_ERROR, "Chroma unpacking failed\n");
|
||||||
return -1;
|
return dec_size;
|
||||||
}
|
}
|
||||||
|
|
||||||
U = s->pic.data[1];
|
U = s->pic.data[1];
|
||||||
@ -384,7 +384,7 @@ static int xan_decode_frame(AVCodecContext *avctx,
|
|||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
av_log(avctx, AV_LOG_ERROR, "Unknown frame type %d\n", ftype);
|
av_log(avctx, AV_LOG_ERROR, "Unknown frame type %d\n", ftype);
|
||||||
return -1;
|
return AVERROR_INVALIDDATA;
|
||||||
}
|
}
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
Loading…
Reference in New Issue
Block a user