Skip to content

update runc binary and vendor to v1.1.2#6934

Merged
AkihiroSuda merged 2 commits intocontainerd:mainfrom
thaJeztah:bump_runc
May 12, 2022
Merged

update runc binary and vendor to v1.1.2#6934
AkihiroSuda merged 2 commits intocontainerd:mainfrom
thaJeztah:bump_runc

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This is the second patch release of the runc 1.1 release branch. It
fixes CVE-2022-29162, a minor security issue (which appears to not be
exploitable) related to process capabilities.

This is a similar bug to the ones found and fixed in Docker and
containerd recently (CVE-2022-24769).

  • A bug was found in runc where runc exec --cap executed processes with
    non-empty inheritable Linux process capabilities, creating an atypical Linux
    environment. For more information, see GHSA-f3fp-gc8g-vw66 and CVE-2022-29162.
  • runc spec no longer sets any inheritable capabilities in the created
    example OCI spec (config.json) file.

thaJeztah added 2 commits May 12, 2022 13:38
This is the second patch release of the runc 1.1 release branch. It
fixes CVE-2022-29162, a minor security issue (which appears to not be
exploitable) related to process capabilities.

This is a similar bug to the ones found and fixed in Docker and
containerd recently (CVE-2022-24769).

- A bug was found in runc where runc exec --cap executed processes with
  non-empty inheritable Linux process capabilities, creating an atypical Linux
  environment. For more information, see GHSA-f3fp-gc8g-vw66 and CVE-2022-29162.
- runc spec no longer sets any inheritable capabilities in the created
  example OCI spec (config.json) file.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
no changes in vendored code

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

Hmmm... fun... another vanity URL down?

go: github.com/containerd/[email protected] requires
	go.opentelemetry.io/otel/exporters/otlp/internal/[email protected]: unrecognized import path "go.opentelemetry.io/otel/exporters/otlp/internal/retry": https fetch: Get "https://go.opentelemetry.io/otel/exporters/otlp/internal/retry?go-get=1": read tcp 10.1.0.15:44416->216.239.32.21:443: read: connection reset by peer
make: *** [integration] Error 1
Makefile:203: recipe for target 'integration' failed
Error: Process completed with exit code 2.

@thaJeztah
Copy link
Copy Markdown
Member Author

curl -v 'https://go.opentelemetry.io/otel/exporters/otlp/internal/retry?go-get=1'
*   Trying 216.239.32.21:443...
* Connected to go.opentelemetry.io (216.239.32.21) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=go.opentelemetry.io
*  start date: Mar 25 19:42:58 2022 GMT
*  expire date: Jun 23 19:42:57 2022 GMT
*  subjectAltName: host "go.opentelemetry.io" matched cert's "go.opentelemetry.io"
*  issuer: C=US; O=Google Trust Services LLC; CN=GTS CA 1D4
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x7f8ed780a600)
> GET /otel/exporters/otlp/internal/retry?go-get=1 HTTP/2
> Host: go.opentelemetry.io
> user-agent: curl/7.79.1
> accept: */*
>
< HTTP/2 200
< content-type: text/html; charset=utf-8
< vary: Accept-Encoding
< cache-control: public, max-age=86400
< x-cloud-trace-context: 1b899b1203f005b031200201b0e4659d;o=1
< date: Thu, 12 May 2022 12:36:46 GMT
< server: Google Frontend
< content-length: 765
<
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
<meta name="go-import" content="go.opentelemetry.io/otel git https://github.com/open-telemetry/opentelemetry-go">
<meta name="go-source" content="go.opentelemetry.io/otel https://github.com/open-telemetry/opentelemetry-go https://github.com/open-telemetry/opentelemetry-go/tree/master{/dir} https://github.com/open-telemetry/opentelemetry-go/blob/master{/dir}/{file}#L{line}">
<meta http-equiv="refresh" content="0; url=https://pkg.go.dev/go.opentelemetry.io/otel/exporters/otlp/internal/retry">
</head>
<body>
Nothing to see here; <a href="https://pkg.go.dev/go.opentelemetry.io/otel/exporters/otlp/internal/retry">see the package on pkg.go.dev</a>.
</body>
* Connection #0 to host go.opentelemetry.io left intact
</html>⏎

@thaJeztah
Copy link
Copy Markdown
Member Author

Or is this a circular import issue?

@estesp
Copy link
Copy Markdown
Member

estesp commented May 12, 2022

Or is this a circular import issue?

I think it's just a flaky HTTP connection; the "connection reset by peer" happens more regularly than I would like to see with git clones, go mod activity, etc. during CI. I restarted just that test to see if it was just a flaky HTTP connection.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Copy Markdown
Member Author

Thanks! Looks green now; for a bit I wondered if it was a misconfigured redirect (from the curl output) 😅

@AkihiroSuda AkihiroSuda merged commit 14af2bd into containerd:main May 12, 2022
@thaJeztah thaJeztah deleted the bump_runc branch May 12, 2022 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants