[Ffmpeg-devel] [PATCH] Fix for dvdsubdec.c

[Michael Niedermayer]

Hi

On Fri, Jan 19, 2007 at 09:00:40AM +0000, Ian Caulfield wrote:
> Hi,
> 
> This quick patch fixes a bug in the DVD subtitle decoder where
> subtitles with odd heights would not have the last line decoded,
> leaving the bottom line of the bitmap array uninitialised.
> 
> Ian
> 
> 
> Index: libavcodec/dvdsubdec.c
> ===================================================================
> --- libavcodec/dvdsubdec.c      (revision 7580)
> +++ libavcodec/dvdsubdec.c      (working copy)
> @@ -249,7 +249,7 @@
>                 sub_header->rects = av_mallocz(sizeof(AVSubtitleRect));
>                 sub_header->num_rects = 1;
>                 sub_header->rects[0].rgba_palette = av_malloc(4 * 4);
> -                decode_rle(bitmap, w * 2, w, h / 2,
> +                decode_rle(bitmap, w * 2, w, (h + 1) / 2,
>                            buf, offset1 * 2, buf_size);
>                 decode_rle(bitmap + w, w * 2, w, h / 2,

something is wrong with the indention here, besides that the patch looks
ok

[...]

-- 
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: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070119/b43010a6/attachment.pgp>