Decouple frida.re script (now uses frida-compiler) #3612
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 3 commits
January 6, 2026 14:15
…dules. this make maintenance and testing much easier.
… the same output. all output.json are updated.
bernhste
changed the title
This was referenced Jan 6, 2026
Diolor
reviewed
Jan 8, 2026
|
|
||
| # create temporary work folder | ||
| randomNumber=$RANDOM | ||
| workDir=/tmp/frida_$randomNumber |
Collaborator
There was a problem hiding this comment.
Potentially out of scope, but let's better use a project-local temporary folder and not a computer global one?
Plus add it on.gitignore
I could do it in #3584 but then we would conflict
| * @returns {string} The decoded rows and columns. | ||
| */ | ||
| function decodeAndroidDatabaseCursor(value) { | ||
| let out = ""; |
Collaborator
There was a problem hiding this comment.
some lets could be consts, + other files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR closes #3451
Description
The for the demos, the script was compiled using string concatenation. Also, all decoders were implemented in one file (
decoders.js).I refactored the code base to use
frida-compiler. Now the decoders are decoupled and easier to maintain. They are all in the folder./utils/frida/android/decoders. One module for each type, a util module and a module which assigns the decoders to the type.The base script always only used
decodeArguments(). This does not change.I tested the refactored code with all demos which use this frida.re script so far (MASTG-DEMO-0058, MASTG-DEMO-0059, MASTG-DEMO-0060, MASTG-DEMO-0081). They all produce the same output.
The base script itself did change slightly (now using modules). Other than that, no changes. So this should not conflict with #3584.
This should make it much easier maintaining and adding decoders and tests, which I would like to implement.
AI Tool Disclosure
kagi assistantclaude sonnet 4.5take this code <code> and decouple modules for each decoder type (all the cases in the big switch case). make a file for each type. name the files so that it is clear what type it decodes (example: java.io.File.js) name the decode function accordingly (example decodeJavaSecurityPrivateKey()) ONLY export decodeArguments() some of the functions are not type decoder, collect them in a util module. don't change anything at the code nor the comments they should remain as they are fully.[x] I have read the contributing guidelines.