Commit f0629472 authored by Paul B Mahol's avatar Paul B Mahol

avcodec/qdrw: do better w/h parsing for direct bit packing

Apparently using 0x0001 opcode solely is not correct.
Try this instead.
Signed-off-by: 's avatarPaul B Mahol <onemda@gmail.com>
parent dcd3418a
...@@ -310,7 +310,7 @@ static int decode_frame(AVCodecContext *avctx, ...@@ -310,7 +310,7 @@ static int decode_frame(AVCodecContext *avctx,
AVFrame * const p = data; AVFrame * const p = data;
GetByteContext gbc; GetByteContext gbc;
int colors; int colors;
int w, h, x0, y0, x1, y1, ret; int w, h, ret;
int ver; int ver;
bytestream2_init(&gbc, avpkt->data, avpkt->size); bytestream2_init(&gbc, avpkt->data, avpkt->size);
...@@ -355,14 +355,7 @@ static int decode_frame(AVCodecContext *avctx, ...@@ -355,14 +355,7 @@ static int decode_frame(AVCodecContext *avctx,
switch(opcode) { switch(opcode) {
case CLIP: case CLIP:
bytestream2_get_be16(&gbc); bytestream2_skip(&gbc, 10);
y0 = bytestream2_get_be16(&gbc);
x0 = bytestream2_get_be16(&gbc);
y1 = bytestream2_get_be16(&gbc);
x1 = bytestream2_get_be16(&gbc);
ret = ff_set_dimensions(avctx, x1 - x0, y1 - y0);
if (ret < 0)
return ret;
break; break;
case PACKBITSRECT: case PACKBITSRECT:
case PACKBITSRGN: case PACKBITSRGN:
...@@ -437,7 +430,15 @@ static int decode_frame(AVCodecContext *avctx, ...@@ -437,7 +430,15 @@ static int decode_frame(AVCodecContext *avctx,
return AVERROR_PATCHWELCOME; return AVERROR_PATCHWELCOME;
} }
bytestream2_skip(&gbc, 10); bytestream2_skip(&gbc, 4);
h = bytestream2_get_be16(&gbc);
w = bytestream2_get_be16(&gbc);
bytestream2_skip(&gbc, 2);
ret = ff_set_dimensions(avctx, w, h);
if (ret < 0)
return ret;
pack_type = bytestream2_get_be16(&gbc); pack_type = bytestream2_get_be16(&gbc);
bytestream2_skip(&gbc, 16); bytestream2_skip(&gbc, 16);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment