xamarin-fingerprint icon indicating copy to clipboard operation
xamarin-fingerprint copied to clipboard

Android Implementation allow to bypass the security

Open rodirigos opened this issue 2 years ago • 7 comments

Hi Everyone. This implementation of the Biometric is not 100% accurate. It allows users to bypass the security since it does not implement the android secret and cypher. I can find here the script: https://codeshare.frida.re/@Saket-taneja/biometricauthenticationbypassnullcryptoobject/

Steps to reproduce

  1. Install Frida and use the script above when prompted the biometric dialog.

Expected behavior

The authentication should fail after the script. It should have some crypto object, at least for Android devices. It does not contain the cipher to ensure the CryptoObject is decrypted.

Actual behavior

It allows bypassing the fingerprint verification

Configuration

Version of the Plugin: 2.1.5

Platform: Android

Device: Any

rodirigos avatar Jun 09 '22 13:06 rodirigos

Hi @rodirigos,

I'm the friendly issue checker. Thanks for using the issue template :star2: I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

smsissuechecker avatar Jun 09 '22 13:06 smsissuechecker

Hi, could you or somebody check if this implementation prevents the bypass https://github.com/DarkIrata/xamarin-fingerprint/tree/android-cryptoobject I added the CryptoObject to the authentication and cipher validation. Currently i don't have any devices i could install frida on and problems with the Android Emulator.

If it fixes the problem, I will create a pull request.

DarkIrata avatar Aug 12 '22 20:08 DarkIrata

Any update on this?

jvillaro avatar Oct 19 '23 03:10 jvillaro

Didnt heard anything back

DarkIrata avatar Oct 22 '23 20:10 DarkIrata

@DarkIrata thanks for your response, I saw that you made a fix but it hasn't been merged right?

jvillaro avatar Oct 23 '23 13:10 jvillaro

@DarkIrata thanks for your response, I saw that you made a fix but it hasn't been merged right?

jvillaro avatar Oct 23 '23 13:10 jvillaro

right, not merged yet. I just compiled it myself and use it like that for now

DarkIrata avatar Oct 23 '23 14:10 DarkIrata