From 70354524f5a55ba6c3ee7ea877cf17b6074a3febf7ec7d375d8abcb79d16be03 Mon Sep 17 00:00:00 2001 From: OBS User autobuild Date: Tue, 25 May 2010 12:10:02 +0000 Subject: [PATCH] Accepting request 40589 from multimedia:libs checked in (request 40589) OBS-URL: https://build.opensuse.org/request/show/40589 OBS-URL: https://build.opensuse.org/package/show/multimedia:libs/dirac?expand=0&rev=18 --- ...d-memory-read-that-causes-the-encode.patch | 52 ------------------- dirac.changes | 12 ----- dirac.spec | 40 +++----------- 3 files changed, 8 insertions(+), 96 deletions(-) delete mode 100644 0001-Fix-uninitialised-memory-read-that-causes-the-encode.patch diff --git a/0001-Fix-uninitialised-memory-read-that-causes-the-encode.patch b/0001-Fix-uninitialised-memory-read-that-causes-the-encode.patch deleted file mode 100644 index 0392ec5..0000000 --- a/0001-Fix-uninitialised-memory-read-that-causes-the-encode.patch +++ /dev/null @@ -1,52 +0,0 @@ -From 88fae6224b54ecf92ac6e3ae5083c3f352adc798 Mon Sep 17 00:00:00 2001 -From: Anuradha Suraparaju -Date: Fri, 17 Apr 2009 17:47:27 +1000 -Subject: [PATCH] Fix uninitialised memory read that causes the encoder to crash when using - non-overlapping blocks. - ---- - libdirac_common/mot_comp.cpp | 21 +++++++++++++++------ - 1 files changed, 15 insertions(+), 6 deletions(-) - -diff --git a/libdirac_common/mot_comp.cpp b/libdirac_common/mot_comp.cpp -index aa8d37c..e0ec2ff 100644 ---- a/libdirac_common/mot_comp.cpp -+++ b/libdirac_common/mot_comp.cpp -@@ -306,6 +306,8 @@ void MotionCompensator::CompensateComponent( Picture* pic , - - int save_from_row = m_bparams.Ybsep()-m_bparams.Yoffset(); - -+ bool row_overlap = ((m_bparams.Yblen() - m_bparams.Ybsep()) > 0); -+ - // unpadded picture dimensions - const int x_end_data = pic_data_out.FirstX() + std::min(pic_data_out.LengthX(), pic_size.x ); - const int y_end_data = pic_data_out.FirstY() + std::min(pic_data_out.LengthY(), pic_size.y ); -@@ -489,12 +491,19 @@ void MotionCompensator::CompensateComponent( Picture* pic , - //Increment the block vertical position - pos.y += m_bparams.Ybsep(); - -- // Copy the rows required to motion compensate the next row of block. -- // This is usually Yblen-Ybsep rows. -- memmove (pic_data[0], pic_data[save_from_row], (m_bparams.Yblen() - save_from_row)*pic_data.LengthX()*sizeof(ValueType)); -- memset( pic_data[m_bparams.Yblen() - save_from_row], 0, save_from_row*pic_data.LengthX()*sizeof(ValueType) ); -- save_from_row = m_bparams.Ybsep(); -- -+ if (row_overlap) -+ { -+ // Copy the rows required to motion compensate the next row of -+ // blocks. This is usually Yblen-Ybsep rows. -+ memmove (pic_data[0], pic_data[save_from_row], (m_bparams.Yblen() - save_from_row)*pic_data.LengthX()*sizeof(ValueType)); -+ memset( pic_data[m_bparams.Yblen() - save_from_row], 0, save_from_row*pic_data.LengthX()*sizeof(ValueType) ); -+ save_from_row = m_bparams.Ybsep(); -+ } -+ else -+ { -+ // no row overlap. So reset pic_data to 0. -+ memset( pic_data[0], 0, m_bparams.Yblen()*pic_data.LengthX()*sizeof(ValueType) ); -+ } - }//yblock - - if ( m_add_or_sub == SUBTRACT) --- -1.7.0.3 - diff --git a/dirac.changes b/dirac.changes index fbdffb3..65278f8 100644 --- a/dirac.changes +++ b/dirac.changes @@ -1,15 +1,3 @@ -------------------------------------------------------------------- -Mon May 24 13:25:36 UTC 2010 - davejplater@gmail.com - -- Split off doc subpackage - -------------------------------------------------------------------- -Mon May 24 07:01:37 UTC 2010 - davejplater@gmail.com - -- Added Provides: and Obsoletes: for packman libdirac-devel -- Included patch from dirac git to fix uninitialised memory read - that causes the encoder to crash when using non-overlapping blocks. - ------------------------------------------------------------------- Sun May 9 08:49:10 UTC 2010 - davejplater@gmail.com diff --git a/dirac.spec b/dirac.spec index bd51699..c96879b 100644 --- a/dirac.spec +++ b/dirac.spec @@ -24,10 +24,9 @@ Release: 4 Url: http://diracvideo.org/ Group: Productivity/Multimedia/Video/Editors and Convertors License: MPL .. -Summary: The Dirac_Video Codec +Summary: The dirac Video Codec Source: %{name}-%{version}.tar.bz2 -Patch0: 0001-Fix-uninitialised-memory-read-that-causes-the-encode.patch -Patch1: %{name}-%{version}-gcc45.patch +Patch0: %{name}-%{version}-gcc45.patch BuildRequires: gcc-c++ libstdc++-devel BuildRequires: libcppunit-devel %ifnarch ia64 s390 s390x %sparc @@ -167,11 +166,8 @@ Authors: License: MPL .. Group: Development/Libraries/C and C++ Summary: Development Files for Dirac Video Codec -AutoReqProv: on Requires: libdirac_decoder0 = %{version} Requires: libdirac_encoder0 = %{version} -Provides: libdirac-devel = %{version}-%{release} -Obsoletes: libdirac-devel < %{version}-%{release} %description devel Dirac is an open source video codec. It uses a traditional hybrid video @@ -212,22 +208,9 @@ Authors: Peter Bleackley , BBC Research and Development Myo Tun , Brunel University -%package doc -License: MPL .. -Group: Documentation/HTML -Summary: Dirac Video Codec api documentation -%if 0%{?suse_version} > 1110 -BuildArch: noarch -%endif - -%description doc -This package provides the api reference documentation for using dirac libraries -in your program. - %prep %setup -q %patch0 -p1 -%patch1 -p1 # Code is not mature enough for -Werror (unused results) sed -i 's/-Werror//' configure.ac @@ -241,10 +224,8 @@ export CXXFLAGS="$RPM_OPT_FLAGS -O0" %endif %endif %configure\ - --disable-static \ - --docdir=%{_docdir}/packages/%{name} -%{__make} %{?jobs:-j%jobs} -#htmldir=%{_docdir}/%{name}/code/api_html + --disable-static +%{__make} %{?jobs:-j%jobs} htmldir=%{_docdir}/%{name}/code/api_html %install %makeinstall htmldir=%{_docdir}/%{name}/code/api_html @@ -265,8 +246,8 @@ export CXXFLAGS="$RPM_OPT_FLAGS -O0" %files %defattr (-, root, root) -#%%doc AUTHORS COPYING ChangeLog NEWS README TODO -#%%doc %{_docdir}/%{name}/[ACNRT]* +%doc %dir %{_docdir}/%{name} +%doc %{_docdir}/%{name}/[ACNRT]* %{_bindir}/* %files -n libdirac_decoder0 @@ -279,15 +260,10 @@ export CXXFLAGS="$RPM_OPT_FLAGS -O0" %files devel %defattr (-, root, root) -#%%doc %{_docdir}/%{name}/[ACNRT]* +%doc %{_docdir}/%{name}/code %{_libdir}/*.so #%{_libdir}/*.*a %{_libdir}/pkgconfig/*.pc -%{_includedir}/%{name} - -%files doc -%defattr (-, root, root) -%doc %{_docdir}/%{name} - +%{_includedir}/dirac %changelog