Skip to content
This repository has been archived by the owner on Apr 26, 2021. It is now read-only.

[GSoC] Upgrade Android analysis components #2845

Open
wants to merge 128 commits into
base: master
Choose a base branch
from

Conversation

muhzii
Copy link

@muhzii muhzii commented Aug 26, 2019

Thanks for contributing! But first: did you read our community guidelines?
https://cuckoo.sh/docs/introduction/community.html

What I have added/changed is:
The goal of my change is:
What I have tested about my change is:

There is a write-up/ summary about the work done in these upgrades: https://gist.github.com/muhzii/a65334123880d808f33dc01a24c0e5fa

muhzii added 18 commits October 16, 2019 23:34
as we would normally need to query the authentication token which is created as user-read-only
so ideally, we need to issue the kill command using the root user to be able to read the token
The current way of handling behavioral events is that one 'process' event needs to be sent before handling any other events.
In case of Android, we have more than one behaviral log with separate handlers. So the best way to remedy this is by
emitting a process event for each handler.. that way the other handlers can recognize the process
@nblog
Copy link

nblog commented Jun 9, 2020

@muhzii

after I build it, submit a sample analysis
the results obtained

analysis.log

2020-06-09 13:00:11,335 [lib.core.Frida] ERROR: ReferenceError: methodName is not defined
at patchMethodCall (/script1.js:697:15)
at /script1.js:862:48
at Array.forEach ()
at setupEnv (/script1.js:862:13)
at applyPreAppLoadingInstrumentation (/script1.js:608:5)
at /script1.js:970:17
at frida/node_modules/frida-java-bridge/index.js:170:1
at o.perform (frida/node_modules/frida-java-bridge/lib/vm.js:12:1)
at m.performNow (frida/node_modules/frida-java-bridge/index.js:164:1)
at Object.start (/script1.js:969:18)

https://github.com/muhzii/cuckoo/blob/master/cuckoo/data/analyzer/android/lib/core/agent.js#L697

I tried to change it to "klass[hookConfig.method].overloads[0].implementation = function () {"

Still error:
ERROR: TypeError: Cannot read property'overloads' of undefined

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants