Remove unused code for PBKDF2 on Android

[vcsjones]

While working on something else, I noticed that we had some code for calling in to the android native shims for PBKDF2... but android doesn't use native code for PBKDF2. It uses a managed implementation. So this LibraryImport wasn't importing anything real.

[ghost]

Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

---

While working on something else, I noticed that we had some code for calling in to the android native shims for PBKDF2... but android doesn't use native code for PBKDF2. It uses a managed implementation. So this LibraryImport wasn't importing anything real.

Author:	vcsjones
Assignees:	-
Labels:	area-System.Security, os-android
Milestone:	-

[vcsjones]

/azp run runtime-android

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jkoritzinsky]

Is there any code on the native side that provides an implementation here? If so, we want to switch to using the native implementation? Otherwise, we should remove the native implementation from the native shim.

[vcsjones]

Is there any code on the native side that provides an implementation here?

No. There is no entry point CryptoNative_Pbkdf2 in the native shim for Android. So there is nothing to clean up there. It was a P/invoke to nothing.

To preemptively answer: Android does not have the native capability to do this, either when I last looked less than a year ago.

[marek-safar]

/azp run runtime-android

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).