-
Notifications
You must be signed in to change notification settings - Fork 253
Added support for relative paths for dumpTree #603
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
base: master
Are you sure you want to change the base?
Added support for relative paths for dumpTree #603
Conversation
* ```ts | ||
* Il2Cpp.perform(() => { | ||
* Il2Cpp.dumpTree(); | ||
* }); | ||
* ``` | ||
*/ | ||
export function dumpTree(path?: string, ignoreAlreadyExistingDirectory: boolean = false): void { | ||
if (path && !path?.startsWith("/")) path = `${Il2Cpp.application.dataPath!}/${path}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid this is this not cross platform...
Hi, #605 has been implemented, please read: https://github.com/vfsfitvnm/frida-il2cpp-bridge/blob/d624db6f016a0a3f6ff9de7a7113ba743a34727e/README.md#dumping. I believe the new way of dumping is much better as have the most flexibility possible and opens up new features :) |
00e222b
to
06c6284
Compare
I think the dumping function should be left for use in a custom agent for when additional things are needed for compatibility with an app. |
@namtacs would you elaborate? What do you mean with "additional things are needed for compatibility with an app"? |
For example, replacing the IL2CPP exports. |
My bad for not documenting it, but you can use frida-il2cpp-bridge/cli/src/app.py Lines 53 to 58 in 18cb9be
|
Figured might as well.
Should close #602.