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

i#1729 offline traces: fix initial tid entry errors

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i1729-initial-tid into master Jun 01, 2017
  • Overview 1
  • Commits 4
  • Pipelines 0
  • Changes 5

Fixes two issues with offline traces (#1729 (closed)): first, changes the reader to not assume a separate tid header right after the initial tid,pid header, as offline traces (as opposed to online) do not contain it. Second, changes raw2trace to not emit a superfluous tid entry of 0 just prior to the real initial tid entry for each thread.

Adds asserts to catch related errors in the future in the test suite.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i1729-initial-tid