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

[burek]

[00:10:40 CEST] <durandal_1707> 767. rule of security: do not put many things on same server
[00:11:37 CEST] <kierank> durandal_1707: did they teach you that at cia?
[00:12:44 CEST] <durandal_1707> no, at mossad
[00:24:19 CEST] <durandal_1707> michaelni: you havent removed me from fate rsync samples upload access?
[00:28:50 CEST] <durandal_1707> im working on msrle encoder and i cant figure out how to simply skip run but without overflowing bounds
[00:29:33 CEST] <durandal_1707> i get bit exact output but with benign errors
[00:30:09 CEST] <kierank> durandal_1707: why you work on encoder for windows 3.0
[00:31:05 CEST] <durandal_1707>  because windows 3.0 cant use dav1d
[02:33:10 CEST] <Compnn> uploads is on vlc server iirc
[02:34:30 CEST] <Compnn> samples repo is on ffmpeg but also vlc has their own
[02:34:37 CEST] <Compnn> vlc has not given us access to that, at least not me
[02:34:55 CEST] <Compnn> kierank,  durandal_1707 why not get access to vlc sample repo? http://streams.videolan.org j-b hmmmm ?
[02:35:13 CEST] <Compnn> and you'll find same reason that ffmpeg doesnt hand it over
[02:36:08 CEST] <Compnn> used to have multimedia_mike running our samples repo
[02:36:30 CEST] <Compnn> kierank,  you run a lot of servers, why not host a samples repo too ?
[02:36:43 CEST] <Compnn> and give everyone access , upload incoming as well
[02:37:25 CEST] <Compnn> kierank, please let me know my login when you get it up and running.
[02:40:34 CEST] <Compnn> last i looked ffbox also was low on space
[10:50:30 CEST] <Heisenberg> Where should I modify h264.c to add new side data to frames ?
[10:54:53 CEST] <kierank> Heisenberg: what are you adding
[11:02:05 CEST] <Heisenberg> It is basically data describing "how much motion there is in a frame" so it can be represented by uint8_t* and I would create a new  AVFrameSideDataType to identify it. In comparison, hevcdec.c has a different control flow which integrates a "set side data step". But I don't see where I should set this side data in h264dec.c. 
[12:11:19 CEST] <cone-962> ffmpeg 03Andreas Rheinhardt 07master:b307d74fe6ae: compat/cuda: Change inclusion guards
[12:12:10 CEST] <cone-962> ffmpeg 03Andreas Rheinhardt 07release/4.2:fc6f02b29772: compat/cuda: Change inclusion guards
[16:28:25 CEST] <durandal_1707> who controls FFmpeg leader?
[16:34:37 CEST] <thilo> we all do
[16:35:38 CEST] <thardin> master blaster controls ffmpeg
[16:35:55 CEST] <rcombs> like, the cleaning product?
[16:42:50 CEST] <relaxed> rcombs: from Mad Max
[16:55:01 CEST] <durandal_1707> the more fuzzing is made faster, the more new fuzzing issues will krept out
[17:26:54 CEST] <thardin> the amount of fuzzing fixes is both a good and bad thing
[17:27:05 CEST] <thardin> bad in that the code is able to have such issues in the first place
[17:27:21 CEST] <thardin> somethingsomething formal verification
[17:27:40 CEST] <thardin> or langsec in general
[17:27:45 CEST] <jdarnley> something somehing twos complement
[17:35:51 CEST] <durandal_1707> rust
[18:03:13 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:005de57d843d: avcodec/bink: Add many end of input checks
[18:03:14 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:d555a220b461: avcodec/atrac9dec: Remove impossible condition
[18:03:15 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:208225bd7822: avcodec/atrac9dec: Check grad_range[1] more tightly
[18:03:16 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:8e4b522c9146: avcodec/apedec: Do not partially clear data array
[18:03:17 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:2cb1f7973508: avcodec/vc1_block: Check get_vlc2() return before use
[18:03:18 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:99a172f3f4d0: avcodec/mss3: Check for the rac stream being invalid in rac_normalize()
[18:03:19 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:392c028cd23d: avcodec/apedec: Fix 2 signed overflows
[18:03:20 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:8174e5c77d8a: avcodec/cbs_av1_syntax_template: Check ref_frame_idx before use
[18:03:21 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:fe2e1cfc191d: avcodec/cfhd: Fix linesize type
[18:03:22 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:58e084bd2c7b: avcodec/cfhd: Check destination space for bayer before writing
[18:03:23 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:76133d7c8bfe: avformat/vividas: Fix memleak of AVIOContext in track_header()
[18:03:24 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:652ea23cb34b: avformat/realtextdec: Check for duplicate extradata in realtext_read_header()
[18:03:25 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:aecc9b96d613: avcodec/vorbisdec: Check parameters in vorbis_floor0_decode() before divide
[18:03:26 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:5a5f12e3b3f2: avcodec/vorbisdec: Implement vr->classifications = 1
[18:03:27 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:ccca484324e0: avcodec/ralf: Fix integer overflow in apply_lpc()
[18:03:28 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:3c06ba171697: avcodec/ralf: Fix undefined pointer in decode_channel()
[18:03:29 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:549fcba8fc83: avcodec/dirac_parser: Fix overflow in dts
[18:03:30 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:8305a4509af2: avcodec/loco: Check for end of input in pixel decode
[18:03:31 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:43015afd7ce9: avcodec/mss1: check for overread and forward errors
[18:03:32 CEST] <cone-962> ffmpeg 03Andreas Rheinhardt 07master:f12e662a3d3f: libavcodec/iff: Use unsigned to avoid undefined behaviour
[19:10:33 CEST] <cone-962> ffmpeg 03Jacob Siddall 07master:9051092e7366: avformat/rtpdec_rfc4175: Fix incorrect copy_offset calculation
[19:10:34 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:172a43ce36e6: avcodec/sanm: Check extradata_size before allocations
[19:10:35 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:02f909dc24b1: avcodec/vqavideo: Set video size
[19:10:36 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07master:be4fb282f9fb: avcodec/alsdec: Check for block_length <= 0 in read_var_block_data()
[19:51:00 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:99ecd0cfc926: avcodec/atrac9dec: Check grad_range[1] more tightly
[19:51:01 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:25aa7ddd3150: avcodec/apedec: Do not partially clear data array
[19:51:02 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:889fdc690a3f: avcodec/vc1_block: Check get_vlc2() return before use
[19:51:03 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:8b8f5fd05e7e: avcodec/mss3: Check for the rac stream being invalid in rac_normalize()
[19:51:04 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:739f93ebe1a2: avcodec/apedec: Fix 2 signed overflows
[19:51:05 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:f06fced6df9f: avcodec/cbs_av1_syntax_template: Check ref_frame_idx before use
[19:51:06 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:8d171808847e: avformat/vividas: Fix memleak of AVIOContext in track_header()
[19:51:07 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:f9597a5a111b: avformat/realtextdec: Check for duplicate extradata in realtext_read_header()
[19:51:08 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:4c16a8fe6719: avcodec/vorbisdec: Check parameters in vorbis_floor0_decode() before divide
[19:51:09 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:8b95d93e078c: avcodec/vorbisdec: Implement vr->classifications = 1
[19:51:10 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:414144a371bf: avcodec/ralf: Fix integer overflow in apply_lpc()
[19:51:11 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:5764b92f82c2: avcodec/ralf: Fix undefined pointer in decode_channel()
[19:51:12 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:3a3c02be9a4d: avcodec/dirac_parser: Fix overflow in dts
[19:51:13 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:11f01ad26c99: avcodec/loco: Check for end of input in pixel decode
[19:51:14 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:772d91d6b0c6: avcodec/mss1: check for overread and forward errors
[19:51:15 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:6443b95de666: avcodec/sanm: Check extradata_size before allocations
[19:51:16 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:06688a8cc51a: avcodec/vqavideo: Set video size
[19:51:17 CEST] <cone-962> ffmpeg 03Michael Niedermayer 07release/4.2:d5cd7fe5af3e: avcodec/alsdec: Check for block_length <= 0 in read_var_block_data()
[19:51:18 CEST] <cone-962> ffmpeg 03Andreas Rheinhardt 07release/4.2:34a40aeb73fe: libavcodec/iff: Use unsigned to avoid undefined behaviour
[20:06:17 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:f4b254e29921: avformat/av1: add color config values to AV1SequenceParameters
[20:06:18 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:a2df7e44b39a: avformat/av1: split off sequence header parsing from the av1C writing function
[20:06:19 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:6e53b43d487e: avformat/av1: rename some AV1SequenceParameters fields
[20:06:20 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:5152602ba8f4: avformat/av1: combine high_bitdepth and twelve_bit into a single bitdepth value
[20:06:21 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:d1c81070bce2: avformat/dashenc: update stream extradata from packet side data
[20:06:22 CEST] <cone-962> ffmpeg 03James Almer 07release/4.2:35e9d9cbf738: avformat/dashenc: fix writing the AV1 codec string in mp4 mode
[20:46:08 CEST] <durandal_1707> i dislike 4.2 release name
[20:55:10 CEST] <BradleyS> FFmpeg 4.3 "Paul"
[20:58:51 CEST] <thilo> FFmpeg 4.3 "Too late again"
[20:59:29 CEST] <kierank> durandal_1707: FFmpeg 4.3 "fuzz"
[21:40:08 CEST] <thardin> hot fuzz
[21:40:34 CEST] <durandal_1707> zzuf toh
[21:56:54 CEST] <jamrial> rcombs, BtbN: there's a configure fix for cuda_llvm
[21:57:17 CEST] <BtbN> I saw it, but I'm not sure if it's really neccesary
[22:18:31 CEST] <durandal_1707> this v360 tweet become almost viral
[22:48:34 CEST] <RiCON> BtbN: here's an example broken config.log: https://i.fsbn.eu/Lxvf.txt
[22:50:08 CEST] <BtbN> That's odd. Almost like the source_link variable is empty
[22:50:19 CEST] <RiCON> you can reproduce it in MSYS2 by not setting "MSYS=winsymlinks:nativestrict"
[22:50:57 CEST] <RiCON> it's not empty, it's the same thing as source_path, which is source_path=/build/ffmpeg-git
[22:51:16 CEST] <BtbN> I don't think the extra space is gonna break anything, so feel free to add that (and backport it to 4.2). But I kinda want to double check that configure is working properly.
[22:51:25 CEST] <RiCON> i don't have commit rights
[22:52:48 CEST] <RiCON> one would expect the -L/mingw64/lib options to also be broken, but maybe GCC has some extra patches that clang doesn't
[22:53:45 CEST] <BtbN> Probably depends on if clang.exe is MSYS-Aware or globally installed
[22:54:07 CEST] <nevcairiel> single-letter options with a path f ollowing it have a special msys hack
[22:54:20 CEST] <nevcairiel> it just doesnt work with longer options like -include
[22:54:27 CEST] <RiCON> yeah, that's it
[22:54:30 CEST] <nevcairiel> unless you use a space
[22:54:47 CEST] <RiCON> just tried with `winpty echo -L/mingw64` and `winpty echo -include/mingw64`
[22:55:02 CEST] <RiCON> the first one converts the path as expected, the second doesn't
[22:55:40 CEST] <BtbN> I really do wonder why source_link isn't just . though
[22:55:52 CEST] <RiCON> out-of-root build
[22:56:09 CEST] <RiCON> configure is in the parent dir
[22:56:15 CEST] <BtbN> With an out-of-tree build it should be just "src", using the src symlink created in the out-of-tree root
[22:56:28 CEST] <RiCON> it's an inner out-of-tree build
[22:56:41 CEST] <RiCON> there's no case for ../
[22:56:43 CEST] <BtbN> That doesn't matter. It's either "." or "src"
[22:57:02 CEST] <BtbN> Or am I reading that equation wrong?
[22:57:29 CEST] <RiCON> ffmpeg-git-repo/build_directory
[22:57:38 CEST] <BtbN> configure, line 5468.
[22:57:51 CEST] <BtbN> Whenever it's not "." or "src", create a symlink called "src" and set it to src
[22:58:03 CEST] <RiCON> yeah, but there's no symlinks enabled by default in MSYS2
[22:58:17 CEST] <BtbN> hm
[22:58:34 CEST] <RiCON> you need to set "MSYS=winsymlinks:nativestrict" before running (ba)sh
[22:58:38 CEST] <nevcairiel> proper symlinks have a spotty history on windows
[22:58:52 CEST] <nevcairiel> only recently you can use them without admin rights and special  settings
[22:59:27 CEST] <RiCON> `ln -s` does a copy if symlinks are disabled
[22:59:37 CEST] <nevcairiel> so out-of-tree windows builds will use the else path often
[23:00:11 CEST] <nevcairiel> msys2 should really detect if windows links are supported without special rights and then just use them automatically
[23:00:13 CEST] <RiCON> at most there could be an extra elif in L3762 for the `test -f ../configure` case
[23:00:19 CEST] <cone-962> ffmpeg 03Ricardo Constantino 07master:5ac28e9cc190: configure: cuda_llvm: fix include path for MSYS2
[23:00:27 CEST] <nevcairiel> that seems slightly excessive
[23:00:41 CEST] <cone-962> ffmpeg 03Ricardo Constantino 07release/4.2:75384bc46441: configure: cuda_llvm: fix include path for MSYS2
[23:04:18 CEST] <RiCON> thanks
[23:36:19 CEST] <cone-962> ffmpeg 03Ricardo Constantino 07n4.2:HEAD: configure: cuda_llvm: fix include path for MSYS2
[23:52:30 CEST] <rcombs> jamrial: looks fine to me, assuming clang is fine with the space
[23:52:58 CEST] <rcombs> jamrial: might want to test with a path containing a space, though?
[23:53:05 CEST] <rcombs> (I don't think the current code will handle that either)
[23:53:53 CEST] <jamrial> paths with a space are escaped, i think
[23:54:44 CEST] <nevcairiel> it'll also yell at you if you try to do a out-of-tree build with a path with spaces
[00:00:00 CEST] --- Tue Aug  6 2019