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

i#5304: Ignore reachability for instr_is_encoding_possible()

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i5304-enc-pos-reach into master Jan 28, 2022
  • Overview 4
  • Commits 2
  • Pipelines 0
  • Changes 2

On ARM, instr_is_encoding_possible() was checking reachabilty, yet it took no target encode address, causing it to fail for any but very small addresses. We remove the reachability checks to solve this.

The test for #5302 (closed) in a forthcoming PR adds a regression test for this.

Fixes #5304 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i5304-enc-pos-reach