Skip to content
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

[search] jadx sometimes freezes during search ( code , case insensitive ) #1568

Closed
mcirsta opened this issue Jul 1, 2022 · 9 comments
Closed

Comments

@mcirsta
Copy link

mcirsta commented Jul 1, 2022

On my machine which has 16 core ( 32 with HT ) sometimes search gets stuck and then all of jadx freezes. This is with 1.4.2 and newer nightly builds.
I've tried different versions of Java and even both Linux and Windows but the result is the same.
The APK that I opened is Waze and decompilation finished before starting the search.

I suspect this is a deadlock somewhere but it doesn't happen all the time.
If you need additional info I can try to get the source code so that I can pause with the debugger and see where it gets stuck.

@daMatz
Copy link
Contributor

daMatz commented Jul 1, 2022

Also happens for me frequently now.

@skylot
Copy link
Owner

skylot commented Jul 1, 2022

I commit a fix for this lock, not sure if it was the only cause.
Anyway, now search task canceling will happen in background and shouldn't cause UI freeze.
Please check latest unstable build.

@Alireza-Razavi
Copy link

@skylot I think the search experience was so better in version 1.2
Now JADX is always processing for every search and makes slower than before.

@mcirsta
Copy link
Author

mcirsta commented Jul 2, 2022

Problem is fixed for me. Thank you very much for the quick fix. It was quite hard to use the app before this fix.

@skylot
Copy link
Owner

skylot commented Jul 3, 2022

Problem is fixed for me

@mcirsta thanks for confirm 👍

@Alireza-Razavi
Jadx now uses limited search, so it stops if results count hit limit, this can cause partial decompilation so on next search decompilation will continue. Such behavior is useful for slow loading from disk if disk cache enabled.
If you want to decompile everything at once you can enable Auto start background decompilation option in preference, it will run full decompilation after loading, and once it finished next search shouldn't process any classes. If it still does, please submit an issue (note that I just commit a fix for such issue, so use the latest unstable build)

@psolyca
Copy link

psolyca commented Jul 4, 2022

With the latest unstable build, search do not freeze and could be cancelled correctly ^^
Thanks for the fix.

@Alireza-Razavi
Copy link

If you want to decompile everything at once you can enable Auto start background decompilation option in preference, it will run full decompilation after loading, and once it finished next search shouldn't process any classes.

@skylot Thanks i didn't know this option :)

@skylot skylot closed this as completed Jul 7, 2022
@daMatz
Copy link
Contributor

daMatz commented Jul 11, 2022

Thank you, also works for me :)

@shezik
Copy link

shezik commented Aug 7, 2022

Working for me as well. Thank you!

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

No branches or pull requests

6 participants