Fix unittest build with MinGW

[ba-evva]

Unit test build crashes if MBEDTLS_PLATFORM_HAS_NON_CONFORMING_VSNPRINTF is defined. Add #include <stdarg.h> to fix this issue.

Description

Building unit tests on Windows 7 with MinGW/MSYS fails at 9% completion because 'va_list' is not declared. Inserting the mentioned #include (like that in the following #if section a few lines later) will fix the issue.

Details

Command issued:

mbed test --unittest --generator "MinGW Makefiles" --make-program mingw32-make

Unfixed build fails with:

In file included from P:\mbed-os\features\lorawan\lorastack\mac\LoRaMacCrypto.cpp:31:0:

p:\mbed-os\features\mbedtls\inc\mbedtls\platform.h:260:75: error: 'va_list' has not been declared
int mbedtls_platform_win32_vsnprintf( char *s, size_t n, const char *fmt, va_list arg );

Compiler version: g++ (MinGW.org GCC-6.3.0-1) 6.3.0

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

[ciarmcom]

@ba-evva, thank you for your changes.
@ARMmbed/mbed-os-tls @ARMmbed/mbed-os-crypto @ARMmbed/mbed-os-maintainers please review.

[Patater]

This fix has already landed in Mbed TLS upstream and will not be blown away next time we update Mbed TLS. However, for those users to depend on https://github.com/ARMmbed/mbed-os/blob/master/features/mbedtls/VERSION.txt to accurately reflect what version of Mbed TLS is being used, I would recommend we instead update to a version of Mbed TLS that includes this fix.

[ba-evva]

recommend we instead update to a version of Mbed TLS that includes this fix.

This would be OK for me.

[ba-evva]

Sorry, clicked the wrong button. Reviewers, please close after final decision.

[0xc0170]

@Patater Shall we close this one and wait for the update?

[0xc0170]

@Patater Shall we close this one and wait for the update?

👀

[Patater]

@Patater Shall we close this one and wait for the update?

eyes

Yes, please.

[Patater]

Fixed via PR #11035