-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[BUG] Cromite v120.0.6099.199 crashes after unmodified recompilation with apktool 2.9.2 #3489
Comments
Sounds like you've done what I was going to ask and isolated the disassembly to the resources. Skipping disassembly of resources and launching resulted in no issue. Odd however the error isn't a common one that references a missing resource. Its almost like the sources are trying to reference a resource that is no longer available. |
I read previous tickets and thought it would be a good idea to do these tests immediately :-) Besides that I had some hope that |
I repeated your steps (disassemble, rebuild, disassemble) and can confirm your findings. The smail-files are identical, except some default values. The missing default assignments seem to be no problem since the missing values are the defaults anyway, so nothing changes there. I carefully compared all resources xml files as well and again can confirm that only the ordering changes between disassemble-1 and disassemble-2 versions, with one exception: In case of "res/layout/otp_verification_dialog.xml" there is an additional backslash escape sequence Hence I assume that the problem is introduced already by the first disassemble step and can not be detected with this method. Just to make sure that it is not a version specific problem I tried the same procedure (disassemble, rebuild, install) with the latest Cromite version
One possible next step is probably to build Cromite and then immediately compare the involved resources, thus the primary and the first disassembled one. I did download the sources from Cromite web page, but unfortunately it's not that simple. Those contain only patches/diffs to Chromium. Chromium is really huge and the build instructions of Cromite are still a TODO ... so difficult. Finally I downloaded Chromium sources and did compare some content of xml resource files (with focus on toolbar and color) - as far as this is possible without compiling the code. Up to now I did not find any good clue what resource might causes the null pointer exception. |
Sorry. Click accident. |
Got the same problem for Vivaldi Snapshot. Any progress? |
I have not. If you preform the same test I did in this comment you can see if its just as confusing or provides additional context. |
Information
apktool -version
) - 2.9.2java --version
) - openjdk version "21.0.1" 2023-10-17No modifications were made to the APK file. The re-compiled APK crashes while clicking through the initialization wizard questions, right before the app home screen is displayed. Most likely the app does not have any protection. So the app works fine without any crash when zipaliging and signing the original APK directly.
All tests were done with Wi-Fi offline.
Thank you so much for the huge effort you put into this project. Apktool is an amazing development tool.
Stacktrace/Logcat
Steps to Reproduce
apktool d
# with--no-src
the app still crashes, but with--no-src --no-res
or only with--no-res
the app works wellapktool b
zipalign 4
apksigner ...
sign with some key.After installing and starting the app, two setup wizard steps are displayed:
When performing only
zipalign 4
andapksigner ...
with the original APK and installing that, the app does not crash.After decompiling the APK with `-no-src' and re-compiling that, the app still crashes (same position, same stack trace).
After decompiling the APK with
--no-src --no-res
or only with--no-res
and re-compiling that, the app doe not crash anymore. But in this case one has no access to AndroidManifest.xml anymore, unfortunately.APK
https://github.com/uazo/cromite/releases
=> v120.0.6099.199-672a5061d34744482fcdd58ee4c9a5cf24acbd4b
=> arm64_ChromePublic.apk
Questions to ask before submission
apktool d
,apktool b
without changing anything? yesThe text was updated successfully, but these errors were encountered: