Fix implementation of GetCertificateStore and GetDeviceCertificate

[josesimoes]

Description

• A config block is now created from heap in order to return the cert store and the device certificate.

Motivation and Context

• The recent changes in Rework TLS to use new config manager API #2093 assumed that memory allocation was happening inside there functions. For memory mapped storage, that's not the case, thus, trying to free this memory, causes an hard fault. With this fix, for memory mapped storage, a copy of the bundle/certificate is now performed. The extra memory penalty is not an issue because this happens very briefly when the certs are being parsed to be added to mbedTLS context.

How Has This Been Tested?

Screenshots

Types of changes

• Improvement (non-breaking change that improves a feature, code or algorithm)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Config and build (change in the configuration and build system, has no impact on code or features)
• Dependencies (update dependencies and changes associated, has no impact on code or features)
• Unit Tests (work on Unit Tests, has no impact on code or features)
• Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[nfbot]

Hi @josesimoes,

I'm nanoFramework bot.
Thank you for your contribution!

A human will be reviewing it shortly. 😉