[FFmpeg-devel] half vertical chroma resolution from JPEGs..

[Michael Niedermayer]

Hi

On Sat, Jul 14, 2007 at 11:36:38AM +0200, Andreas ?man wrote:
> Hi
> 
> Michael Niedermayer wrote:
> >Hi
> >
> >see the thread about " [PATCH] MJPEG wrongly displayed with vertical 
> >YUV422"
> 
> Ah, thanks
> 
> >also the patch is ok only if it works with swscaler too
> >
> 
> It works, but when dealing with large images, swscale seem
> to crash (i doubt this is related to yuv440 images only)
> If I compile without libswscale, it works fine.
> 
> After some valgrinding (see below) i suspected the
> 
> c->lumPixBuf[i]= c->lumPixBuf[i+c->vLumBufSize]= av_mallocz(4000);
> 
> -line. Just setting it to 8000 makes it work all fine, but i
> believe the real solution is to compute this value, but how?
> I dont know the swscale internals enough to figure that out.

it should be (dstW+1)*sizeof(int16_t) _IIRC_ (totally untested!)

and patch looks ok

also maybe you want to add the new format to tests/regression.sh ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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/20070717/5e721a26/attachment.pgp>