-
Notifications
You must be signed in to change notification settings - Fork 61
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
[Features Request] Various Improvements #6
Comments
Having second thoughts on no.3, this will prevent marking items that are called from piece of code that is not marked as function. This would be undesirable behavior when reversing bare metal firmware (which we do a lot) as it often happens that all functions are not marked as functions even though they are. |
I have addressed no. 1 and no. 3 in the dev branch. Please use that for your PR with no.4 and no.5. There is still need for some fine tuning and adjsutments, I will merge to main once verything is ready. |
You're right it's better to have this type of information rather than completely losing it. Worst case scenario one can "hide" this case from the results after exporting them. |
This (#9) fix the "Missing Rules: |
For the Windows API/Windows-related vulnerabilities I'll need a bit more time as I'm quite busy atm |
I will merger the branches to propagate changes so that those can be included in the new IDA plugin manager. Addition for the Win API calls will be included separately. |
Sure, go ahead :D. I'm sorry but I didn't had the time to add that yet :( |
I'm looking forward to your idea of Add "dangerous" Windows API/Windows-related vulnerabilities. At the same time, I want to know what progress it has currently. I think we can start with the high-risk functions of the Windwos api and some high-risk functions of the kernel. |
Hi, I've "compiled" a list of IMHO useful improvements for the plugin. I'd also like to offer my support in coding some of them (namely #1, #4 and #5) for which I'll try to make PRs in the upcoming days.
Quality of life:
Missing Rules:
_stdio_common_vsprintf
case #9Windows Rules:
PS: feel free to split them into specific issues we can address or rework any of those items. :)
The text was updated successfully, but these errors were encountered: