[MPlayer-cvslog] CVS: main/libmpcodecs vd_libmpeg2.c,1.37,1.38

[Jindrich Makovicka]

Michael Niedermayer wrote:
> vd_ffmpeg certainly didnt need either width or height to be a multiple of 8, 
> if it does now someone broke it

i wanted to write 16, but it doesn't actually matter. the segfault 
doesn't occur in directly ffmpeg, but the image created with

  mpi= mpcodecs_get_image(sh,type, flags,
                         (width+align)&(~align), (height+align)&(~align));

can eventually propagate to a VO previously configured to width x 
height, which blows up then. to correct the things, mpi should be 
allocated with the physical dimensions according to the alignment, while 
logical dimensions mpi->w and mpi->h should be equal to width and 
height. however, this is impossible with current vf api.

-- 
Jindrich Makovicka