diff options
Diffstat (limited to 'media-libs')
3 files changed, 426 insertions, 0 deletions
diff --git a/media-libs/openh264/files/openh264-2.4.0-fix-off-by-one-decode-regression.patch b/media-libs/openh264/files/openh264-2.4.0-fix-off-by-one-decode-regression.patch new file mode 100644 index 000000000000..0ef8e3dc56f2 --- /dev/null +++ b/media-libs/openh264/files/openh264-2.4.0-fix-off-by-one-decode-regression.patch @@ -0,0 +1,26 @@ +From ff7cc30b59bea581b3a9455009cc0deb67fee98c Mon Sep 17 00:00:00 2001 +From: Kalev Lember <klember@redhat.com> +Date: Wed, 29 Nov 2023 13:40:53 +0100 +Subject: [PATCH] Fix off by one regression in decoder + +Fix iPicBuffIdx bounds check introduced in commit +986bd65b711191d4883c54ace32a9879e17729c2 and allow 0 as an index value. + +This fixes Big_Buck_Bunny_720_10s_30MB.mp4 playback with gst-play-1.0. +--- + codec/decoder/plus/src/welsDecoderExt.cpp | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/codec/decoder/plus/src/welsDecoderExt.cpp b/codec/decoder/plus/src/welsDecoderExt.cpp +index 170d17dec..457effb68 100644 +--- a/codec/decoder/plus/src/welsDecoderExt.cpp ++++ b/codec/decoder/plus/src/welsDecoderExt.cpp +@@ -1136,7 +1136,7 @@ void CWelsDecoder::ReleaseBufferedReadyPictureReorder (PWelsDecoderContext pCtx, + m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPOC = IMinInt32; + int32_t iPicBuffIdx = m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPicBuffIdx; + if (pPicBuff != NULL) { +- if (iPicBuffIdx > 0 && iPicBuffIdx < pPicBuff->iCapacity) ++ if (iPicBuffIdx >= 0 && iPicBuffIdx < pPicBuff->iCapacity) + { + PPicture pPic = pPicBuff->ppPic[iPicBuffIdx]; + --pPic->iRefCount; diff --git a/media-libs/openh264/files/openh264-2.4.0-unordered-frame-glitches.patch b/media-libs/openh264/files/openh264-2.4.0-unordered-frame-glitches.patch new file mode 100644 index 000000000000..373fac95f940 --- /dev/null +++ b/media-libs/openh264/files/openh264-2.4.0-unordered-frame-glitches.patch @@ -0,0 +1,264 @@ +From 56d3ec4c7bfdc545a840512a8f2c72545889c538 Mon Sep 17 00:00:00 2001 +From: Takashi Yano <takashi.yano@nifty.ne.jp> +Date: Tue, 5 Dec 2023 21:48:40 +0900 +Subject: [PATCH 1/3] Add missing iLastWrittenPOC setting in unbuffered + reordering. + +In CWelsDecoder::ReorderPicturesInDisplay(), iLastWrittenPOC was +not set in unbuffered-reordering case. Due to this problem, it +sometimes reordered the frames incorrectly. This patch fixes the +issue. +--- + codec/decoder/plus/src/welsDecoderExt.cpp | 1 + + 1 file changed, 1 insertion(+) + +diff --git a/codec/decoder/plus/src/welsDecoderExt.cpp b/codec/decoder/plus/src/welsDecoderExt.cpp +index 457effb68..a1b26c2cb 100644 +--- a/codec/decoder/plus/src/welsDecoderExt.cpp ++++ b/codec/decoder/plus/src/welsDecoderExt.cpp +@@ -1206,6 +1206,7 @@ DECODING_STATE CWelsDecoder::ReorderPicturesInDisplay(PWelsDecoderContext pDecCo + if (pDstInfo->iBufferStatus == 1) { + if (m_sReoderingStatus.iLastGOPRemainPicts == 0 && pDecContext->pSliceHeader->eSliceType == B_SLICE && + pDecContext->pSliceHeader->iPicOrderCntLsb <= m_sReoderingStatus.iLastWrittenPOC + 2) { ++ m_sReoderingStatus.iLastWrittenPOC = pDecContext->pSliceHeader->iPicOrderCntLsb; + //issue #3478, use b-slice type to determine correct picture order as the first priority as POC order is not as reliable as based on b-slice + ppDst[0] = pDstInfo->pDst[0]; + ppDst[1] = pDstInfo->pDst[1]; + +From 10d0998a966dc98ba1a93122f214c2aef5bcd33f Mon Sep 17 00:00:00 2001 +From: Takashi Yano <takashi.yano@nifty.ne.jp> +Date: Wed, 6 Dec 2023 02:08:19 +0900 +Subject: [PATCH 2/3] Prevent frame buffer from overwrite at GOP change. + +When bNewSeqBegin got true, iRefCount was forcibly reset even if the +buffer was still used for reordering. Due to this problem, the buffer +in use was sometimes overwritten with newly decoded frame. This commit +is for fixing that problem. +--- + codec/decoder/core/src/manage_dec_ref.cpp | 37 ++++++++++++----------- + codec/decoder/plus/src/welsDecoderExt.cpp | 4 +++ + 2 files changed, 24 insertions(+), 17 deletions(-) + +diff --git a/codec/decoder/core/src/manage_dec_ref.cpp b/codec/decoder/core/src/manage_dec_ref.cpp +index 88099b60e..c3c983129 100644 +--- a/codec/decoder/core/src/manage_dec_ref.cpp ++++ b/codec/decoder/core/src/manage_dec_ref.cpp +@@ -66,8 +66,10 @@ int32_t GetLTRFrameIndex (PRefPic pRefPic, int32_t iAncLTRFrameNum); + #endif + static int32_t RemainOneBufferInDpbForEC (PWelsDecoderContext pCtx, PRefPic pRefPic); + +-static void SetUnRef (PPicture pRef) { +- if (NULL != pRef) { ++static void SetUnRef (PPicture pRef, bool bNewSeqBegin) { ++ if (pRef == NULL) return; ++ ++ if (pRef->iRefCount <= 0 && (!pRef->bUsedAsRef || bNewSeqBegin)) { + pRef->bUsedAsRef = false; + pRef->bIsLongRef = false; + pRef->iFrameNum = -1; +@@ -81,17 +83,18 @@ static void SetUnRef (PPicture pRef) { + pRef->iSpsId = -1; + pRef->bIsComplete = false; + pRef->iRefCount = 0; ++ } + +- if (pRef->eSliceType == I_SLICE) { +- return; +- } +- int32_t lists = pRef->eSliceType == P_SLICE ? 1 : 2; +- for (int32_t i = 0; i < MAX_DPB_COUNT; ++i) { +- for (int32_t list = 0; list < lists; ++list) { +- if (pRef->pRefPic[list][i] != NULL) { +- pRef->pRefPic[list][i]->iRefCount = 0; +- pRef->pRefPic[list][i] = NULL; +- } ++ if (pRef->eSliceType == I_SLICE) { ++ return; ++ } ++ int32_t lists = pRef->eSliceType == P_SLICE ? 1 : 2; ++ for (int32_t i = 0; i < MAX_DPB_COUNT; ++i) { ++ for (int32_t list = 0; list < lists; ++list) { ++ if (pRef->pRefPic[list][i] != NULL) { ++ if (pRef->pRefPic[list][i]->iRefCount > 0) continue; ++ pRef->pRefPic[list][i]->iRefCount = 0; ++ pRef->pRefPic[list][i] = NULL; + } + } + } +@@ -111,7 +114,7 @@ void WelsResetRefPic (PWelsDecoderContext pCtx) { + + for (i = 0; i < MAX_DPB_COUNT; i++) { + if (pRefPic->pShortRefList[LIST_0][i] != NULL) { +- SetUnRef (pRefPic->pShortRefList[LIST_0][i]); ++ SetUnRef (pRefPic->pShortRefList[LIST_0][i], pCtx->bNewSeqBegin); + pRefPic->pShortRefList[LIST_0][i] = NULL; + } + } +@@ -119,7 +122,7 @@ void WelsResetRefPic (PWelsDecoderContext pCtx) { + + for (i = 0; i < MAX_DPB_COUNT; i++) { + if (pRefPic->pLongRefList[LIST_0][i] != NULL) { +- SetUnRef (pRefPic->pLongRefList[LIST_0][i]); ++ SetUnRef (pRefPic->pLongRefList[LIST_0][i], pCtx->bNewSeqBegin); + pRefPic->pLongRefList[LIST_0][i] = NULL; + } + } +@@ -767,7 +770,7 @@ static int32_t SlidingWindow (PWelsDecoderContext pCtx, PRefPic pRefPic) { + for (i = pRefPic->uiShortRefCount[LIST_0] - 1; i >= 0; i--) { + pPic = WelsDelShortFromList (pRefPic, pRefPic->pShortRefList[LIST_0][i]->iFrameNum); + if (pPic) { +- SetUnRef (pPic); ++ SetUnRef (pPic, pCtx->bNewSeqBegin); + break; + } else { + return ERR_INFO_INVALID_MMCO_REF_NUM_OVERFLOW; +@@ -803,7 +806,7 @@ static PPicture WelsDelShortFromList (PRefPic pRefPic, int32_t iFrameNum) { + static PPicture WelsDelShortFromListSetUnref (PRefPic pRefPic, int32_t iFrameNum) { + PPicture pPic = WelsDelShortFromList (pRefPic, iFrameNum); + if (pPic) { +- SetUnRef (pPic); ++ SetUnRef (pPic, false); + } + return pPic; + } +@@ -832,7 +835,7 @@ static PPicture WelsDelLongFromList (PRefPic pRefPic, uint32_t uiLongTermFrameId + static PPicture WelsDelLongFromListSetUnref (PRefPic pRefPic, uint32_t uiLongTermFrameIdx) { + PPicture pPic = WelsDelLongFromList (pRefPic, uiLongTermFrameIdx); + if (pPic) { +- SetUnRef (pPic); ++ SetUnRef (pPic, false); + } + return pPic; + } +diff --git a/codec/decoder/plus/src/welsDecoderExt.cpp b/codec/decoder/plus/src/welsDecoderExt.cpp +index a1b26c2cb..14257afff 100644 +--- a/codec/decoder/plus/src/welsDecoderExt.cpp ++++ b/codec/decoder/plus/src/welsDecoderExt.cpp +@@ -1084,6 +1084,8 @@ void CWelsDecoder::ReleaseBufferedReadyPictureReorder (PWelsDecoderContext pCtx, + if (pPicBuff != NULL) { + PPicture pPic = pPicBuff->ppPic[m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPicBuffIdx]; + --pPic->iRefCount; ++ if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) ++ pPic->bUsedAsRef = false; + } + m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP = false; + m_sReoderingStatus.iMinPOC = IMinInt32; +@@ -1187,6 +1189,8 @@ void CWelsDecoder::ReleaseBufferedReadyPictureNoReorder(PWelsDecoderContext pCtx + PPicBuff pPicBuff = pCtx ? pCtx->pPicBuff : m_pPicBuff; + PPicture pPic = pPicBuff->ppPic[m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPicBuffIdx]; + --pPic->iRefCount; ++ if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) ++ pPic->bUsedAsRef = false; + } + if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) { + --m_sReoderingStatus.iLastGOPRemainPicts; + +From f82abc09a3e7d723e21a06ec7e5f38afc218357f Mon Sep 17 00:00:00 2001 +From: Takashi Yano <takashi.yano@nifty.ne.jp> +Date: Tue, 12 Dec 2023 19:23:32 +0900 +Subject: [PATCH 3/3] Fix test failure due to the issue of the previous commit. + +--- + codec/decoder/core/src/manage_dec_ref.cpp | 14 +++++++------- + codec/decoder/plus/src/welsDecoderExt.cpp | 9 +++++++-- + 2 files changed, 14 insertions(+), 9 deletions(-) + +diff --git a/codec/decoder/core/src/manage_dec_ref.cpp b/codec/decoder/core/src/manage_dec_ref.cpp +index c3c983129..410b9c47f 100644 +--- a/codec/decoder/core/src/manage_dec_ref.cpp ++++ b/codec/decoder/core/src/manage_dec_ref.cpp +@@ -66,10 +66,10 @@ int32_t GetLTRFrameIndex (PRefPic pRefPic, int32_t iAncLTRFrameNum); + #endif + static int32_t RemainOneBufferInDpbForEC (PWelsDecoderContext pCtx, PRefPic pRefPic); + +-static void SetUnRef (PPicture pRef, bool bNewSeqBegin) { ++static void SetUnRef (PPicture pRef) { + if (pRef == NULL) return; + +- if (pRef->iRefCount <= 0 && (!pRef->bUsedAsRef || bNewSeqBegin)) { ++ if (pRef->iRefCount <= 0) { + pRef->bUsedAsRef = false; + pRef->bIsLongRef = false; + pRef->iFrameNum = -1; +@@ -114,7 +114,7 @@ void WelsResetRefPic (PWelsDecoderContext pCtx) { + + for (i = 0; i < MAX_DPB_COUNT; i++) { + if (pRefPic->pShortRefList[LIST_0][i] != NULL) { +- SetUnRef (pRefPic->pShortRefList[LIST_0][i], pCtx->bNewSeqBegin); ++ SetUnRef (pRefPic->pShortRefList[LIST_0][i]); + pRefPic->pShortRefList[LIST_0][i] = NULL; + } + } +@@ -122,7 +122,7 @@ void WelsResetRefPic (PWelsDecoderContext pCtx) { + + for (i = 0; i < MAX_DPB_COUNT; i++) { + if (pRefPic->pLongRefList[LIST_0][i] != NULL) { +- SetUnRef (pRefPic->pLongRefList[LIST_0][i], pCtx->bNewSeqBegin); ++ SetUnRef (pRefPic->pLongRefList[LIST_0][i]); + pRefPic->pLongRefList[LIST_0][i] = NULL; + } + } +@@ -770,7 +770,7 @@ static int32_t SlidingWindow (PWelsDecoderContext pCtx, PRefPic pRefPic) { + for (i = pRefPic->uiShortRefCount[LIST_0] - 1; i >= 0; i--) { + pPic = WelsDelShortFromList (pRefPic, pRefPic->pShortRefList[LIST_0][i]->iFrameNum); + if (pPic) { +- SetUnRef (pPic, pCtx->bNewSeqBegin); ++ SetUnRef (pPic); + break; + } else { + return ERR_INFO_INVALID_MMCO_REF_NUM_OVERFLOW; +@@ -806,7 +806,7 @@ static PPicture WelsDelShortFromList (PRefPic pRefPic, int32_t iFrameNum) { + static PPicture WelsDelShortFromListSetUnref (PRefPic pRefPic, int32_t iFrameNum) { + PPicture pPic = WelsDelShortFromList (pRefPic, iFrameNum); + if (pPic) { +- SetUnRef (pPic, false); ++ SetUnRef (pPic); + } + return pPic; + } +@@ -835,7 +835,7 @@ static PPicture WelsDelLongFromList (PRefPic pRefPic, uint32_t uiLongTermFrameId + static PPicture WelsDelLongFromListSetUnref (PRefPic pRefPic, uint32_t uiLongTermFrameIdx) { + PPicture pPic = WelsDelLongFromList (pRefPic, uiLongTermFrameIdx); + if (pPic) { +- SetUnRef (pPic, false); ++ SetUnRef (pPic); + } + return pPic; + } +diff --git a/codec/decoder/plus/src/welsDecoderExt.cpp b/codec/decoder/plus/src/welsDecoderExt.cpp +index 14257afff..b0936a7d8 100644 +--- a/codec/decoder/plus/src/welsDecoderExt.cpp ++++ b/codec/decoder/plus/src/welsDecoderExt.cpp +@@ -1083,8 +1083,9 @@ void CWelsDecoder::ReleaseBufferedReadyPictureReorder (PWelsDecoderContext pCtx, + m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPOC = IMinInt32; + if (pPicBuff != NULL) { + PPicture pPic = pPicBuff->ppPic[m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPicBuffIdx]; ++ bool bLastGOP = m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP; + --pPic->iRefCount; +- if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) ++ if (pPic->iRefCount <= 0 && bLastGOP) + pPic->bUsedAsRef = false; + } + m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP = false; +@@ -1141,7 +1142,10 @@ void CWelsDecoder::ReleaseBufferedReadyPictureReorder (PWelsDecoderContext pCtx, + if (iPicBuffIdx >= 0 && iPicBuffIdx < pPicBuff->iCapacity) + { + PPicture pPic = pPicBuff->ppPic[iPicBuffIdx]; ++ bool bLastGOP = m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP; + --pPic->iRefCount; ++ if (pPic->iRefCount <= 0 && bLastGOP) ++ pPic->bUsedAsRef = false; + } + } + m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP = false; +@@ -1188,8 +1192,9 @@ void CWelsDecoder::ReleaseBufferedReadyPictureNoReorder(PWelsDecoderContext pCtx + if (pCtx || m_pPicBuff) { + PPicBuff pPicBuff = pCtx ? pCtx->pPicBuff : m_pPicBuff; + PPicture pPic = pPicBuff->ppPic[m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].iPicBuffIdx]; ++ bool bLastGOP = m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP; + --pPic->iRefCount; +- if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) ++ if (pPic->iRefCount <= 0 && bLastGOP) + pPic->bUsedAsRef = false; + } + if (m_sPictInfoList[m_sReoderingStatus.iPictInfoIndex].bLastGOP) { diff --git a/media-libs/openh264/openh264-2.4.0-r1.ebuild b/media-libs/openh264/openh264-2.4.0-r1.ebuild new file mode 100644 index 000000000000..a98825f85f9b --- /dev/null +++ b/media-libs/openh264/openh264-2.4.0-r1.ebuild @@ -0,0 +1,136 @@ +# Copyright 1999-2023 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +EAPI=8 + +inherit toolchain-funcs multilib-minimal + +MOZVER=114_2 +MY_GMP_COMMIT="e7d30b921df736a1121a0c8e0cf3ab1ce5b8a4b7" + +DESCRIPTION="Cisco OpenH264 library and Gecko Media Plugin for Mozilla packages" +HOMEPAGE="https://www.openh264.org/ https://github.com/cisco/openh264" +SRC_URI="https://github.com/cisco/openh264/archive/refs/tags/v${PV}.tar.gz -> ${P}.tar.gz + https://github.com/mozilla/gmp-api/archive/${MY_GMP_COMMIT}.tar.gz -> gmp-api-Firefox${MOZVER}-${MY_GMP_COMMIT}.tar.gz" +LICENSE="BSD" + +# openh264 soname version. +# (2.2.0 needed a minor bump due to undocumented but breaking ABI changes, just to be sure. +# https://github.com/cisco/openh264/issues/3459 ) +SLOT="0/7" +KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~loong ~mips ~ppc ~ppc64 ~riscv ~sparc ~x86" +IUSE="cpu_flags_arm_neon cpu_flags_x86_avx2 +plugin test utils" + +RESTRICT="bindist !test? ( test )" + +BDEPEND=" + abi_x86_32? ( dev-lang/nasm ) + abi_x86_64? ( dev-lang/nasm ) + test? ( dev-cpp/gtest[${MULTILIB_USEDEP}] )" + +DOCS=( LICENSE CONTRIBUTORS README.md ) + +PATCHES=( + "${FILESDIR}"/openh264-2.3.0-pkgconfig-pathfix.patch + "${FILESDIR}"/${PN}-2.3.1-pr3630.patch + "${FILESDIR}"/openh264-2.4.0-fix-off-by-one-decode-regression.patch + "${FILESDIR}"/openh264-2.4.0-unordered-frame-glitches.patch +) + +src_prepare() { + default + + ln -svf "/dev/null" "build/gtest-targets.mk" || die + sed -i -e 's/$(LIBPREFIX)gtest.$(LIBSUFFIX)//g' Makefile || die + + sed -i -e 's/ | generate-version//g' Makefile || die + sed -e 's|$FULL_VERSION|""|g' codec/common/inc/version_gen.h.template > \ + codec/common/inc/version_gen.h + + multilib_copy_sources +} + +multilib_src_configure() { + ln -s "${WORKDIR}"/gmp-api-${MY_GMP_COMMIT} gmp-api || die +} + +emakecmd() { + CC="$(tc-getCC)" CXX="$(tc-getCXX)" LD="$(tc-getLD)" AR="$(tc-getAR)" \ + emake V=Yes CFLAGS_M32="" CFLAGS_M64="" CFLAGS_OPT="" \ + PREFIX="${EPREFIX}/usr" \ + LIBDIR_NAME="$(get_libdir)" \ + SHAREDLIB_DIR="${EPREFIX}/usr/$(get_libdir)" \ + INCLUDES_DIR="${EPREFIX}/usr/include/${PN}" \ + HAVE_AVX2=$(usex cpu_flags_x86_avx2 Yes No) \ + HAVE_GTEST=$(usex test Yes No) \ + ARCH="$(tc-arch)" \ + ENABLEPIC="Yes" \ + $@ +} + +multilib_src_compile() { + local myopts="ENABLE64BIT=No" + case "${ABI}" in + s390x|alpha|*64) myopts="ENABLE64BIT=Yes";; + esac + + if use arm; then + myopts+=" USE_ASM=$(usex cpu_flags_arm_neon Yes No)" + fi + + emakecmd ${myopts} + use plugin && emakecmd ${myopts} plugin +} + +multilib_src_test() { + emakecmd test +} + +multilib_src_install() { + emakecmd DESTDIR="${D}" install-shared + + if use utils; then + newbin h264enc openh264enc + newbin h264dec openh264dec + fi + + if use plugin; then + local plugpath="${ROOT}/usr/$(get_libdir)/nsbrowser/plugins/gmp-gmp${PN}/system-installed" + insinto "${plugpath}" + doins libgmpopenh264.so* gmpopenh264.info + echo "MOZ_GMP_PATH=\"${plugpath}\"" >"${T}"/98-moz-gmp-${PN} + doenvd "${T}"/98-moz-gmp-${PN} + + cat <<PREFEOF >"${T}"/${P}.js +pref("media.gmp-gmp${PN}.autoupdate", false); +pref("media.gmp-gmp${PN}.version", "system-installed"); +PREFEOF + + insinto /usr/$(get_libdir)/firefox/defaults/pref + newins "${T}"/${P}.js ${PN}-${PV/_p*/}.js + + insinto /usr/$(get_libdir)/seamonkey/defaults/pref + newins "${T}"/${P}.js ${PN}-${PV/_p*/}.js + fi +} + +pkg_postinst() { + if use plugin; then + if [[ -z ${REPLACING_VERSIONS} ]]; then + elog "Please restart your login session, in order for the session's environment" + elog "to include the new MOZ_GMP_PATH variable." + elog "" + fi + elog "This package attempts to override the Mozilla GMPInstaller auto-update process," + elog "however even if it is not successful in doing so the profile-installed plugin" + elog "will not be used unless this package is removed. This package will take precedence" + elog "over any gmp-gmpopenh264 that may be installed in a user's profile." + elog "" + fi + + if use utils; then + elog "Utilities h264enc and h264dec are installed as openh264enc and openh264dec" + elog "to avoid file collisions with media-video/h264enc" + elog "" + fi +} |