Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • D dynamorio
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,467
    • Issues 1,467
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 44
    • Merge requests 44
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • DynamoRIO
  • dynamorio
  • Merge requests
  • !5008

i#4998: Fix incorrectly nested signals

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i4998-nested-signals into master Jul 13, 2021
  • Overview 1
  • Commits 4
  • Pipelines 0
  • Changes 5

Fixes a bug where DR delivers a signal while an app is inside its signal handler for that same signal and has blocked such self-nesting. The fix is to undo some of the sigsuspend-handling changes that cause this regression by moving the sigsuspend state restore to actual delivery rather than at pending signal time.

Improves the linux.signest test to reproduce the original bug and verify the fix. Also adds a test case to ensure that signals are nested when not blocked.

Fixes #4998 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i4998-nested-signals