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
  • !4880

i#4878: Fix whereami overflow corruption of signal field

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i4878-fix-field-overflow into master Apr 27, 2021
  • Overview 1
  • Commits 1
  • Pipelines 0
  • Changes 1

Fixes stores to dcontext_t.whereami on AArch64 where a 64-bit register is written to a 32-bit field, overflowing it and clobbering the adjacent signals_pending field. This causes signals to not be delivered and fragments that were unlinked to not be relinked, raising an assert.

Tested on the proprietary application that reproduced the signal assert in every run: it is gone with this fix.

Fixes #4878 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i4878-fix-field-overflow