search

skylot / jadx

Dex to Java decompiler
Apache License 2.0
41.82k stars 4.88k forks source link

[core]Endless jadx #769

Closed superpoussin22 closed 5 years ago

superpoussin22 commented 5 years ago

Checks before report

Describe error apk used is : https://f-droid.org/repo/com.nextcloud.client_30080190.apk

output of jadx provide ( on ubuntu 18.04) `` WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by com.rits.cloning.Cloner (file:/root/Mobile-Security-Framework-MobSF/StaticAnalyzer/ tools/jadx/lib/cloning-1.9.12.jar) to field java.util.TreeSet.m WARNING: Please consider reporting this to the maintainers of com.rits.cloning.Cloner WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations WARNING: All illegal access operations will be denied in a future release ^Croot@b7138f185e6e:~/Mobile-Security-Framework-MobSF# ./StaticAnalyzer/tools/jadx/bin/jadx -ds /tmp -r /root/.MobSF/uploads/60ac13724348dac5f1f5ba427f5eebbd/60ac13724348dac5f1f5ba427f5eebbd.apk WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by com.rits.cloning.Cloner (file:/root/Mobile-Security-Framework-MobSF/StaticAnalyzer/tools/jadx/lib/cloning-1.9.12.jar) to field java.util.TreeSet.m WARNING: Please consider reporting this to the maintainers of com.rits.cloning.Cloner WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations WARNING: All illegal access operations will be denied in a future release

and nothing more :(

Note: no need to copy method fallback code (commented pseudocode)

jpstotz commented 5 years ago

I am sorry but I can't reproduce your problem.

Using latest unstable version "jadx-1.0.0-b1157-f228a721.zip" the command you posted works without problems and decompiles the APK file you have linked.

BTW: using /tmp without any subdirectory is a bad idea as the decompilation will create ~16 subdirectories which then mic-up with the other content of /tmp.

superpoussin22 commented 5 years ago

will test again and let you know ( by default we only include stable release in MobSF) using /tmp was only to grab output: when used in MobSF we don't use /tmp

superpoussin22 commented 5 years ago

perfect for me :)