[FFmpeg-devel] [PATCH 1/2] ffplay: avoid window resize crash on osx with libsdl 1.2.14

Marton Balint cus at passwd.hu
Sun Oct 16 20:26:06 CEST 2011


On Wed, 12 Oct 2011, Carl Eugen Hoyos wrote:

> Jean First <jeanfirst <at> gmail.com> writes:
>
>> +++ b/ffplay.c
>> @@ -947,13 +947,7 @@ static int video_open(VideoState *is){
>>      if(screen && is->width == screen->w && screen->w == w
>>         && is->height== screen->h && screen->h == h)
>>          return 0;
>> -
>> -#ifndef __APPLE__
>>      screen = SDL_SetVideoMode(w, h, 0, flags);
>> -#else
>> -    /* setting bits_per_pixel = 0 or 32 causes blank video on OS X */
>> -    screen = SDL_SetVideoMode(w, h, 24, flags);
>> -#endif
>
> This patch causes a double-free on exit in SDL_FreeYUVOverlay() for me on PPC
> oSX 10.5, but since it fixes wrong colours in many samples, I am all for it.

Applied to my stable branch, thanks.

Michael, please merge from ffplay stable for the following patches:

72776ad ffplay: avoid window resize crash on osx with libsdl 1.2.14
abb0e4f ffplay: add delay multiple times to frame_timer if it is less than current time
d2d8e1e ffplay: remove early frame drop functionality
4e268aa ffplay: calculate target clock dynamically, make code more readable

Regards,
Marton


More information about the ffmpeg-devel mailing list