Reducing stack memory consumption #355

New issue

Open

opened 2020-01-27 04:58:27 -06:00 by pouyaashraf · 2 comments

pouyaashraf commented

2020-01-27 04:58:27 -06:00

(Migrated from github.com)

Description

When using the mbedtls library on certain embedded platforms, a stack-allocated buffer (buf in mbedtls_rsa_rsassa_pss_verify_ext) provokes a stack overflow. The platform in question is LittleKernel running on ARM, where we unfortunately don't have the option of increasing the stack alotted to the kernel. Changing the buf variable to have static storage duration resolves this, but then the function becomes thread un-safe which is not desirable.

Issue request type

[ ] Question
[x] Enhancement
[ ] Bug

### Description When using the mbedtls library on certain embedded platforms, a stack-allocated buffer (`buf` in `mbedtls_rsa_rsassa_pss_verify_ext`) provokes a stack overflow. The platform in question is LittleKernel running on ARM, where we unfortunately don't have the option of increasing the stack alotted to the kernel. Changing the `buf` variable to have static storage duration resolves this, but then the function becomes thread un-safe which is not desirable.  ### Issue request type  [ ] Question [x] Enhancement [ ] Bug

👍 1

pouyaashraf commented

2020-01-27 05:02:30 -06:00

(Migrated from github.com)

To add to this, the amount of memory allocated by buf is 1KiB, and it's done here: https://github.com/ARMmbed/mbed-crypto/blob/development/library/rsa.c#L2165

To add to this, the amount of memory allocated by `buf` is 1KiB, and it's done here: https://github.com/ARMmbed/mbed-crypto/blob/development/library/rsa.c#L2165

ciarmcom commented

2020-01-27 07:26:08 -06:00

(Migrated from github.com)