[FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

[Steven Liu]

2017-08-30 17:43 GMT+08:00 Timo Rothenpieler <timo at rothenpieler.org>:
>> v16 fixed:
>> 1. remove the snprintf and instead by get_segment_filename make safety
>> 2. remove unnecessary loops
>> 3. updated xmlStrcmp and xmlFree to av_* functions
>> 4. merge code repeat into one function
>> 5. add memory alloc faild check
>> 6. update update_init_section and open_url
>> 7. output safety error message when filename template not safe
>> Suggested-by : wm4 <nfxjfg at googlemail.com>
>>
>> v17 fixed:
>> 1. add memory alloc faild check
>> 2. fix resource space error at free_representation
>>
>> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
>> Signed-off-by: samsamsam <samsamsam at o2.pl>
>
>
> Just wanted to point out, as stated on IRC, that dashenc already has a
> function that seems to safely solve the filename template issue, and looks
> like it can be moved into a common file + header, and reused as is:
>
> http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/dashenc.c;h=bdb332c71ab4aae87201c87c3d43938a07a9ed5c;hb=HEAD#l314
>
I have try to used that APIs, but it need some modify :
template $Number-%d$ will ok, but $Number$ will wrong, maybe mux
operation different to demux, or i misunderstand some thing?

>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>