[FFmpeg-devel] stlport with ffmpeg: Issue with time_is_forbidden_due_to_security_issues

Raymond Toy rtoy at google.com
Thu Sep 13 20:24:09 CEST 2012


On Wed, Sep 12, 2012 at 4:20 PM, Raymond Toy <rtoy at google.com> wrote:

>
> Thanks for the patch.  It seems the idea is right, but it's much more
> compilicated than that.  For example libavformat/mov.c includes avformat.h,
> which includes <time.h>.  And vp8.c includes vp8.h which includes
> <pthreads.h> and stlport pthreads.h includes <time.h>
>
> I can work on some patch to take care of this, but it seems that it would
> be pretty messy to have to move all of the various system includes buried
> in header files to the c files.  Some guidance on what would be acceptable
> would be helpful.
>
>
Ok, I've gone through and made a bare minimum patch that will allow ffmpeg
to work with stlport.  The patch basically removes <time.h> from a few
header files and adds #include <time.h> to the files that need it.  Also
pthreads.h is moved from a few places too.

It works for me with stlport (where I only build the codecs I need).  It
also works with ffmpeg itself and builds all of the programs.

If can provide the patch (about 500 lines) if anyone is interested.

Ray


More information about the ffmpeg-devel mailing list