Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

thaJeztah
Copy link
Member

full diff: opencontainers/runc@6635b4f...2b18fe1

This includes an improved fix for CVE-2019-5736 to reduce the
increased memory-consumption introduced by the original patch,
RHEL 7.6 getting into a loop due to a kernel bug in those kernels,
and improve compatibility with older kernels.

changes included:

Signed-off-by: Sebastiaan van Stijn [email protected]

This includes an improved fix for CVE-2019-5736 to reduce the
increased memory-consumption introduced by the original patch,
RHEL 7.6 getting into a loop due to a kernel bug in those kernels,
and improve compatibility with older kernels.

changes included:

- opencontainers/runc#1973 Vendor opencontainers/runtime-spec 29686dbc
- opencontainers/runc#1978 Remove detection for scope properties, which have always been broken
- opencontainers/runc#1963 Vendor in go-criu and use it for CRIU's RPC definition
- opencontainers/runc#1995 exec: expose --preserve-fds
- opencontainers/runc#2000 fix preserve-fds flag may cause runc hang
- opencontainers/runc#1968 Create bind mount mountpoints during restore
- opencontainers/runc#1984 nsenter: cloned_binary: "memfd" cleanups

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

ping @kolyshkin ptal

@thaJeztah
Copy link
Member Author

Note that those changes are in the runc binary (not the vendored code), but the runc version guidelines in this repository describe that the binary should have the same version as is used in vendor.conf.

@codecov-io
Copy link

Codecov Report

Merging #3081 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3081   +/-   ##
=======================================
  Coverage   43.54%   43.54%           
=======================================
  Files         103      103           
  Lines       11015    11015           
=======================================
  Hits         4796     4796           
  Misses       5483     5483           
  Partials      736      736
Flag Coverage Δ
#linux 47.52% <ø> (ø) ⬆️
#windows 40.37% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa328df...b8d40b3. Read the comment docs.

Copy link
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

@crosbymichael
Copy link
Member

LGTM

@crosbymichael crosbymichael merged commit a15b6e2 into containerd:master Mar 7, 2019
@crosbymichael
Copy link
Member

Needs to be cherry picked to 1.2

@thaJeztah thaJeztah deleted the bump_runc branch March 7, 2019 21:12
@thaJeztah
Copy link
Member Author

yup, working on that (1.1 also uses the same version, so I'll pick it there as well)

@crosbymichael
Copy link
Member

ok, thanks

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.

5 participants