New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 811049 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Security



Sign in to add a comment

CrOS: Vulnerability reported in net-misc/curl

Project Member Reported by vomit.go...@appspot.gserviceaccount.com, Feb 10 2018

Issue description

Automated analysis has detected that the following third party packages have had vulnerabilities publicly reported. 

NOTE: There may be several bugs listed below - in almost all cases, all bugs can be quickly addressed by upgrading to the latest version of the package.

Package Name: net-misc/curl
Package Version: [cpe:/a:curl:curl:7.51.0 cpe:/a:curl:libcurl:7.51.0 cpe:/a:haxx:curl:7.51.0 cpe:/a:haxx:libcurl:7.51.0]

Advisory: CVE-2018-1000005
  Details: https://vomit.googleplex.com/advisory?id=CVE/CVE-2018-1000005
  CVSS severity score: 6.4/10.0
  Confidence: high
  Description:

libcurl 7.49.0 to and including 7.57.0 contains an out bounds read in code handling HTTP/2 trailers. It was reported (https://github.com/curl/curl/pull/2231) that reading an HTTP/2 trailer could mess up future trailers since the stored size was one byte less than required. The problem is that the code that creates HTTP/1-like headers from the HTTP/2 trailer data once appended a string like `:` to the target buffer, while this was recently changed to `: ` (a space was added after the colon) but the following math wasn't updated correspondingly. When accessed, the data is read out of bounds and causes either a crash or that the (too large) data gets passed to client write. This could lead to a denial-of-service situation or an information disclosure if someone has a service that echoes back or uses the trailers for something.



 
Labels: Security_Severity-Low
Owner: briannorris@chromium.org
Status: Assigned (was: Untriaged)
It is probably worth up-reving curl over this, but it is primarily a DoS risk.
Cc: vapier@chromium.org cernekee@chromium.org kirtika@chromium.org benchan@chromium.org kirtika@google.com
Components: OS>Packages OS>Systems>Network
Status: Started (was: Assigned)
https://chromium-review.googlesource.com/c/chromiumos/overlays/portage-stable/+/914810
Status: Fixed (was: Started)
AIUI, this isn't urgent, so probably doesn't need ported to any branches? Closing.
Project Member

Comment 5 by sheriffbot@chromium.org, Feb 14 2018

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 6 by bugdroid1@chromium.org, Feb 20 2018

Labels: merge-merged-release-R65-10323.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/8ef973467f578205d21903014a98fe0d714875b7

commit 8ef973467f578205d21903014a98fe0d714875b7
Author: Brian Norris <briannorris@chromium.org>
Date: Tue Feb 20 23:30:18 2018

curl: upgraded package to upstream

Upgraded net-misc/curl to version 7.58.0 on amd64, arm

Changed EAPI to 5, based on CL:919534

BUG= chromium:811049 ,b:73183479
TEST=precq; local tests

Change-Id: I4fdf70cdb47e4bb65aabf3993d77958413c21cf8
Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/914810
Reviewed-by: Mike Frysinger <vapier@chromium.org>
(cherry picked from commit 63aa5b34f725b2291618c074ebb8d30da2ba957f)
Reviewed-on: https://chromium-review.googlesource.com/927142
Commit-Queue: Daniel Wang <wonderfly@google.com>
Tested-by: Daniel Wang <wonderfly@google.com>

[modify] https://crrev.com/8ef973467f578205d21903014a98fe0d714875b7/net-misc/curl/Manifest
[modify] https://crrev.com/8ef973467f578205d21903014a98fe0d714875b7/net-misc/curl/metadata.xml
[add] https://crrev.com/8ef973467f578205d21903014a98fe0d714875b7/metadata/md5-cache/net-misc/curl-7.58.0
[rename] https://crrev.com/8ef973467f578205d21903014a98fe0d714875b7/net-misc/curl/curl-7.58.0.ebuild
[delete] https://crrev.com/6505549ac10878b487c95e83ca8e6f097a5ae83d/metadata/md5-cache/net-misc/curl-7.57.0
[delete] https://crrev.com/6505549ac10878b487c95e83ca8e6f097a5ae83d/net-misc/curl/files/curl-7.55.1-fix-build.patch

Project Member

Comment 7 by sheriffbot@chromium.org, May 23 2018

Labels: -Restrict-View-SecurityNotify allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment