[Ffmpeg-devel-irc] ffmpeg-devel.log.20120415

[burek]

[01:53] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rc354576457 10ffmpeg/libavcodec/mpegvideo_enc.c: 
[01:54] <CIA-17> ffmpeg: mpegvideo_enc: fix dts
[01:54] <CIA-17> ffmpeg: Fixes Ticket1200
[01:54] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[04:57] <Compn> j-b : did you know that veetle.com player is using vlc as a backend ?
[04:57] <Compn> at least they include a copy of the gpl i guess
[04:57] <Compn> but didnt see any source code
[04:58] <Compn> btw, a pox on anyone who embeds vlc underneath flash player. you're just asking for mem leaks and trouble doing that
[10:21] <CIA-17> ffmpeg: 03Derek Buitenhuis 07master * r5a59d2c40b 10ffmpeg/libavcodec/utvideo.c: 
[10:21] <CIA-17> ffmpeg: utvideo: Remove unused variable 'src_size'
[10:21] <CIA-17> ffmpeg: Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
[11:44] <j-b> Compn: yes, I do know. They offer the full source and patches.
[13:55] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r0846719dd1 10ffmpeg/libavcodec/ (indeo4.c ivi_common.c ivi_common.h): 
[13:55] <CIA-17> ffmpeg: indeo4: check transform size.
[13:55] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[13:55] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[13:56] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * raf38823709 10ffmpeg/libavcodec/indeo5.c: 
[13:56] <CIA-17> ffmpeg: indeo5: set transform_size like indeo4 does to prevent useage of too large transforms.
[13:56] <CIA-17> ffmpeg: I dont know if this is needed, its a precautionary change.
[13:56] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[14:22] <CIA-17> ffmpeg: 03Nicolas George 07master * rb7027685b8 10ffmpeg/configure: configure: add dependencies for colormatrix test.
[15:15] <ubitux> we have no helper whatsoever for linked list, right?
[15:16] <ubitux> it seems that we use av_fast_realloc() or similar most of the time
[15:17] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r5ad7335eba 10ffmpeg/libavcodec/indeo5.c: 
[15:17] <CIA-17> ffmpeg: indeo5: check tile size in decode_mb_info().
[15:17] <CIA-17> ffmpeg: This prevents writing into a too small array if some parameters changed
[15:17] <CIA-17> ffmpeg: without the tile being reallocated.
[15:17] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[15:17] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[15:17] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rf0bf9e9c2a 10ffmpeg/libavcodec/ivi_common.c: 
[15:17] <CIA-17> ffmpeg: indeo: Check allocated tile size in ff_ivi_process_empty_tile()
[15:17] <CIA-17> ffmpeg: This prevents writing into a too small array if some parameters changed
[15:17] <CIA-17> ffmpeg: without the tile being reallocated.
[15:17] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[15:17] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[15:17] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r93927eb334 10ffmpeg/libavcodec/ivi_common.c: 
[15:17] <CIA-17> ffmpeg: ff_ivi_decode_blocks: fix negative scan_pos case.
[15:17] <CIA-17> ffmpeg: Fixes out of global array read.
[15:17] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[15:17] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[15:23] <michaelni> ubitux, what kind of helper would you want for a linked list ?
[15:26] <ubitux> well in my case i just need to make a list of text entries and sort them (at insert time, or afterward)
[15:26] <ubitux> the same way lavf/assdec.c does
[15:27] <ubitux> (this one seems overcomplicated though)
[15:58] <CIA-17> ffmpeg: 03Nicolas George 07master * re6f7e22d18 10ffmpeg/tests/tiny_psnr.c: 
[15:58] <CIA-17> ffmpeg: tiny_psnr: support unseekable raw files.
[15:58] <CIA-17> ffmpeg: If either one of the files if not seekable,
[15:58] <CIA-17> ffmpeg: do not try to read headers.
[15:58] <CIA-17> ffmpeg: It allows to use it with pipes on whole rawvideo data.
[15:59] <CIA-17> ffmpeg: 03Nicolas George 07master * r34123ab979 10ffmpeg/tests/tiny_psnr.c: tiny_psnr: reindent after last commit.
[16:09] <michaelni> ubitux, an array and a qsort() seem easiest but there are million ways to do it
[16:09] <michaelni> for sort during insert theres our tree.c/h or one could use a heap but these will be slower than a qsort at the end
[16:11] <ubitux> ok :)
[16:11] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rb962932cba 10ffmpeg/libavcodec/eatgv.c: 
[16:11] <CIA-17> ffmpeg: eatgv: check vector_bits
[16:11] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[16:11] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[16:11] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rb1096b6ee7 10ffmpeg/libavcodec/xan.c: 
[16:11] <CIA-17> ffmpeg: xan: check for vector_segment overread
[16:11] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[16:11] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[16:11] <ubitux> i don't think performance matters much in this case anyway
[16:12] <ubitux> i'll go for something simpler
[16:12] <ubitux> no one care about jacosub anyway
[16:12] <ubitux> it's more like an subtitle decoder/demuxer example 
[16:36] <CIA-17> ffmpeg: 03Carl Eugen Hoyos 07master * r8cb9f99e9f 10ffmpeg/libavcodec/lagarith.c: 
[16:36] <CIA-17> ffmpeg: Support decoding unaligned rgb24 lagarith.
[16:36] <CIA-17> ffmpeg: Fixes ticket #1214.
[16:37] <CIA-17> ffmpeg: 03Carl Eugen Hoyos 07master * rcb8f5194b3 10ffmpeg/ (3 files in 3 dirs): 
[16:37] <CIA-17> ffmpeg: Set bits_per_coded_sample when encoding Microsoft Video-1.
[16:37] <CIA-17> ffmpeg: Fixes ticket #1193.
[17:15] <ubitux> http://mailman.videolan.org/pipermail/vlc-devel/2012-April/088379.html heh. :)
[17:15] <ubitux> michaelni ^
[17:16] <j-b> :)
[17:16] <Compn> is that like af format ?
[17:20] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r1a974679d0 10ffmpeg/libavcodec/qdm2.c: 
[17:20] <CIA-17> ffmpeg: qdm2: Check vlc_stage3_values index.
[17:20] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[17:20] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[17:20] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r20335598f2 10ffmpeg/libavcodec/qdm2.c: 
[17:20] <CIA-17> ffmpeg: qdm2_fft_decode_tones: fix infinite loop
[17:20] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[17:20] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[17:21] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r3118e3b137 10ffmpeg/libavcodec/lagarithrac.c: 
[17:21] <CIA-17> ffmpeg: ff_lag_rac_init: fix signedness error leading to out of array read.
[17:21] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[17:21] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[17:24] <michaelni> :)
[18:14] <CIA-17> ffmpeg: 03Nicolas George 07master * ra9e7e98021 10ffmpeg/libavutil/error.h: 
[18:14] <CIA-17> ffmpeg: error: ensure error codes are signed negative ints.
[18:14] <CIA-17> ffmpeg: Based on a patch by Robert Nagy <ronag89 at gmail.com>.
[18:14] <CIA-17> ffmpeg: It makes a difference when the error code is immediately cast
[18:14] <CIA-17> ffmpeg: into a larger integer, such as an int64_t.
[18:26] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r44c4170c52 10ffmpeg/libavcodec/lzw.c: 
[18:26] <CIA-17> ffmpeg: lzw: check for overread
[18:26] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[18:26] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[18:26] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rc7dc19d68f 10ffmpeg/libavcodec/pngdec.c: 
[18:26] <CIA-17> ffmpeg: png: make sure the previous frames dimensions match before using it as reference
[18:26] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[18:26] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[18:26] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * ra63c813797 10ffmpeg/libavcodec/pngdec.c: 
[18:26] <CIA-17> ffmpeg: pngdec: check bits_per_pixel for palette mode.
[18:26] <CIA-17> ffmpeg: This fixes a out of array read
[18:26] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[18:26] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[19:07] <ubitux> mmh, can't we use EAGAIN in probing?
[19:09] <ubitux> well, whatever.
[19:17] <j-b> ubitux: michaelni: resampling speed is an important issue in Vlc 2.0
[19:18] <Daemon404> i'd assume it needs to be on-the-fly, even on crappy cpus, no?
[19:19] <ubitux> i'd be interested to try to write some asm in it when it will already have some
[19:19] <ubitux> but i guess i'll have to purge my todo list a bit first too :)
[19:20] <Daemon404> rm -f TODO
[19:20] <Daemon404> easy :)
[19:21] <ubitux> hehe :)
[19:21] <kierank> there's tons of AVX asm to write
[21:32] <CIA-17> ffmpeg: 03Thilo Borgmann 07master * r2837d8dc27 10ffmpeg/libavcodec/alsdec.c: 
[21:32] <CIA-17> ffmpeg: alsdec: fix number of decoded samples in first sub-block in BGMC mode.
[21:32] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[21:32] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[21:32] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * rfefc65675e 10ffmpeg/libavcodec/tiff.c: 
[21:32] <CIA-17> ffmpeg: tiffdec: check overread for packbits
[21:32] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[21:32] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[21:32] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r88a97d660d 10ffmpeg/libavcodec/indeo5.c: 
[21:32] <CIA-17> ffmpeg: indeo5: check for unsupported luma block type
[21:32] <CIA-17> ffmpeg: Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
[21:32] <CIA-17> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[23:36] <CIA-17> ffmpeg: 03Ronald S. Bultje 07master * r4860625236 10ffmpeg/libswscale/output.c: 
[23:36] <CIA-17> ffmpeg: swscale: fix off-by-one in second coefficient in bilinear filters.
[23:36] <CIA-17> ffmpeg: If coefficient A is 12-bits xixed-point number "X", then the other
[23:36] <CIA-17> ffmpeg: coefficient is (1 << 12) - X, not (1 << 12) - X - 1.
[23:36] <CIA-17> ffmpeg: 03Ronald S. Bultje 07master * r7f77e9041a 10ffmpeg/libswscale/output.c: swscale: clip before assigning tables in RGB output functions.
[23:36] <CIA-17> ffmpeg: 03Dale Curtis 07master * r4ebd422c04 10ffmpeg/libavformat/mov.c: 
[23:36] <CIA-17> ffmpeg: mov: fix leaking memory with multiple drefs.
[23:36] <CIA-17> ffmpeg: Instead of allocating over the original, free first. MOVStreamContext
[23:36] <CIA-17> ffmpeg: is zero initialized so no double free will occur. Same style as other
[23:36] <CIA-17> ffmpeg: fixes for the same problem in this file.
[23:36] <CIA-17> ffmpeg: Signed-off-by: Dale Curtis <dalecurtis at chromium.org>
[23:36] <CIA-17> ffmpeg: Signed-off-by: Luca Barbato <lu_zero at gentoo.org>
[23:36] <CIA-17> ffmpeg: 03Derek Buitenhuis 07master * r5096399df2 10ffmpeg/libavcodec/utvideo.c: 
[23:36] <CIA-17> ffmpeg: utvideo: Remove unused variable 'src_size'
[23:36] <CIA-17> ffmpeg: Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
[23:36] <CIA-17> ffmpeg: 03Diego Biurrun 07master * r3892e784f2 10ffmpeg/doc/developer.texi: 
[23:37] <CIA-17> ffmpeg: 03Michael Niedermayer 07master * r35e0496a7b 10ffmpeg/: (log message trimmed)
[23:37] <CIA-17> ffmpeg: Merge remote-tracking branch 'qatar/master'
[23:37] <CIA-17> ffmpeg: * qatar/master:
[23:37] <CIA-17> ffmpeg:  doc: Improve suggested Emacs settings for our coding style.
[23:37] <CIA-17> ffmpeg:  utvideo: Remove unused variable 'src_size'
[23:37] <CIA-17> ffmpeg:  mov: free memory on header parsing failure
[23:37] <CIA-17> ffmpeg:  mov: fix leaking memory with multiple drefs.
[23:37] <CIA-17> ffmpeg: 03Dale Curtis 07master * rc788782c7d 10ffmpeg/libavformat/mov.c: 
[23:37] <CIA-17> ffmpeg: mov: free memory on header parsing failure
[23:37] <CIA-17> ffmpeg: Call mov_read_close when mov_read_header fails.
[23:37] <CIA-17> ffmpeg: Signed-off-by: Luca Barbato <lu_zero at gentoo.org>
[00:00] --- Mon Apr 16 2012