Builder ffmpegsos-solaris10-sparc Build #11904
Results:
Failed shell_2 shell_3 shell_4 shell_5
SourceStamp:
Project | ffmpeg |
Repository | https://git.ffmpeg.org/ffmpeg.git |
Branch | master |
Revision | d71c86313274465062d400b65df684ceb7659caf |
Got Revision | d71c86313274465062d400b65df684ceb7659caf |
Changes | 11 changes |
BuildSlave:
unstable10sReason:
The SingleBranchScheduler scheduler named 'schedule-ffmpegsos-solaris10-sparc' triggered this build
Steps and Logfiles:
-
git update ( 46 secs )
-
shell 'gsed -i ...' ( 0 secs )
-
shell_1 'gsed -i ...' ( 1 secs )
-
shell_2 'gsed -i ...' failed ( 1 secs )
-
shell_3 './configure --samples="../../../ffmpeg/fate-suite" ...' failed ( 29 secs )
-
shell_4 'gmake fate-rsync' failed ( 0 secs )
-
shell_5 '../../../ffmpeg/fate.sh ../../../ffmpeg/fate_config_sos.sh' failed ( 2 secs )
Build Properties:
Name | Value | Source |
---|---|---|
branch | master | Build |
builddir | /export/home/buildbot-unstable10s/slave/ffmpegsos-solaris10-sparc | slave |
buildername | ffmpegsos-solaris10-sparc | Builder |
buildnumber | 11904 | Build |
codebase | Build | |
got_revision | d71c86313274465062d400b65df684ceb7659caf | Git |
project | ffmpeg | Build |
repository | https://git.ffmpeg.org/ffmpeg.git | Build |
revision | d71c86313274465062d400b65df684ceb7659caf | Build |
scheduler | schedule-ffmpegsos-solaris10-sparc | Scheduler |
slavename | unstable10s | BuildSlave |
workdir | /export/home/buildbot-unstable10s/slave/ffmpegsos-solaris10-sparc | slave (deprecated) |
Forced Build Properties:
Name | Label | Value |
---|
Responsible Users:
- Andreas Rheinhardtandreas.rheinhardt@outlook.com
Timing:
Start | Tue Jun 17 01:16:19 2025 |
End | Tue Jun 17 01:17:42 2025 |
Elapsed | 1 mins, 23 secs |
All Changes:
:
Change #236853
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:30:30 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision e3ba364c5ee9747f53c94904608a075cadd53042 Comments
avformat/dhav: Fix check for seekability AVIOContext.seekable is a bitfield. Also check for seekability earlier. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavformat/dhav.c
Change #236854
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:30:30 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 0ae55affa3e5584594749e0d8531402b6b235373 Comments
avformat/dhav: Check reading data Prevents potential use of uninitialized data. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavformat/dhav.c
Change #236855
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision eeb82a440641e690ef4f26873ae605485a1169b3 Comments
avfilter/vf_overlay: Don't perform UB pointer arithmetic This happens when the pixel format of the output does not have an alpha channel. It leads to FATE failures with the ffmpeg-filter_colorkey, filter-overlay-dvdsub-2397 filter-overlay, filter-overlay_{gbrp_gbrap,nv12,nv21,yuv420,yuv420_yuva420, yuv420p10,yuv422_yuva422,yuv422p10,yuv444_yuva444,yuv444p10} and sub2video tests when using Clang UBSan. Fix this by only performing the pointer arithmetic when it is going to be used. This can be checked via variables that compile-time constants due to inlining, so that the checks are free. Given that the pointer is potentially used as a function argument, the compiler could elide the calculation, but not it can. The size of .text decreased by 1632B with GCC 14 and by 1392B with Clang 19 (both -O3). Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236856
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision ca51a0fef78a2ce5872c391ea4866dedf66c3679 Comments
avfilter/vf_overlay: Avoid converting stride to uint16_t and back Just keep the pointers for the beginning of a line uint8_t* and use uint16_t* to do the actual processing. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236857
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision b3edc848727201123bbcc822164f7cf1d5a85353 Comments
avfilter/vf_overlay: Pass variable type directly in macro Improves readability. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236858
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 48427b012daf37653e8111f82812cb91a7f66362 Comments
avfilter/vf_overlay: Use correct alpha when > 8 bits When chroma subsampling is in use, the filter averages the corresponding (non subsampled) alpha values to get the actual alpha value. When vertical subsampling is in use, the next line is accessed via a[src->linesize[3]], yet a is an uint16_t* for >8 bit formats and linesize is always in bytes, so that this actually uses the second line below the current one. This is fixed in this commit. No FATE test needed updates, because the filter-overlay-yuv420p10 and filter-overlay-yuv444p10 tests use a yuv420p test file that has constant opacity after conversion to yuva. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236859
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision a4b3474de6d4df02b6473d335d74a46a0952bfd6 Comments
avfilter/vf_overlay: Keep dst_step in bytes Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236860
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:33:09 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 730ffc5d35e0a2d48644da6f6880859bbbebd1c5 Comments
avfilter/vf_overlay: Hoist calculations out of loop Also use const where appropriate. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavfilter/vf_overlay.c
Change #236861
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:35:13 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 9102a90a95d9b65f482c281d886d94ef74ff0bd3 Comments
Revert "avcodec/decode: Fix avcodec parameters when bsfs are enable by decoder" This reverts commit 1c170613975d3cbcbb5aaa469b4a3cf0df5d4c2b. The commit intended to provide certain codecs using *_mp4toannexb bitstream filters with updated (annex B) extradata (even when the user-supplied one was ISOBMFF), yet BSFs are allowed to change way more. The media100_to_mjpegb BSF used by the media100 decoder changes the codec id; the commit being reverted therefore changed AVCodecContext.codec_id which is an API violation and broke media100 decoding with the FFmpeg cli tool. This commit also made changes from the internal BSF externally visible. extradata is documented to be "owned by the codec and freed in avcodec_free_context()" which does not include replacing it with something else in avcodec_open2() and may surprise users who think that AVCodecContext.extradata is immutable before avcodec_free_context(). It also incurred a memdup which is completely unnecessary for most decoders. Therefore this commit is reverted. The problem it tried to solve will be solved differently in the next commit. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/decode.c
Change #236862
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:35:13 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision ecd7c9244d9e8966ff167d5875f878cdd4ed0c98 Comments
avcodec/amfdec,rkmppdec: Use correct extradata with BSFs Otherwise the extradata used would be ISOBMFF if the input is even though we use the *_mp4toannexb BSFs to convert it to annex B to feed it to the actual decoder. (The mediacodec decoders also use said BSFs, yet they process the extradata in a way that works even when using the ISOBMFF extradata; in fact, using the converted extradata would break their check for whether to warn for missing extradata for the ISOBMFF without-in-band-header profiles. Furthermore, there are several users of the *_mp4toannexb BSFs that don't ever touch extradata. They have not been touched.) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/amfdec.c
- libavcodec/decode_bsf.h
- libavcodec/rkmppdec.c
Change #236863
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt @outlook.com>Changed at Tue 17 Jun 2025 00:35:13 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision d71c86313274465062d400b65df684ceb7659caf Comments
fate/video: Add media100 test Tests both the Media 100 decoder (using the media100_to_mjpegb BSF implicitly) as well as using said BSF, followed by the MJPEGB decoder. (We currently hit a bug when remuxing: The demuxer treats compressorname as encoded in a Mac character encoding (Mac OS Roman?) and converts it to UTF-8, yet the muxer just writes it.) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- tests/fate/video.mak
- tests/ref/fate/media100