Bug 653066 (CVE-2018-1000168) – <net-libs/nghttp2-1.31.1: Denial of service due to NULL pointer dereference (CVE-2018-1000168)

• Product: Gentoo Security
• Status: RESOLVED FIXED

[Description GLSAMaker/CVETool Bot 2018-04-12 21:22:00 UTC]

CVE-2018-1000168 (https://nvd.nist.gov/vuln/detail/CVE-2018-1000168):
  ** RESERVED ** This candidate has been reserved by an organization or
  individual that will use it when announcing a new security problem. When the
  candidate has been publicized, the details for this candidate will be
  provided.


If ALTSVC frame is received by libnghttp2 and it is larger than it can accept, the pointer field which points to ALTSVC frame payload is left NULL. Later libnghttp2 attempts to access another field through the pointer, and gets segmentation fault.

ALTSVC frame is defined by RFC 7838.

The largest frame size libnghttp2 accept is by default 16384 bytes.

Receiving ALTSVC frame is disabled by default. Application has to enable it explicitly by calling nghttp2_option_set_builtin_recv_extension_type(opt, NGHTTP2_ALTSVC).

Transmission of ALTSVC is always enabled, and it does not cause this vulnerability.

ALTSVC frame is expected to be sent by server, and received by client as defined in RFC 7838.

Client and server are both affected by this vulnerability if the reception of ALTSVC frame is enabled. As written earlier, it is useless to enable reception of ALTSVC frame on server side. So, server is generally safe unless application accidentally enabled the reception of ALTSVC frame.

Affected Versions

    Affected versions: nghttp2 >= 1.10.0 and nghttp2 <= v1.31.0
    Not affected versions: nghttp2 >= 1.31.1

[Comment 1 Lars Wendler (Polynomial-C) (RETIRED) 2018-04-13 05:40:49 UTC]

commit 88c85f32a68ff188cb9f815ea67112a1d1a8e476
Author: Jeroen Roovers <[e-mail]>
Date:   Fri Apr 13 06:23:20 2018

    net-libs/nghttp2: Version 1.31.1.

    Package-Manager: Portage-2.3.28, Repoman-2.3.9

[Comment 2 Larry the Git Cow 2018-04-13 20:08:04 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9bd6daf026ab5608fa5da75e2da0e75fa4174735

commit 9bd6daf026ab5608fa5da75e2da0e75fa4174735
Author:     Rolf Eike Beer <[e-mail]>
AuthorDate: 2018-04-13 19:57:33 +0000
Commit:     Sergei Trofimovich <[e-mail]>
CommitDate: 2018-04-13 20:07:41 +0000

    net-libs/nghttp2: stable 1.31.1 for sparc
    
    Bug: https://bugs.gentoo.org/653066
    Package-Manager: Portage-2.3.24, Repoman-2.3.6
    RepoMan-Options: --include-arches="sparc"

 net-libs/nghttp2/nghttp2-1.31.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)}

[Comment 3 Larry the Git Cow 2018-04-14 14:54:26 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9f283f1e604a0a52460b47e393b16ec093f8843c

commit 9f283f1e604a0a52460b47e393b16ec093f8843c
Author:     Aaron Bauman <[e-mail]>
AuthorDate: 2018-04-14 14:52:29 +0000
Commit:     Aaron Bauman <[e-mail]>
CommitDate: 2018-04-14 14:54:15 +0000

    net-libs/nghttp2: amd64 stable
    
    Bug: https://bugs.gentoo.org/653066
    Package-Manager: Portage-2.3.28, Repoman-2.3.9

 net-libs/nghttp2/nghttp2-1.31.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)}

[Comment 4 Mart Raudsepp 2018-04-14 20:24:48 UTC]

arm64 stable

[Comment 5 Thomas Deutschmann (RETIRED) 2018-04-15 18:59:05 UTC]

x86 stable

[Comment 6 Larry the Git Cow 2018-04-18 21:39:12 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bfe84eb0195242e899a8c9216b08acf7b860e9a3

commit bfe84eb0195242e899a8c9216b08acf7b860e9a3
Author:     Sergei Trofimovich <[e-mail]>
AuthorDate: 2018-04-18 21:38:21 +0000
Commit:     Sergei Trofimovich <[e-mail]>
CommitDate: 2018-04-18 21:38:49 +0000

    net-libs/nghttp2: stable 1.31.1 for ia64, bug #653066
    
    Bug: https://bugs.gentoo.org/653066
    Package-Manager: Portage-2.3.28, Repoman-2.3.9
    RepoMan-Options: --include-arches="ia64"

 net-libs/nghttp2/nghttp2-1.31.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)}

[Comment 7 Matt Turner 2018-04-22 20:29:34 UTC]

alpha stable

[Comment 8 Mikle Kolyada 2018-04-26 09:49:17 UTC]

arm stable

[Comment 9 Larry the Git Cow 2018-05-11 22:43:18 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2c59d7bba8ae474c62450bf174a10151bd0ab931

commit 2c59d7bba8ae474c62450bf174a10151bd0ab931
Author:     Sergei Trofimovich <[e-mail]>
AuthorDate: 2018-05-11 22:41:39 +0000
Commit:     Sergei Trofimovich <[e-mail]>
CommitDate: 2018-05-11 22:43:06 +0000

    net-libs/nghttp2: stable 1.31.1 for ppc64, bug #653066
    
    Bug: https://bugs.gentoo.org/653066
    Package-Manager: Portage-2.3.36, Repoman-2.3.9
    RepoMan-Options: --include-arches="ppc64"

 net-libs/nghttp2/nghttp2-1.31.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

[Comment 10 Larry the Git Cow 2018-07-15 10:25:10 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c299b80835911d59d860d4e07dd82745768d0c47

commit c299b80835911d59d860d4e07dd82745768d0c47
Author:     Sergei Trofimovich <[e-mail]>
AuthorDate: 2018-07-15 10:00:10 +0000
Commit:     Sergei Trofimovich <[e-mail]>
CommitDate: 2018-07-15 10:24:31 +0000

    net-libs/nghttp2: stable 1.31.1 for ppc, bug #653066
    
    Bug: https://bugs.gentoo.org/653066
    Package-Manager: Portage-2.3.42, Repoman-2.3.9
    RepoMan-Options: --include-arches="ppc"

 net-libs/nghttp2/nghttp2-1.31.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

[Comment 11 Lars Wendler (Polynomial-C) (RETIRED) 2018-10-26 09:13:34 UTC]

Security cleanup has already been performed.

[Comment 12 Aaron Bauman (RETIRED) 2018-11-24 22:34:39 UTC]

tree is clean