From 4a9f7d2bf9fb38c334386deef19ffef58a998cc6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Sun, 23 Dec 2012 21:34:06 +0200 Subject: [PATCH 1/4] hlsenc: Don't duplicate a string constant MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Martin Storsjö --- libavformat/hlsenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index c5157dc6c7..eade740f07 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -219,7 +219,7 @@ static int hls_write_header(AVFormatContext *s) if (p) *p = '\0'; - av_strlcat(hls->basename, "%d.ts", basename_size); + av_strlcat(hls->basename, pattern, basename_size); if ((ret = hls_mux_init(s)) < 0) goto fail; From 3eab60075240bd9855661a20667077d5435e065a Mon Sep 17 00:00:00 2001 From: Anton Khirnov Date: Thu, 15 Nov 2012 09:23:55 +0100 Subject: [PATCH 2/4] bmp: cosmetics, reformat --- libavcodec/bmp.c | 132 ++++++++++++++++++++++++----------------------- 1 file changed, 68 insertions(+), 64 deletions(-) diff --git a/libavcodec/bmp.c b/libavcodec/bmp.c index 648d0ac31b..9c8988c83a 100644 --- a/libavcodec/bmp.c +++ b/libavcodec/bmp.c @@ -25,7 +25,8 @@ #include "internal.h" #include "msrledec.h" -static av_cold int bmp_decode_init(AVCodecContext *avctx){ +static av_cold int bmp_decode_init(AVCodecContext *avctx) +{ BMPContext *s = avctx->priv_data; avcodec_get_frame_defaults(&s->picture); @@ -39,10 +40,10 @@ static int bmp_decode_frame(AVCodecContext *avctx, AVPacket *avpkt) { const uint8_t *buf = avpkt->data; - int buf_size = avpkt->size; - BMPContext *s = avctx->priv_data; - AVFrame *picture = data; - AVFrame *p = &s->picture; + int buf_size = avpkt->size; + BMPContext *s = avctx->priv_data; + AVFrame *picture = data; + AVFrame *p = &s->picture; unsigned int fsize, hsize; int width, height; unsigned int depth; @@ -55,19 +56,19 @@ static int bmp_decode_frame(AVCodecContext *avctx, const uint8_t *buf0 = buf; GetByteContext gb; - if(buf_size < 14){ + if (buf_size < 14) { av_log(avctx, AV_LOG_ERROR, "buf size too small (%d)\n", buf_size); return -1; } - if(bytestream_get_byte(&buf) != 'B' || - bytestream_get_byte(&buf) != 'M') { + if (bytestream_get_byte(&buf) != 'B' || + bytestream_get_byte(&buf) != 'M') { av_log(avctx, AV_LOG_ERROR, "bad magic number\n"); return -1; } fsize = bytestream_get_le32(&buf); - if(buf_size < fsize){ + if (buf_size < fsize) { av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d), trying to decode anyway\n", buf_size, fsize); fsize = buf_size; @@ -76,29 +77,29 @@ static int bmp_decode_frame(AVCodecContext *avctx, buf += 2; /* reserved1 */ buf += 2; /* reserved2 */ - hsize = bytestream_get_le32(&buf); /* header size */ - ihsize = bytestream_get_le32(&buf); /* more header size */ - if(ihsize + 14 > hsize){ + hsize = bytestream_get_le32(&buf); /* header size */ + ihsize = bytestream_get_le32(&buf); /* more header size */ + if (ihsize + 14 > hsize) { av_log(avctx, AV_LOG_ERROR, "invalid header size %d\n", hsize); return -1; } /* sometimes file size is set to some headers size, set a real size in that case */ - if(fsize == 14 || fsize == ihsize + 14) + if (fsize == 14 || fsize == ihsize + 14) fsize = buf_size - 2; - if(fsize <= hsize){ + if (fsize <= hsize) { av_log(avctx, AV_LOG_ERROR, "declared file size is less than header size (%d < %d)\n", fsize, hsize); return -1; } - switch(ihsize){ + switch (ihsize) { case 40: // windib v3 case 64: // OS/2 v2 case 108: // windib v4 case 124: // windib v5 - width = bytestream_get_le32(&buf); + width = bytestream_get_le32(&buf); height = bytestream_get_le32(&buf); break; case 12: // OS/2 v1 @@ -110,44 +111,46 @@ static int bmp_decode_frame(AVCodecContext *avctx, return -1; } - if(bytestream_get_le16(&buf) != 1){ /* planes */ + /* planes */ + if (bytestream_get_le16(&buf) != 1) { av_log(avctx, AV_LOG_ERROR, "invalid BMP header\n"); return -1; } depth = bytestream_get_le16(&buf); - if(ihsize == 40) + if (ihsize == 40) comp = bytestream_get_le32(&buf); else comp = BMP_RGB; - if(comp != BMP_RGB && comp != BMP_BITFIELDS && comp != BMP_RLE4 && comp != BMP_RLE8){ + if (comp != BMP_RGB && comp != BMP_BITFIELDS && comp != BMP_RLE4 && + comp != BMP_RLE8) { av_log(avctx, AV_LOG_ERROR, "BMP coding %d not supported\n", comp); return -1; } - if(comp == BMP_BITFIELDS){ + if (comp == BMP_BITFIELDS) { buf += 20; rgb[0] = bytestream_get_le32(&buf); rgb[1] = bytestream_get_le32(&buf); rgb[2] = bytestream_get_le32(&buf); } - avctx->width = width; - avctx->height = height > 0? height: -height; + avctx->width = width; + avctx->height = height > 0 ? height : -height; avctx->pix_fmt = AV_PIX_FMT_NONE; - switch(depth){ + switch (depth) { case 32: - if(comp == BMP_BITFIELDS){ + if (comp == BMP_BITFIELDS) { rgb[0] = (rgb[0] >> 15) & 3; rgb[1] = (rgb[1] >> 15) & 3; rgb[2] = (rgb[2] >> 15) & 3; - if(rgb[0] + rgb[1] + rgb[2] != 3 || - rgb[0] == rgb[1] || rgb[0] == rgb[2] || rgb[1] == rgb[2]){ + if (rgb[0] + rgb[1] + rgb[2] != 3 || + rgb[0] == rgb[1] || rgb[0] == rgb[2] || rgb[1] == rgb[2]) { break; } } else { @@ -162,7 +165,7 @@ static int bmp_decode_frame(AVCodecContext *avctx, avctx->pix_fmt = AV_PIX_FMT_BGR24; break; case 16: - if(comp == BMP_RGB) + if (comp == BMP_RGB) avctx->pix_fmt = AV_PIX_FMT_RGB555; else if (comp == BMP_BITFIELDS) { if (rgb[0] == 0xF800 && rgb[1] == 0x07E0 && rgb[2] == 0x001F) @@ -178,16 +181,16 @@ static int bmp_decode_frame(AVCodecContext *avctx, } break; case 8: - if(hsize - ihsize - 14 > 0) + if (hsize - ihsize - 14 > 0) avctx->pix_fmt = AV_PIX_FMT_PAL8; else avctx->pix_fmt = AV_PIX_FMT_GRAY8; break; case 1: case 4: - if(hsize - ihsize - 14 > 0){ + if (hsize - ihsize - 14 > 0) { avctx->pix_fmt = AV_PIX_FMT_PAL8; - }else{ + } else { av_log(avctx, AV_LOG_ERROR, "Unknown palette for %d-colour BMP\n", 1<pix_fmt == AV_PIX_FMT_NONE){ + if (avctx->pix_fmt == AV_PIX_FMT_NONE) { av_log(avctx, AV_LOG_ERROR, "unsupported pixel format\n"); return -1; } - if(p->data[0]) + if (p->data[0]) avctx->release_buffer(avctx, p); p->reference = 0; - if(ff_get_buffer(avctx, p) < 0){ + if (ff_get_buffer(avctx, p) < 0) { av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n"); return -1; } p->pict_type = AV_PICTURE_TYPE_I; p->key_frame = 1; - buf = buf0 + hsize; + buf = buf0 + hsize; dsize = buf_size - hsize; /* Line size in file multiple of 4 */ n = ((avctx->width * depth) / 8 + 3) & ~3; - if(n * avctx->height > dsize && comp != BMP_RLE4 && comp != BMP_RLE8){ + if (n * avctx->height > dsize && comp != BMP_RLE4 && comp != BMP_RLE8) { av_log(avctx, AV_LOG_ERROR, "not enough data (%d < %d)\n", dsize, n * avctx->height); return -1; } // RLE may skip decoding some picture areas, so blank picture before decoding - if(comp == BMP_RLE4 || comp == BMP_RLE8) + if (comp == BMP_RLE4 || comp == BMP_RLE8) memset(p->data[0], 0, avctx->height * p->linesize[0]); - if(height > 0){ - ptr = p->data[0] + (avctx->height - 1) * p->linesize[0]; + if (height > 0) { + ptr = p->data[0] + (avctx->height - 1) * p->linesize[0]; linesize = -p->linesize[0]; } else { - ptr = p->data[0]; + ptr = p->data[0]; linesize = p->linesize[0]; } - if(avctx->pix_fmt == AV_PIX_FMT_PAL8){ + if (avctx->pix_fmt == AV_PIX_FMT_PAL8) { int colors = 1 << depth; memset(p->data[1], 0, 1024); - if(ihsize >= 36){ + if (ihsize >= 36) { int t; buf = buf0 + 46; - t = bytestream_get_le32(&buf); - if(t < 0 || t > (1 << depth)){ + t = bytestream_get_le32(&buf); + if (t < 0 || t > (1 << depth)) { av_log(avctx, AV_LOG_ERROR, "Incorrect number of colors - %X for bitdepth %d\n", t, depth); - }else if(t){ + } else if (t) { colors = t; } } buf = buf0 + 14 + ihsize; //palette location - if((hsize-ihsize-14) < (colors << 2)){ // OS/2 bitmap, 3 bytes per palette entry - for(i = 0; i < colors; i++) + // OS/2 bitmap, 3 bytes per palette entry + if ((hsize-ihsize-14) < (colors << 2)) { + for (i = 0; i < colors; i++) ((uint32_t*)p->data[1])[i] = bytestream_get_le24(&buf); - }else{ - for(i = 0; i < colors; i++) + } else { + for (i = 0; i < colors; i++) ((uint32_t*)p->data[1])[i] = bytestream_get_le32(&buf); } buf = buf0 + hsize; } - if(comp == BMP_RLE4 || comp == BMP_RLE8){ - if(height < 0){ - p->data[0] += p->linesize[0] * (avctx->height - 1); + if (comp == BMP_RLE4 || comp == BMP_RLE8) { + if (height < 0) { + p->data[0] += p->linesize[0] * (avctx->height - 1); p->linesize[0] = -p->linesize[0]; } bytestream2_init(&gb, buf, dsize); ff_msrle_decode(avctx, (AVPicture*)p, depth, &gb); - if(height < 0){ - p->data[0] += p->linesize[0] * (avctx->height - 1); + if (height < 0) { + p->data[0] += p->linesize[0] * (avctx->height - 1); p->linesize[0] = -p->linesize[0]; } - }else{ - switch(depth){ + } else { + switch (depth) { case 1: for (i = 0; i < avctx->height; i++) { int j; @@ -294,16 +298,16 @@ static int bmp_decode_frame(AVCodecContext *avctx, break; case 8: case 24: - for(i = 0; i < avctx->height; i++){ + for (i = 0; i < avctx->height; i++) { memcpy(ptr, buf, n); buf += n; ptr += linesize; } break; case 4: - for(i = 0; i < avctx->height; i++){ + for (i = 0; i < avctx->height; i++) { int j; - for(j = 0; j < n; j++){ + for (j = 0; j < n; j++) { ptr[j*2+0] = (buf[j] >> 4) & 0xF; ptr[j*2+1] = buf[j] & 0xF; } @@ -312,11 +316,11 @@ static int bmp_decode_frame(AVCodecContext *avctx, } break; case 16: - for(i = 0; i < avctx->height; i++){ + for (i = 0; i < avctx->height; i++) { const uint16_t *src = (const uint16_t *) buf; - uint16_t *dst = (uint16_t *) ptr; + uint16_t *dst = (uint16_t *) ptr; - for(j = 0; j < avctx->width; j++) + for (j = 0; j < avctx->width; j++) *dst++ = av_le2ne16(*src++); buf += n; @@ -324,11 +328,11 @@ static int bmp_decode_frame(AVCodecContext *avctx, } break; case 32: - for(i = 0; i < avctx->height; i++){ + for (i = 0; i < avctx->height; i++) { const uint8_t *src = buf; - uint8_t *dst = ptr; + uint8_t *dst = ptr; - for(j = 0; j < avctx->width; j++){ + for (j = 0; j < avctx->width; j++) { dst[0] = src[rgb[2]]; dst[1] = src[rgb[1]]; dst[2] = src[rgb[0]]; From f6e395e1320c52cc0720a83787622b15f6762d4f Mon Sep 17 00:00:00 2001 From: Anton Khirnov Date: Thu, 15 Nov 2012 10:38:38 +0100 Subject: [PATCH 3/4] c93: set palette_has_changed. --- libavcodec/c93.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/c93.c b/libavcodec/c93.c index e8071ac69b..b820f5a9da 100644 --- a/libavcodec/c93.c +++ b/libavcodec/c93.c @@ -233,6 +233,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, for (i = 0; i < 256; i++) { palette[i] = bytestream2_get_be24(&gb); } + newpic->palette_has_changed = 1; } else { if (oldpic->data[1]) memcpy(newpic->data[1], oldpic->data[1], 256 * 4); From c6303f8d70c25dd6c6e6486c78bf99c9924e2b6b Mon Sep 17 00:00:00 2001 From: Anton Khirnov Date: Thu, 15 Nov 2012 22:03:00 +0100 Subject: [PATCH 4/4] yop: simplify/sanitize the decoding loop Do not overwrite linesize set by get_buffer(). The last frame in the FATE test is not decoded anymore, since the file is cut and a part of it is missing. --- libavcodec/yop.c | 56 +++++++++++++++++----------------------------- tests/ref/fate/yop | 1 - 2 files changed, 21 insertions(+), 36 deletions(-) diff --git a/libavcodec/yop.c b/libavcodec/yop.c index c3bb577119..0b9cdfebe2 100644 --- a/libavcodec/yop.c +++ b/libavcodec/yop.c @@ -36,7 +36,6 @@ typedef struct YopDecContext { int num_pal_colors; int first_color[2]; int frame_data_length; - int row_pos; uint8_t *low_nibble; uint8_t *srcptr; @@ -173,27 +172,12 @@ static uint8_t yop_get_next_nibble(YopDecContext *s) return ret; } -/** - * Take s->dstptr to the next macroblock in sequence. - */ -static void yop_next_macroblock(YopDecContext *s) -{ - // If we are advancing to the next row of macroblocks - if (s->row_pos == s->frame.linesize[0] - 2) { - s->dstptr += s->frame.linesize[0]; - s->row_pos = 0; - }else { - s->row_pos += 2; - } - s->dstptr += 2; -} - static int yop_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPacket *avpkt) { YopDecContext *s = avctx->priv_data; int tag, firstcolor, is_odd_frame; - int ret, i; + int ret, i, x, y; uint32_t *palette; if (s->frame.data[0]) @@ -205,12 +189,9 @@ static int yop_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, return ret; } - s->frame.linesize[0] = avctx->width; - s->dstbuf = s->frame.data[0]; s->dstptr = s->frame.data[0]; s->srcptr = avpkt->data + 4; - s->row_pos = 0; s->low_nibble = NULL; is_odd_frame = avpkt->data[0]; @@ -224,23 +205,28 @@ static int yop_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, s->frame.palette_has_changed = 1; - while (s->dstptr - s->dstbuf < - avctx->width * avctx->height && - s->srcptr - avpkt->data < avpkt->size) { - - tag = yop_get_next_nibble(s); - - if (tag != 0xf) { - yop_paint_block(s, tag); - }else { - tag = yop_get_next_nibble(s); - ret = yop_copy_previous_block(s, tag); - if (ret < 0) { - avctx->release_buffer(avctx, &s->frame); - return ret; + for (y = 0; y < avctx->height; y += 2) { + for (x = 0; x < avctx->width; x += 2) { + if (s->srcptr - avpkt->data >= avpkt->size) { + av_log(avctx, AV_LOG_ERROR, "Packet too small.\n"); + return AVERROR_INVALIDDATA; } + + tag = yop_get_next_nibble(s); + + if (tag != 0xf) { + yop_paint_block(s, tag); + } else { + tag = yop_get_next_nibble(s); + ret = yop_copy_previous_block(s, tag); + if (ret < 0) { + avctx->release_buffer(avctx, &s->frame); + return ret; + } + } + s->dstptr += 2; } - yop_next_macroblock(s); + s->dstptr += 2*s->frame.linesize[0] - x; } *got_frame = 1; diff --git a/tests/ref/fate/yop b/tests/ref/fate/yop index 5629825688..57f565d5f8 100644 --- a/tests/ref/fate/yop +++ b/tests/ref/fate/yop @@ -5,4 +5,3 @@ 0, 3, 3, 1, 302760, 0xc34b20bd 0, 4, 4, 1, 302760, 0x461d29a1 0, 5, 5, 1, 302760, 0x45abca02 -0, 6, 6, 1, 302760, 0xb05448b9