diff mbox series

[v3,1/6] package/libupnp: security bump to version 1.14.0

Message ID 20210125194949.1173139-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [v3,1/6] package/libupnp: security bump to version 1.14.0 | expand

Commit Message

Fabrice Fontaine Jan. 25, 2021, 7:49 p.m. UTC
- Fix CallStranger a.k.a. CVE-2020-12695 as well as CVE-2020-13848
- Update indentation in hash file (two spaces)
- Backport all changes from libupnp18 to libupnp:
  - Use COPYING instead of LICENSE (no license change)
  - Add host-pkgconf dependency
  - Add --enable-reuseaddr
  - Add openssl optional dependency

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
Changes v2 -> v3:
 - Rebase on current master

Changes v1 -> v2:
 - Bump libupnp instead of libupnp18 and drop libupnp18
 - Update ushare and igd2-for-linux
 - Drop libupnp18

 package/libupnp/libupnp.hash |  4 ++--
 package/libupnp/libupnp.mk   | 18 +++++++++++++++---
 2 files changed, 17 insertions(+), 5 deletions(-)

Comments

Yann E. MORIN Jan. 25, 2021, 9:26 p.m. UTC | #1
Fabrice, All,

On 2021-01-25 20:49 +0100, Fabrice Fontaine spake thusly:
> - Fix CallStranger a.k.a. CVE-2020-12695 as well as CVE-2020-13848
> - Update indentation in hash file (two spaces)
> - Backport all changes from libupnp18 to libupnp:
>   - Use COPYING instead of LICENSE (no license change)
>   - Add host-pkgconf dependency
>   - Add --enable-reuseaddr
>   - Add openssl optional dependency
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Series of 6 applied to master, thanks!

I'll further reply to some of those for additional details, but
otherwise: great job, thanks a lot!

Regards,
Yann E. MORIN.

> ---
> Changes v2 -> v3:
>  - Rebase on current master
> 
> Changes v1 -> v2:
>  - Bump libupnp instead of libupnp18 and drop libupnp18
>  - Update ushare and igd2-for-linux
>  - Drop libupnp18
> 
>  package/libupnp/libupnp.hash |  4 ++--
>  package/libupnp/libupnp.mk   | 18 +++++++++++++++---
>  2 files changed, 17 insertions(+), 5 deletions(-)
> 
> diff --git a/package/libupnp/libupnp.hash b/package/libupnp/libupnp.hash
> index e52b7ea9d7..6b16eff3c8 100644
> --- a/package/libupnp/libupnp.hash
> +++ b/package/libupnp/libupnp.hash
> @@ -1,3 +1,3 @@
>  # Locally computed:
> -sha256	c5a300b86775435c076d58a79cc0d5a977d76027d2a7d721590729b7f369fa43	libupnp-1.6.25.tar.bz2
> -sha256	0375955c8a79d6e8fa0792d45d00fc4e7710d7ac95bcbd27f9225a83f5c946fd	LICENSE
> +sha256  ecb23d4291968c8a7bdd4eb16fc2250dbacc16b354345a13342d67f571d35ceb  libupnp-1.14.0.tar.bz2
> +sha256  c8b99423cad48bb44e2cf52a496361404290865eac259a82da6d1e4331ececb3  COPYING
> diff --git a/package/libupnp/libupnp.mk b/package/libupnp/libupnp.mk
> index b7836590c2..ebc5e83765 100644
> --- a/package/libupnp/libupnp.mk
> +++ b/package/libupnp/libupnp.mk
> @@ -4,13 +4,25 @@
>  #
>  ################################################################################
>  
> -LIBUPNP_VERSION = 1.6.25
> +LIBUPNP_VERSION = 1.14.0
>  LIBUPNP_SOURCE = libupnp-$(LIBUPNP_VERSION).tar.bz2
> -LIBUPNP_SITE = http://downloads.sourceforge.net/project/pupnp/pupnp/libUPnP%20$(LIBUPNP_VERSION)
> +LIBUPNP_SITE = \
> +	http://downloads.sourceforge.net/project/pupnp/pupnp/libupnp-$(LIBUPNP_VERSION)
>  LIBUPNP_CONF_ENV = ac_cv_lib_compat_ftime=no
>  LIBUPNP_INSTALL_STAGING = YES
>  LIBUPNP_LICENSE = BSD-3-Clause
> -LIBUPNP_LICENSE_FILES = LICENSE
> +LIBUPNP_LICENSE_FILES = COPYING
>  LIBUPNP_CPE_ID_VALID = YES
> +LIBUPNP_DEPENDENCIES = host-pkgconf
> +
> +# Bind the internal miniserver socket with reuseaddr to allow clean restarts.
> +LIBUPNP_CONF_OPTS += --enable-reuseaddr
> +
> +ifeq ($(BR2_PACKAGE_OPENSSL),y)
> +LIBUPNP_CONF_OPTS += --enable-open-ssl
> +LIBUPNP_DEPENDENCIES += openssl
> +else
> +LIBUPNP_CONF_OPTS += --disable-open-ssl
> +endif
>  
>  $(eval $(autotools-package))
> -- 
> 2.29.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Jan. 28, 2021, 7:13 p.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > - Fix CallStranger a.k.a. CVE-2020-12695 as well as CVE-2020-13848
 > - Update indentation in hash file (two spaces)
 > - Backport all changes from libupnp18 to libupnp:
 >   - Use COPYING instead of LICENSE (no license change)
 >   - Add host-pkgconf dependency
 >   - Add --enable-reuseaddr
 >   - Add openssl optional dependency

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
 > ---
 > Changes v2 -> v3:
 >  - Rebase on current master

Committed to 2020.02.x and 2020.11.x, thanks.
Peter Korsgaard Jan. 29, 2021, 8:09 a.m. UTC | #3
>>>>> "Peter" == Peter Korsgaard <peter@korsgaard.com> writes:

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
 >> - Fix CallStranger a.k.a. CVE-2020-12695 as well as CVE-2020-13848
 >> - Update indentation in hash file (two spaces)
 >> - Backport all changes from libupnp18 to libupnp:
 >> - Use COPYING instead of LICENSE (no license change)
 >> - Add host-pkgconf dependency
 >> - Add --enable-reuseaddr
 >> - Add openssl optional dependency

 >> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
 >> ---
 >> Changes v2 -> v3:
 >> - Rebase on current master

 > Committed to 2020.02.x and 2020.11.x, thanks.

This unfortunately breaks the old linphone stack on 2020.02.x, so I will
bump bctoolbox/ortp/mediastreamer/linphone.
Fabrice Fontaine Jan. 30, 2021, 9:28 a.m. UTC | #4
Le ven. 29 janv. 2021 à 09:09, Peter Korsgaard <peter@korsgaard.com> a écrit :
>
> >>>>> "Peter" == Peter Korsgaard <peter@korsgaard.com> writes:
>
> >>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
>  >> - Fix CallStranger a.k.a. CVE-2020-12695 as well as CVE-2020-13848
>  >> - Update indentation in hash file (two spaces)
>  >> - Backport all changes from libupnp18 to libupnp:
>  >> - Use COPYING instead of LICENSE (no license change)
>  >> - Add host-pkgconf dependency
>  >> - Add --enable-reuseaddr
>  >> - Add openssl optional dependency
>
>  >> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>  >> ---
>  >> Changes v2 -> v3:
>  >> - Rebase on current master
>
>  > Committed to 2020.02.x and 2020.11.x, thanks.
>
> This unfortunately breaks the old linphone stack on 2020.02.x, so I will
> bump bctoolbox/ortp/mediastreamer/linphone.
linphone also needs belle-sip and belr, I'll send a patch to add them
in 2020.02.x.
>
> --
> Bye, Peter Korsgaard
Best Regards,

Fabrice
Peter Korsgaard Jan. 30, 2021, 9:37 a.m. UTC | #5
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

Hi,

 >> > Committed to 2020.02.x and 2020.11.x, thanks.
 >> 
 >> This unfortunately breaks the old linphone stack on 2020.02.x, so I will
 >> bump bctoolbox/ortp/mediastreamer/linphone.
 > linphone also needs belle-sip and belr, I'll send a patch to add them
 > in 2020.02.x.

Ahh, thanks.
diff mbox series

Patch

diff --git a/package/libupnp/libupnp.hash b/package/libupnp/libupnp.hash
index e52b7ea9d7..6b16eff3c8 100644
--- a/package/libupnp/libupnp.hash
+++ b/package/libupnp/libupnp.hash
@@ -1,3 +1,3 @@ 
 # Locally computed:
-sha256	c5a300b86775435c076d58a79cc0d5a977d76027d2a7d721590729b7f369fa43	libupnp-1.6.25.tar.bz2
-sha256	0375955c8a79d6e8fa0792d45d00fc4e7710d7ac95bcbd27f9225a83f5c946fd	LICENSE
+sha256  ecb23d4291968c8a7bdd4eb16fc2250dbacc16b354345a13342d67f571d35ceb  libupnp-1.14.0.tar.bz2
+sha256  c8b99423cad48bb44e2cf52a496361404290865eac259a82da6d1e4331ececb3  COPYING
diff --git a/package/libupnp/libupnp.mk b/package/libupnp/libupnp.mk
index b7836590c2..ebc5e83765 100644
--- a/package/libupnp/libupnp.mk
+++ b/package/libupnp/libupnp.mk
@@ -4,13 +4,25 @@ 
 #
 ################################################################################
 
-LIBUPNP_VERSION = 1.6.25
+LIBUPNP_VERSION = 1.14.0
 LIBUPNP_SOURCE = libupnp-$(LIBUPNP_VERSION).tar.bz2
-LIBUPNP_SITE = http://downloads.sourceforge.net/project/pupnp/pupnp/libUPnP%20$(LIBUPNP_VERSION)
+LIBUPNP_SITE = \
+	http://downloads.sourceforge.net/project/pupnp/pupnp/libupnp-$(LIBUPNP_VERSION)
 LIBUPNP_CONF_ENV = ac_cv_lib_compat_ftime=no
 LIBUPNP_INSTALL_STAGING = YES
 LIBUPNP_LICENSE = BSD-3-Clause
-LIBUPNP_LICENSE_FILES = LICENSE
+LIBUPNP_LICENSE_FILES = COPYING
 LIBUPNP_CPE_ID_VALID = YES
+LIBUPNP_DEPENDENCIES = host-pkgconf
+
+# Bind the internal miniserver socket with reuseaddr to allow clean restarts.
+LIBUPNP_CONF_OPTS += --enable-reuseaddr
+
+ifeq ($(BR2_PACKAGE_OPENSSL),y)
+LIBUPNP_CONF_OPTS += --enable-open-ssl
+LIBUPNP_DEPENDENCIES += openssl
+else
+LIBUPNP_CONF_OPTS += --disable-open-ssl
+endif
 
 $(eval $(autotools-package))