2017-08-30 17:43 GMT+08:00 Timo Rothenpieler <[email protected]>: >> 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 <[email protected]> >> >> v17 fixed: >> 1. add memory alloc faild check >> 2. fix resource space error at free_representation >> >> Signed-off-by: Steven Liu <[email protected]> >> Signed-off-by: samsamsam <[email protected]> > > > 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 > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
