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

Magisk canary 122b4d6 will break OP7T fingerprint unlock #2803

Closed
sabpprook opened this issue May 15, 2020 · 62 comments
Closed

Magisk canary 122b4d6 will break OP7T fingerprint unlock #2803

sabpprook opened this issue May 15, 2020 · 62 comments
Labels
regression Something works in previous versions but not in the current one

Comments

@sabpprook
Copy link
Contributor

OnePlus 7T HD1900 OOS India 10.3.2.HD65AA

It only happened on 122b4d6, previous build a62bdc5 works fine.
I'm not able to get logcat right now, sorry for this.

@osm0sis osm0sis added the regression Something works in previous versions but not in the current one label May 15, 2020
@osm0sis
Copy link
Collaborator

osm0sis commented May 15, 2020

Can confirm on my North America 7T. More likely it's caused by 6a2e781, specifically vendor.boot.verifiedbootstate, which is similar to custom kernels before this engstk/op7@c8b1bcf

I know it seems to be fine when set later in boot so I'll see if there's a way to fix that without just removing it entirely.

@sabpprook
Copy link
Contributor Author

After some quick test, the latest canary build will lead fingerprint scan to fast that unable to detect whole fingerprint. Can not figure out why this will happen

@osm0sis
Copy link
Collaborator

osm0sis commented May 15, 2020

Here's a workaround script for now.

/data/adb/service.d/verifiedboot-fp-fix.sh:

#!/system/bin/sh

# Magisk service.d script
resetprop vendor.boot.verifiedbootstate orange
until [ "$(getprop sys.boot_completed)" == 1 ]; do
  sleep 1
done
resetprop vendor.boot.verifiedbootstate green

@sabpprook
Copy link
Contributor Author

sabpprook commented May 15, 2020

Problem sort out

Due to this issue, maybe the codes of props fix will move to boot_complete to avoid this?

@osm0sis
Copy link
Collaborator

osm0sis commented May 15, 2020

Just that prop needs to move later. Best to leave the others as early as possible.

@Chartman123
Copy link

Chartman123 commented May 16, 2020

Seems to affect OnePlus 8, too. Can't use my fp sensor since I've installed the latest canary update.

Adding the script to the service.d directory doesn't seem to be working here :/

@osm0sis
Copy link
Collaborator

osm0sis commented May 16, 2020

MagiskHide runs as soon as module post-fs-data scripts and system props are done, doing hide_sensitive_props(). Using service.d to flip it back to orange in my workaround unfortunately might just still be too late on the OP8, but the actual solution would likely be the same for all.

What we'll need to avoid removing resetting vendor.boot.verifiedbootstate green entirely is a separate hide_sensitive_props_late() which runs at sys.boot_completed like my script above.

Adding a new trigger for that to magiskinit/hide isn't something I'm sure how to do so hopefully @topjohnwu can take a look at it.

@osm0sis osm0sis mentioned this issue May 16, 2020
@osm0sis osm0sis changed the title Magisk canary 122b4d6 will break OP7T fingerprint unlock Magisk canary 122b4d6 will break OP7T + OP8/8Pro fingerprint unlock May 16, 2020
@sabpprook
Copy link
Contributor Author

A quick fix sabpprook@e71bdb4

@osm0sis
Copy link
Collaborator

osm0sis commented May 17, 2020

That's pretty much what I had in mind, actually. Nice job! Open a PR?

@ekka1993
Copy link

Will this fix the fingerprint error on oneplus 8 pro ? With magisk 20.4?

@osm0sis
Copy link
Collaborator

osm0sis commented May 17, 2020

@ekka1993 This has nothing to do with 20.4, it's only broken in Canary 20411.

@osm0sis
Copy link
Collaborator

osm0sis commented May 17, 2020

Ah, looks like @topjohnwu also took a crack at it: 97db49a

@ekka1993
Copy link

Is it solved?

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

Yep, working again on my 7T in 20412. 👍

It stays reporting orange though due to a typo I just found and let @topjohnwu know about. That should be fixed for 20413.

@bittylicious
Copy link

Sadly 20412 isn't working here on my OnePlus 8.

I still get the "Fingerprint hardware not accessible" error on the lock screen. This is using the latest OnePlus release for the Europe variant, 10.5.6.IN21BA.

I'm not sure if it's related, but SafetyNet fails too even with MagiskHide on (ctsProfile fails, basicIntegrity is fine). It worked with 10.5.5.IN21BA but I can't confirm which version of Magisk Canary I was on then. I'm aware you don't generally investigate SafetyNet with canary releases though.

@s12nkyp1g

This comment has been minimized.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

@bittylicious can you clarify if 20411 was where this issue started for your 8? That's where it started on my 7T. If you didn't try 20411 then please try 20409 and let me know if it works.

@bittylicious
Copy link

@osm0sis It definitely was not working with 20411. I can't easily confirm whether it was the upgrade to 20411 that caused it to fail, or the Android update to 10.5.6.

I sadly don't have a build environment to prepare specific releases. Is there an easy way for me to try 20409, e.g. if you can provide me with a custom channel URL or something like that?

Ignore the SafetyNet error - disabling all modules enables SafetyNet to pass again.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

@bittylicious all Canary releases are contained here: https://github.com/topjohnwu/magisk_files/commits/canary

Simply go back to the commit for 20409 and download the magisk-debug.zip from that commit, then flash it via Manager's Modules page "Install from storage"

@bittylicious
Copy link

@osm0sis Thanks for the instructions.

I'm now on 20409 but sadly the fingerprint doesn't work, so this issue is probably unrelated. Best guess is that it's the same issue @ekka1993 is experiencing with his OnePlus 8 Pro.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

Does any previous Magisk build work on the 8 Series? Are you using custom kernels?

@Chartman123 you reported it happened with 20411 on your 8, can you update us on whether 20412 fixes it for your setup?

@osm0sis osm0sis changed the title Magisk canary 122b4d6 will break OP7T + OP8/8Pro fingerprint unlock Magisk canary 122b4d6 will break OP7T fingerprint unlock May 18, 2020
@bittylicious
Copy link

@osm0sis Yes, everything was working wonderfully before, fingerprint, SafetyNet, the lot. It was beautiful. No custom kernels.

Based on the Canary timestamps, I think it's the Android upgrade which cased things to break, not any change made in Magisk. Most likely the Android upgrade has introduced some sort of incompatibility with Magisk though.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

@bittylicious And you aren't using my workaround script, presumably?

@ekka1993
Copy link

Mine worked with 10.5.8 then i restored with brick tools then i stopp worked.
I removed magisk then mine phone got soft bricked So i hade to restore it with bricked tools. Then i stopp worked.

@bittylicious
Copy link

@ekka1993 Change your settings Update Channel to Canary. It's the latest Canary version.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

If turning MagiskHide off and rebooting doesn't help then the latest Canary won't make a difference either, unfortunately.

@ekka1993
Copy link

I Still have 20400 hmm

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

Yeah, so @ekka1993, when did it work last? What's your ROM version?

@ekka1993
Copy link

I have 10.5.8

@ekka1993
Copy link

It worked on 10.5.8 turn i bricked mine phone and i restored it then i stop to work

@bittylicious
Copy link

bittylicious commented May 18, 2020

FYI, 10.5.8 is the latest update for the 8 Pro. It will include similar patches to 10.5.6, the latest version on the 8. So, me and @ekka1993 are likely experiencing the same error if it stopped working after the latest Android update for him.

If @Chartman123 can confirm he's not yet on the latest Android/OnePlus version, this puts together quite a clear picture. If he is on 10.5.6 (8) or 10.5.8 (8 Pro), then who knows!

One of the changelog entries for this update is:

System
Optimized touch and interaction experience

So it could be related to this.

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

ekka was already on latest and had it working, then it broke after bricking the device and using some tools to recover it.

Sounds like they were on 10.5.8 and Magisk 20.4(20400) stable both when working and broken, so, a bit less clear of a picture there.

@Chartman123
Copy link

I'm on 10.5.4 European version, the latest that is offered to me in the updater

@osm0sis
Copy link
Collaborator

osm0sis commented May 18, 2020

@bittylicious have you tried rebooting with MagiskHide off yet to rule out some further interaction there?

@bittylicious
Copy link

@osm0sis Yes tried that, doesn't make a difference here.

@Chartman123 Interesting. You should get 10.5.6 soon with a bit of luck. I would have suggested using the OnePlus downloads page but it doesn't appear to be on there yet, so maybe they've spotted issues with the OTA update? Let us know when it comes please.

@zyonee
Copy link

zyonee commented May 18, 2020

The latest canary has the fingerprint reader working on my OP8 Pro with OOS 10.5.8. Not sure if it matters, but it's the European version. Went directly from canary a62bdc5 to 99ef206. I did try 122b4d6 yesterday with the fingerprint reader not working, but for me the latest canary fixed the issue. Good work!

@ekka1993
Copy link

Should i use direct update ? do i need to patch it and install with adb ?

@0xCCD
Copy link

0xCCD commented May 19, 2020

Mine worked with 10.5.8 then i restored with brick tools then i stopp worked.
I removed magisk then mine phone got soft bricked So i hade to restore it with bricked tools. Then i stopp worked.

I had the same issue, in my case downgrade to v20.4 solved it and new canary also. But on oneplus forums they talk also about the fact that a fastboot flash will damage one partition which is related for the fingerprint

https://forums.oneplus.com/threads/enrollment-was-not-completed-fingerprint-registration-error.1218379/page-2#post-21617960

@osm0sis
Copy link
Collaborator

osm0sis commented May 19, 2020

@sabpprook @Chartman123 @zyonee @0xCCD
The original reported issue is now properly resolved in 20414 with vendor.boot.verifiedbootstate green after 3c04dab. 👍

Those of you still having issues must have something else going on, will need to look into that further and open a new issue if appropriate. Be sure to include last working Magisk build, test stable and canary, try in Safe Mode, try with MagiskHide off and on, and all the logs.

@ekka1993

This comment has been minimized.

@0xCCD
Copy link

0xCCD commented May 21, 2020

Hey guys, do you have the stock boot img for oneplus 8 pro 10.5.8 Europe (IN11BA) ?
Did know how to Contact you

https://send.firefox.com/download/32475d18c3fc0745/#t93WiF7h3IGEWhzTFola5w

@ekka1993
Copy link

Is it patched with magisk or stock ?

@0xCCD
Copy link

0xCCD commented May 21, 2020

Is it patched with magisk or stock ?

fresh from OTA on my sdcard, not patched

@ekka1993
Copy link

Thanks, i Still has problem with fp, i Will try a new boot img IF it Will work

@ekka1993
Copy link

Is your fp working with 10.5.8 and magisk ?

@0xCCD
Copy link

0xCCD commented May 21, 2020

Thanks, i Still has problem with fp, i Will try a new boot img IF it Will work

check this https://forums.oneplus.com/threads/enrollment-was-not-completed-fingerprint-registration-error.1218379/page-2#post-21617960

@0xCCD
Copy link

0xCCD commented May 21, 2020

Is your fp working with 10.5.8 and magisk ?

Yes it is working but there are two different issues atm with fingerprint. one of them was caused by Magisk update another is caused while flashing through fastboot

@ekka1993
Copy link

ekka1993 commented May 21, 2020

Flashing with adb/fastboot cant be solved right now ? Or am i wrong ?

@0xCCD
Copy link

0xCCD commented May 21, 2020

As far as I know you can only solve the issue caused by Magisk, the other issue is caused by a damaged partition which need to get fixed with a software update from Oneplus

@ekka1993
Copy link

I Will try reflash a new magisk boot img and see IF it works

@0xCCD
Copy link

0xCCD commented May 21, 2020

v20.4 and latest canary works fine for me

@ekka1993
Copy link

Same error so have to wait for oneplus to slove it.

New boot img magisk 20.4 and latest canary. 20414

@bittylicious
Copy link

I just want to confirm with you that I have this error still, and I never flashed anything from fastboot manually.

My error (which granted is NOT the issue on this issue) came about when I had Canary (probably 20410 or 20411, and then upgraded to the latest OnePlus version (10.5.6 in my region). This would have been a full ROM download, not the OTA patch, so maybe that also involves a flash of other partitions? I'm not sure.

@0xCCD
Copy link

0xCCD commented May 21, 2020

I just want to confirm with you that I have this error still, and I never flashed anything from fastboot manually.

My error (which granted is NOT the issue on this issue) came about when I had Canary (probably 20410 or 20411, and then upgraded to the latest OnePlus version (10.5.6 in my region). This would have been a full ROM download, not the OTA patch, so maybe that also involves a flash of other partitions? I'm not sure.

I usually also have full rom updates, but as long it is flashed through Local Upgrade functionality of Oxygen OS it should be safe. Probably you can ask on oneplus forum

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Something works in previous versions but not in the current one
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants