kdrag0n / safetynet-fix

Google SafetyNet attestation workarounds for Magisk
MIT License
6.16k stars 325 forks source link

Fingerprint print scanner not working #177

Closed Athulnalin closed 2 years ago

Athulnalin commented 2 years ago

Describe the bug A clear and concise description of what the bug is.

To reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Device info Device model: Android version: ROM name/version:

Logs Connect your phone to a computer and run adb logcat > issue.log. Attach the log file to this issue.

Additional context Add any other context about the problem here.

Checklist

Jon8RFC commented 2 years ago

You'll need to come back here and look at your report you made, and then fill in all of the blanks.

JakobLichterfeld commented 2 years ago

Possible duplicate of #173

frnode commented 2 years ago

Hello everyone, @Athulnalin

If you wish, I worked on the issue and now it works (fingerprint and SafetyNet verification), you can find the change below: https://github.com/kdrag0n/safetynet-fix/blob/57b726c260bb40b838c5d942965282a5a482bdbe/zygisk/module/jni/module.cpp?plain=1#L134 safetynetriru is wrong, he must be safetynetfix

For information, the change present here (https://github.com/kdrag0n/safetynet-fix/commit/55f892fb8855a7867a186970e0bfc31e665ab183), will not solve all the problems once compiled. The change mentioned above must also be applied.

I applied the change on my fork: https://github.com/frnode/safetynet-fix/commit/dc3ec64eff578bc88aa6bb2970c026d71f9799e0#diff-e39ee2cf32c559774a334de83bb183fcd361c35bdd40c208cd809e1915df402dL134

I also compiled it, here: https://github.com/frnode/safetynet-fix/releases/tag/v2.3.0 If you prefer, you can also compile it yourself, with the docker I have also added (just launch docker-compose.yml) 🙂.

With pleasure ! 🚀

ruszok commented 2 years ago

Works fine! Thanks!

adwil13 commented 2 years ago

@frnode Your build works perfectly with the new OxygenOS 12.1 build, however I still fail on CTS profile check, any ideas how to get around that?

frnode commented 2 years ago

@adwil13 Yes, device with the latest C20 update, there is a problem with CTS verification. I will look soon and I will update the fork if I find the solution.

188

adwil13 commented 2 years ago

@frnode both c16 and c17 had same issue, unfortunately I need Google pay and root on this phone ;) Got around this by spoofing OxygenOS 11 device fingerprint using MagiskHidePropsConf, however this breaks fingerprint again. So I had to revert back to original OxygenOS 12 fingerprint after setting up Google Pay. I did this before and it's a temporary solution as Google Pay will re-check safetyNet after some time, but It's good for now.

ruszok commented 2 years ago

+1 props works but then fingerprint doesn't.

adwil13 commented 2 years ago

For complete solution that fixes both Google Play Certification, CTS profile and Google Pay please see: https://github.com/kdrag0n/safetynet-fix/issues/188#issuecomment-1147295375

kdrag0n commented 2 years ago

Fixed in v2.3.1.

ruszok commented 2 years ago

Fixed in v2.3.1.

Should both works - fingerprint and Safetynet? In my case FP works fine Safetynet not pass.

kdrag0n commented 2 years ago

Fixed in v2.3.1.

Should both works - fingerprint and Safetynet? In my case FP works fine Safetynet not pass.

That's a different issue. #188