Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

[18.09 backport] Fix nil pointer derefence on failure to connect to containerd #162

Conversation

thaJeztah
Copy link
Member

backport of moby#38653 for 18.09

fixes moby#38636
addresses docker/for-linux#517 (comment)

Signed-off-by: Simão Reis <smnrsti@gmail.com>
(cherry picked from commit 3134161)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added this to the 18.09.3 milestone Feb 22, 2019
@thaJeztah
Copy link
Member Author

ping @dmcgowan @tonistiigi @tiborvass PTAL

@tiborvass tiborvass merged commit 24c6c3e into docker-archive:18.09 Feb 22, 2019
@thaJeztah thaJeztah deleted the 18.09_backport_38636_fix_nil_pointer_dereference branch February 22, 2019 21:40
algitbot pushed a commit to alpinelinux/aports that referenced this pull request Mar 12, 2019
https://github.com/docker/docker-ce/releases/tag/v18.09.3

The more important fixes in this version:
* When copying existing folder, ignore xattr set errors when the target filesystem doesn't support xattr. docker-archive/engine#135
* Graphdriver: fix device mode not being detected if character-device bit is set. docker-archive/engine#160
* Fix nil pointer derefence on failure to connect to containerd. docker-archive/engine#162
* Delete stale containerd object on start failure. docker-archive/engine#154
liske pushed a commit to liske/aports that referenced this pull request Apr 7, 2019
https://github.com/docker/docker-ce/releases/tag/v18.09.3

The more important fixes in this version:
* When copying existing folder, ignore xattr set errors when the target filesystem doesn't support xattr. docker-archive/engine#135
* Graphdriver: fix device mode not being detected if character-device bit is set. docker-archive/engine#160
* Fix nil pointer derefence on failure to connect to containerd. docker-archive/engine#162
* Delete stale containerd object on start failure. docker-archive/engine#154
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants