[FFmpeg-devel] [PATCH] Use correct colorspace in Cinepak decoder.

Michael Niedermayer michaelni at gmx.at
Tue Feb 19 13:55:55 CET 2013


On Tue, Feb 19, 2013 at 12:53:10PM +0100, u-bo1b at 0w.se wrote:
> On Tue, Feb 19, 2013 at 06:24:25AM -0500, compn wrote:
> > >eliminating it. The second patch fixes this by rejecting unsafe strips.
> > 
> > in ffmpeg usually we print a message "invalid amount of strips" and
> > then continue decoding of the file.
> > 
> > so the user is warned, but at least can still play his/her file.
> 
> Given that strips are frame-wide for all samples and encoders known to me,
> rejecting per strip implies rejecting the whole file.
> 
> A safe and efficient approach would be to be able to adjust the frame
> size up to a multiple of 4 (mplayer actually seems to do that?).
> Is there any way to properly set this from inside the decoder?

Please see avcodec_set_dimensions()
and
avcodec_align_dimensions2()

Note, also later provides a large enough buffer for yuv420 but not for
rgb, as cinepack isnt listed in it as requireing one


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- 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/20130219/dfca72b3/attachment.asc>


More information about the ffmpeg-devel mailing list