[FFmpeg-devel] [PATCH]Fix 2bpp and 4bpp rawvideo in mov

Michael Niedermayer michaelni at gmx.at
Wed Jun 12 17:19:44 CEST 2013


On Mon, May 06, 2013 at 01:51:58AM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes ticket #2536 for me.
> 
> Please comment, Carl Eugen

>  rawdec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 50ee1f5ea2e8873e0e767a63d5083e1b9fd09e46  patch2_4bppmov.diff
> diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c
> index f45ff4c..ae920c5 100644
> --- a/libavcodec/rawdec.c
> +++ b/libavcodec/rawdec.c
> @@ -133,7 +133,7 @@ static av_cold int raw_init_decoder(AVCodecContext *avctx)
>              memset(context->palette->data, 0, AVPALETTE_SIZE);
>      }
>  
> -    context->frame_size = avpicture_get_size(avctx->pix_fmt, avctx->width,
> +    context->frame_size = avpicture_get_size(avctx->pix_fmt, FFALIGN(avctx->width, 16),
>                                               avctx->height);

if 16 is correct for all pixel formats then ok with a commit message
mentioning that frame_size is fixed

Also please send patches with git send-mail or attach git format-patch
patches. That makes applying them for testing easier

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130612/cbbe1d7a/attachment.asc>


More information about the ffmpeg-devel mailing list