Stop using ERR_GET_FUNC, since it has been removed in OSSL3 Beta2.

[bartonjs]

Restructure CryptoNative_LookupFriendlyNameByOid to not depend on the so-deprecated-its-removed ERR_GET_FUNC.

This approach calls a2d_ASN1_OBJECT directly, rather than looking for it having been the source of the error.
While that does make for double work on success, it's just a state walker for a short string, so it's not too bad.

While noodling around in the function, I noticed that there was a small memory leak, so now that's fixed, too.

Fixes#57674 (in main).

[ghost]

Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq, @GrabYourPitchforks
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Restructure CryptoNative_LookupFriendlyNameByOid to not depend on the so-deprecated-its-removed ERR_GET_FUNC.

This approach calls a2d_ASN1_OBJECT directly, rather than looking for it having been the source of the error.
While that does make for double work on success, it's just a state walker for a short string, so it's not too bad.

While noodling around in the function, I noticed that there was a small memory leak, so now that's fixed, too.

Fixes #57674 (in main).

Author:	bartonjs
Assignees:	bartonjs
Labels:	area-System.Security
Milestone:	7.0.0

[stephentoub]

Does this need to be backported?

[bartonjs]

/backport to release/6.0

[github-actions]

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1153824338