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

ida 9.x support #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

MeguminSama
Copy link

inf.min_ea and inf.max_ea were replaced with inf_get_min_ea() and inf_get_max_ea() respectively.

@MeguminSama
Copy link
Author

This is most likely a breaking change with older versions of IDA, so it'd probably need to be made clear if a release is made.

@kweatherman
Copy link
Owner

kweatherman commented Aug 11, 2024

Okay, I see where it comes from. They exposed the beta version
Thanks for doing the work already.

I normally just update to what ever the latest release is as they come out.
I will think and at how to handle this. Either have to have multiple versions, or if possible load different versions of functions dynamically, or just wait until IDA 9 is officially released.

@xenontrioxide
Copy link

IDA 9.0 Beta is particular about plugin file naming. The extension must be exactly .dll to load. As in previous versions, the 64-bit IDA (now the default) only loads plugins with 64 directly before the extension. For public use, rename the plugin to IDA_SigMaker64.dll instead of the project's IDA_SigMaker64.dLL

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.

3 participants