rpms/gstreamer-plugins-bad/devel 0001-fix-faad2-version-check.patch, NONE, 1.1 gstreamer-plugins-bad.spec, 1.52, 1.53

[Hans de Goede]

Author: jwrdegoede

Update of /cvs/free/rpms/gstreamer-plugins-bad/devel
In directory old02.ovh.rpmfusion.lan:/tmp/cvs-serv23095

Modified Files:
	gstreamer-plugins-bad.spec 
Added Files:
	0001-fix-faad2-version-check.patch 
Log Message:
* Wed May 27 2015 Hans de Goede <j.w.r.degoede at gmail.com> - 0.10.23-7
- Add a patch from upstream fixing a faad2 crash (rf3664)


0001-fix-faad2-version-check.patch:
 configure.ac |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

--- NEW FILE 0001-fix-faad2-version-check.patch ---
>From ea2cf4aae775c36793b31fa2726fa084c426d0cf Mon Sep 17 00:00:00 2001
From: Wim Taymans <wtaymans at redhat.com>
Date: Wed, 13 May 2015 16:23:26 +0200
Subject: fix faad2 version check

On fedora 22, the output of cpp inserts extra debug comments, which
makes our regexp for the faad2 version check fail. This in turn causes
it to compile with the wrong arguments passed which then causes stack
corruption and crashes.

Fix this by only checking for the version (which should be by itself on
a single line). This is potentially less safe, it might be possible that
a similar string would appear in a later version in the header file.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=748571

diff --git a/configure.ac b/configure.ac
index 29b10a6..ceaf31f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1132,9 +1132,9 @@ AG_GST_CHECK_FEATURE(FAAD, [AAC decoder plug-in], faad, [
       AC_MSG_CHECKING([Checking FAAD2 version in $faad_hdr])
       for minor in 10 9 8 7 6 5 0; do
         if test x$faad2_minor_version = "x"; then
-          AC_EGREP_CPP([GST_CHECK_FAAD_VERSION \"2\.$minor\"], [
+          AC_EGREP_CPP([\"2\.$minor\"$], [
               #include <$faad_hdr>
-              GST_CHECK_FAAD_VERSION FAAD2_VERSION
+              FAAD2_VERSION
             ], [
               faad2_minor_version=$minor
             ])
-- 
cgit v0.10.2



Index: gstreamer-plugins-bad.spec
===================================================================
RCS file: /cvs/free/rpms/gstreamer-plugins-bad/devel/gstreamer-plugins-bad.spec,v
retrieving revision 1.52
retrieving revision 1.53
diff -u -r1.52 -r1.53
--- gstreamer-plugins-bad.spec	1 Sep 2014 02:20:54 -0000	1.52
+++ gstreamer-plugins-bad.spec	27 May 2015 15:31:18 -0000	1.53
@@ -15,11 +15,12 @@
 Summary: GStreamer streaming media framework "bad" plug-ins
 Name: gstreamer-plugins-bad
 Version: 0.10.23
-Release: 6%{?dist}
+Release: 7%{?dist}
 License: LGPLv2+
 Group: Applications/Multimedia
 URL: http://gstreamer.freedesktop.org/
 Source: http://gstreamer.freedesktop.org/src/gst-plugins-bad/gst-plugins-bad-%{version}.tar.bz2
+Patch1: 0001-fix-faad2-version-check.patch
 Requires: %{gstreamer} >= %{gst_minver}
 # Drag in the free plugins which are in Fedora now, for upgrade path
 Requires: gstreamer-plugins-bad-free >= %{version}
@@ -40,6 +41,8 @@
 BuildRequires: libmimic-devel
 BuildRequires: librtmp-devel
 BuildRequires: vo-amrwbenc-devel
+# For autoreconf
+BuildRequires: libtool
 
 %description
 GStreamer is a streaming media framework, based on graphs of elements which
@@ -51,6 +54,9 @@
 
 %prep
 %setup -q -n gst-plugins-bad-%{version}
+%patch1 -p1
+# For patch1
+autoreconf -ivf
 
 
 %build
@@ -82,7 +88,6 @@
 
 
 %files
-%defattr(-,root,root,-)
 %doc AUTHORS COPYING README REQUIREMENTS
 # Take the whole dir for proper dir ownership (shared with other plugin pkgs)
 %{_datadir}/gstreamer-0.10
@@ -107,6 +112,9 @@
 
 
 %changelog
+* Wed May 27 2015 Hans de Goede <j.w.r.degoede at gmail.com> - 0.10.23-7
+- Add a patch from upstream fixing a faad2 crash (rf3664)
+
 * Mon Sep 01 2014 Sérgio Basto <sergio at serjux.com> - 0.10.23-6
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_21_22_Mass_Rebuild
 
@@ -357,7 +365,7 @@
 - Re-add devel package now that we have a main shared lib and header files.
 - Add check build requirement.
 
-* Wed Mar 30 2007 Matthias Saou <http://freshrpms.net/> 0.10.4-1
+* Fri Mar 30 2007 Matthias Saou <http://freshrpms.net/> 0.10.4-1
 - Update to 0.10.4 for F7.
 - Disable swfdec... does anything/anyone even use it here? Once it stabilizes
   somewhat more, maybe then it'll be worth re-enabling.