Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version bump release #13

Open
wants to merge 1 commit into
base: 1p/release-0.9999
Choose a base branch
from

Conversation

complexspaces
Copy link
Collaborator

This is a rebase of #12, combined with a pregenerated build of the object files as required for a Git dependency of ring to work during Windows builds.

There are quite a few diffs in the object files this time (see more information on this here in the README) but after some analysis I believe they are expected and inert.

The tools I used for the analysis were:

  • delta (better git diff viewer)
  • Binary Ninja for looking at symbol names, exports, and the x86 instructions in each function.
  • ImHex for Hex/ASCII side-by-side diffing
  • ghidriff for performing static analysis on the exported function's similarity.

AFAICT the changes were primarily caused by the increase in length of the exported symbol names (since we made the version number very large intentionally) instead of just increasing a single digit place. Here are examples of that and the other changes:

Here the drive letter and timestamp changed:
old_nasm_example

Here you can see length offsets and string lengths increasing:
old_nasm_example_2

And then here's the comparison between the old and new object file via the Ghidra-powered binary diffing of ghidriff, which indicates that nothing about the functions changed:

old_nasm_example_3

@complexspaces
Copy link
Collaborator Author

Once this merges I will create and push a tag for use internally per the final README steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant