[Ffmpeg-devel] Three ffserver patches for review
Thu Apr 19 09:47:54 CEST 2007
Alex Beregszaszi wrote:
> Better if we don't have that much unused code. Remove it.
> I guess ok.
Good. I'll commit these two in the afternoon
>> when ffserver is initialized, it allocates the AVStream->priv_data field
>> for containing "feed information" in every AVStream, even if they do not
>> use any feed. This is ok if the input stream is a "live feed" coming
>> from ffmpeg, but is not ok for regular files.
> Rejected, it must be a bug somewhere else.
Ok; I'll try to debug the problem some more...
> add_av_stream1 is basically equivalent to av_new_stream in utils.c.
The key difference is the thin which is creating the problem:
fst->priv_data = av_mallocz(sizeof(FeedData));
And this field is copied into the stream which is dynamically created
when an RTP connection is created. When the RTP connection terminates,
av_write_trailer() is called, and it does
So, it's freeing fst->priv_data every time an RTP connection terminates,
but such field is only allocated at the program startup...
Anyway, this thing is confusing; maybe I got it wrong. I'll try have a
better look today.
> My local tree uses av_new_stream and removes duplicates.
Good. I gues this will automatically solve the problem. I'll try your
patch today. BTW, how do you allocate the priv_data field in AVStream?
More information about the ffmpeg-devel