Closed VishnuPratapagiri closed 2 years ago
The rest of the code looks fine. Do we proceed and merge it? We will have to look into this module at the end anyways. What do you guys say? @adityavardhanpadala @akulpillai
@Freakston yup, sounds good. Merge it after he removes the unwanted files
this done?
Nah it isn't. Not update on the pr after that
On Mon, Nov 8, 2021 at 4:54 AM Akul Pillai @.***> wrote:
this done?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/teambi0s/secREtary/pull/30#issuecomment-962701375, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKQRKF54VB2WYC7V5PKR7HLUK4DDNANCNFSM43S4AIIA .
close this? I assume this is redundant stuff from #32 ?
okay
Looks like we are replacing the function call with a dummy function. But the issue I see here with this patching method is what would we do if the function's return value saved into a variable affects control flow of the tracee. @Freakston looks like we need to have another look at what all targets are we thinking to bypass using Anti2RE