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

[FIXED] [REFERENCE] Broken patch on current LineageOS 13 builds (20/10) #21

Open
rafifos opened this issue Oct 26, 2017 · 8 comments
Open

Comments

@rafifos
Copy link

rafifos commented Oct 26, 2017

At some point, the patch in com.android.server.wm.WindowManagerService.addWindow broke. Causing it to not boot anymore, the addWindow method is being rejected at boot-time.

Here is the paste of the output on logcat: https://ghostbin.com/paste/w5gt3

TLDR; [0x4BD] monitor-exit on non-object (Undefined)

Here is a tree to test with: https://gitlab.com/h4ndshake/flyme_devices_motorola_ghost
TLDR²; (The method is located at 14478-16533) https://gitlab.com/h4ndshake/flyme_devices_motorola_ghost/blob/72969ae206f5c3b7d79229858ad7d0dd574cb4e1/services.jar.out/smali/com/android/server/wm/WindowManagerService.smali

@aymende7
Copy link

+1

@rafifos
Copy link
Author

rafifos commented Oct 26, 2017

As a TEMPORARY fix, you can copy the entire method from this repo to your patchrom tree.. it worked here.

@aymende7
Copy link

i am always getting this error while building
out/obj/system/framework/services.a7s/smali/com/android/server/policy/PhoneWindowManager.smali[9706,4] There is already a label with that name.
Exception in thread "main" brut.androlib.AndrolibException: Could not smali file: com/android/server/policy/PhoneWindowManager.smali

any ideas?

@rorre
Copy link

rorre commented Oct 28, 2017

Got this on your method
monitor-exit v33
this should be monitor-exit v34 right?

EDIT: explaination, there is no monitor-enter v33 on your method that's why its undefined.

@rafifos
Copy link
Author

rafifos commented Oct 28, 2017

On mine? where?

@rorre
Copy link

rorre commented Oct 28, 2017

Line 15967

@rafifos
Copy link
Author

rafifos commented Oct 28, 2017

I will revert the patch I've made and test it, just give me about an hour..

I'm an absolute beginner at smali :'v

@rafifos
Copy link
Author

rafifos commented Oct 28, 2017

It fixed the method, thanks.

I'm not closing this issue, if anyone have the same problem, it's easier to find the solution.

TLDR; #21 (comment) fixes it.

@rafifos rafifos changed the title Broken patch on current LineageOS 13 builds (20/10) [FIXED] [REFERENCE] Broken patch on current LineageOS 13 builds (20/10) Oct 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants