Skip to contentSkip to conversation

faad2: security bump to version 2.8.1

PATCH New Baruch Siach 3 participants 5 messages Aug 9, 2017 – Sep 5, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 faad2: security bump to version 2.8.1

Fixes: CVE-2017-9218, CVE-2017-9219, CVE-2017-9220, CVE-2017-9221, CVE-2017-9222, CVE-2017-9223, CVE-2017-9253, CVE-2017-9254, CVE-2017-9255, CVE-2017-9256, CVE-2017-9257

http://seclists.org/fulldisclosure/2017/Jun/32

Switch to .tar.bz2 to save some bandwidth.

Add autoreconf since unfortunately upstream tarball does not ship the configure script.

Cc Gustavo Zacarias <[email protected]>
---
 package/faad2/faad2.hash | 6 +++---
 package/faad2/faad2.mk   | 8 ++++++--
 2 files changed, 9 insertions(+), 5 deletions(-)
package/faad2/faad2.hash +3 -3
index 691645b7c07e..d298e909a97c 100644
@@ -1,4 +1,4 @@
-# From http://sourceforge.net/projects/faac/files/faad2-src/faad2-2.7/ (used by upstream):
-sha1	80eaaa5cc576c35dd28863767b795c50cbcc0511  faad2-2.7.tar.gz
+# From http://sourceforge.net/projects/faac/files/faad2-src/faad2-2.8.0/ (used by upstream):
+sha1	a5caa71cd915acd502d96cba56f38296277f2350  faad2-2.8.1.tar.bz2
 # Locally computed
-sha256  ee26ed1e177c0cd8fa8458a481b14a0b24ca0b51468c8b4c8b676fd3ceccd330  faad2-2.7.tar.gz
+sha256  f4042496f6b0a60f5ded6acd11093230044ef8a2fd965360c1bbd5b58780933d  faad2-2.8.1.tar.bz2
package/faad2/faad2.mk +6 -2
index d7b55d3efaef..fa965fe5909c 100644
@@ -4,10 +4,14 @@
 #
 ################################################################################
 
-FAAD2_VERSION = 2.7
-FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION)
+FAAD2_VERSION_MAJOR = 2.8
+FAAD2_VERSION = $(FAAD2_VERSION_MAJOR).1
+FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION_MAJOR).0
+FAAD2_SOURCE = faad2-$(FAAD2_VERSION).tar.bz2
 FAAD2_LICENSE = GPL-2.0
 FAAD2_LICENSE_FILES = COPYING
+# No configure script in upstream tarball
+FAAD2_AUTORECONF = YES
 # frontend/faad calls frexp()
 FAAD2_CONF_ENV = LIBS=-lm
 FAAD2_INSTALL_STAGING = YES
Signature · 1 line
2.13.2
#1 lore raw

PATCH 1/1 faad2: security bump to version 2.8.1

Hi Baruch,

Quoted 5 lines · Baruch Siach

Fixes: CVE-2017-9218, CVE-2017-9219, CVE-2017-9220, CVE-2017-9221, CVE-2017-9222, CVE-2017-9223, CVE-2017-9253, CVE-2017-9254, CVE-2017-9255, CVE-2017-9256, CVE-2017-9257

http://seclists.org/fulldisclosure/2017/Jun/32

[snip]

package/faad2/faad2.mk line 8

Quoted 4 lines
package/faad2/faad2.mk +2 -2
-FAAD2_VERSION = 2.7
-FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION)
+FAAD2_VERSION_MAJOR = 2.8
+FAAD2_VERSION = $(FAAD2_VERSION_MAJOR).1

Hm, "security bumps" are typically only affecting the minor version number,
this smells like a major bump... Or does faad have a slightly unconventional version numbering scheme?

package/faad2/faad2.mk line 10

Quoted 2 lines
package/faad2/faad2.mk +2 -0
+FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION_MAJOR).0
+FAAD2_SOURCE = faad2-$(FAAD2_VERSION).tar.bz2

Gah, what kind of stupid download URL is that :-)

Regards,
Arnout

Quoted 8 lines

FAAD2_LICENSE = GPL-2.0
FAAD2_LICENSE_FILES = COPYING

package/faad2/faad2.mk +2 -0
+# No configure script in upstream tarball
+FAAD2_AUTORECONF = YES
 # frontend/faad calls frexp()
 FAAD2_CONF_ENV = LIBS=-lm
 FAAD2_INSTALL_STAGING = YES
Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
#2 lore raw

PATCH 1/1 faad2: security bump to version 2.8.1

Hi Arnout,

Quoted 7 lines · Arnout Vandecappelle
Quoted 5 lines · Baruch Siach

Fixes: CVE-2017-9218, CVE-2017-9219, CVE-2017-9220, CVE-2017-9221, CVE-2017-9222, CVE-2017-9223, CVE-2017-9253, CVE-2017-9254, CVE-2017-9255, CVE-2017-9256, CVE-2017-9257

http://seclists.org/fulldisclosure/2017/Jun/32

[snip]

Quoted 8 lines

package/faad2/faad2.mk line 8

Quoted 4 lines
package/faad2/faad2.mk +2 -2
-FAAD2_VERSION = 2.7
-FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION)
+FAAD2_VERSION_MAJOR = 2.8
+FAAD2_VERSION = $(FAAD2_VERSION_MAJOR).1

Hm, "security bumps" are typically only affecting the minor version number,
this smells like a major bump... Or does faad have a slightly unconventional version numbering scheme?

It's only called _MAJOR here because I reuse that in the URL, in line with the DRY principle.

Although version 2.8.0 (followed by 2.8.1 a week later) is the first release since February 2009, it does not contain a lot of code changes. I guess that the disclosed security issue were the main motivation of the release at this point.

Quoted 4 lines

package/faad2/faad2.mk line 10

Quoted 2 lines
package/faad2/faad2.mk +2 -0
+FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION_MAJOR).0
+FAAD2_SOURCE = faad2-$(FAAD2_VERSION).tar.bz2

Gah, what kind of stupid download URL is that :-)

Well, that's upstream.

Quoted 7 lines

package/faad2/faad2.mk line 17

Quoted 7 lines

FAAD2_LICENSE = GPL-2.0
FAAD2_LICENSE_FILES = COPYING

package/faad2/faad2.mk +2 -0
+# No configure script in upstream tarball
+FAAD2_AUTORECONF = YES
 # frontend/faad calls frexp()
 FAAD2_CONF_ENV = LIBS=-lm
 FAAD2_INSTALL_STAGING = YES

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
#3 lore raw

PATCH 1/1 faad2: security bump to version 2.8.1

Quoted 19 lines · Baruch Siach

Hi Arnout,

Quoted 7 lines · Arnout Vandecappelle
Quoted 5 lines · Baruch Siach

Fixes: CVE-2017-9218, CVE-2017-9219, CVE-2017-9220, CVE-2017-9221, CVE-2017-9222, CVE-2017-9223, CVE-2017-9253, CVE-2017-9254, CVE-2017-9255, CVE-2017-9256, CVE-2017-9257

http://seclists.org/fulldisclosure/2017/Jun/32

[snip]

Quoted 8 lines

package/faad2/faad2.mk line 8

Quoted 4 lines
package/faad2/faad2.mk +2 -2
-FAAD2_VERSION = 2.7
-FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION)
+FAAD2_VERSION_MAJOR = 2.8
+FAAD2_VERSION = $(FAAD2_VERSION_MAJOR).1

Hm, "security bumps" are typically only affecting the minor version number,
this smells like a major bump... Or does faad have a slightly unconventional version numbering scheme?

Quoted 7 lines

It's only called _MAJOR here because I reuse that in the URL, in line with the DRY principle.

Although version 2.8.0 (followed by 2.8.1 a week later) is the first release since February 2009, it does not contain a lot of code changes. I guess that the disclosed security issue were the main motivation of the release at this point.

OK, applied to master, thanks.

Regards,
Arnout

Quoted 18 lines
Quoted 5 lines

package/faad2/faad2.mk line 10

Quoted 3 lines
package/faad2/faad2.mk +2 -0
+FAAD2_SITE = http://downloads.sourceforge.net/project/faac/faad2-src/faad2-$(FAAD2_VERSION_MAJOR).0
+FAAD2_SOURCE = faad2-$(FAAD2_VERSION).tar.bz2

Gah, what kind of stupid download URL is that :-)

Well, that's upstream.

Quoted 7 lines

package/faad2/faad2.mk line 17

Quoted 7 lines

FAAD2_LICENSE = GPL-2.0
FAAD2_LICENSE_FILES = COPYING

package/faad2/faad2.mk +2 -0
+# No configure script in upstream tarball
+FAAD2_AUTORECONF = YES
 # frontend/faad calls frexp()
 FAAD2_CONF_ENV = LIBS=-lm
 FAAD2_INSTALL_STAGING = YES

baruch

Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
#4 lore raw

PATCH 1/1 faad2: security bump to version 2.8.1

In reply to Baruch Siach
Quoted 8 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Fixes: CVE-2017-9218, CVE-2017-9219, CVE-2017-9220, CVE-2017-9221, CVE-2017-9222, CVE-2017-9223, CVE-2017-9253, CVE-2017-9254, CVE-2017-9255, CVE-2017-9256, CVE-2017-9257

http://seclists.org/fulldisclosure/2017/Jun/32

Quoted 7 lines

Switch to .tar.bz2 to save some bandwidth.

Add autoreconf since unfortunately upstream tarball does not ship the configure script.

Cc Gustavo Zacarias <[email protected]>

Committed to 2017.02.x, thanks.

Signature · 1 line
Bye, Peter Korsgaard