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