New ffmpeg CVS versions

Vladimir Kushnir vkushnir at i.kiev.ua
Tue May 3 14:48:59 PDT 2005



On Tue, 3 May 2005, Jacob Meuser wrote:

<snip>
>
> notes on grab_bsdbktr.c
>
<snip>
>  i think av_set_pts_info() needs to be moved up to before other
>  parameters in 's' are set in grab_read_header().

Actually, with av_set_pts_info() as it is grabbing doesn't work even in 
the present ffmpeg port. It should be
av_set_pts_info(st, 48, 1, 1000000);
not
av_set_pts_info(s1, 48, 1, 1000000);
(as av_set_pts_info() takes AVStream and not AVFormatContext). With this 
change ffmpeg grabs even without any move.
>
>  i'm pretty sure the av_free() in grab_read_close() can result in
>  double free errors, i.e. ffmpeg free()s the memory automagically.

Yep, it does.

Regards,
Vladimir


More information about the freebsd-multimedia mailing list