Skip to content

[20.10 backport] pkg/signal: ignore SIGURG on all platforms#42421

Merged
AkihiroSuda merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_more_ignore
May 26, 2021
Merged

[20.10 backport] pkg/signal: ignore SIGURG on all platforms#42421
AkihiroSuda merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_more_ignore

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport of #42415

follow-up to #42397
relates to docker/cli#3103

Other Unix platforms (e.g. Darwin) are also affected by the Go
runtime sending SIGURG.

This patch changes how we match the signal by just looking for the
"URG" name, which should handle any platform that has this signal
defined in the SignalMap.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Other Unix platforms (e.g. Darwin) are also affected by the Go
runtime sending SIGURG.

This patch changes how we match the signal by just looking for the
"URG" name, which should handle any platform that has this signal
defined in the SignalMap.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 05f520d)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 20.10.7 milestone May 25, 2021
@AkihiroSuda
AkihiroSuda marked this pull request as ready for review May 26, 2021 07:18
@thaJeztah

Copy link
Copy Markdown
Member Author

@cpuguy83 @tonistiigi ptal

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@AkihiroSuda
AkihiroSuda merged commit d844987 into moby:20.10 May 26, 2021
@thaJeztah
thaJeztah deleted the 20.10_backport_more_ignore branch May 26, 2021 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants