forked from nrfconnect/sdk-zephyr
-
Notifications
You must be signed in to change notification settings - Fork 0
[nrf noup] tests: secure_storage: Fix build issues #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… strlcpy" This reverts commit 03363a9. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit cd1c689. Signed-off-by: Andrzej Głąbek <[email protected]>
… descriptors" This reverts commit 643506a. Signed-off-by: Andrzej Głąbek <[email protected]>
…verflowing" This reverts commit f1dc0e4. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit e814b7e. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit e42d302. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit f535df7. Signed-off-by: Andrzej Głąbek <[email protected]>
…port get" This reverts commit 1f8e370. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 5b87a8a. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit fde19b1. Signed-off-by: Andrzej Głąbek <[email protected]>
…ror conditions" This reverts commit c57352e. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit d43b88f. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 29b9b3e. Signed-off-by: Andrzej Głąbek <[email protected]>
…flash by default" This reverts commit 90d4047. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 50d56af. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 5afe89d. Signed-off-by: Andrzej Głąbek <[email protected]>
…warder Kconfig" This reverts commit e2c87e8. Signed-off-by: Andrzej Głąbek <[email protected]>
…utomatic" This reverts commit 4ba4cab. Signed-off-by: Andrzej Głąbek <[email protected]>
…ies" This reverts commit 173223e. Signed-off-by: Andrzej Głąbek <[email protected]>
…ase" This reverts commit 841d6ce. Signed-off-by: Andrzej Głąbek <[email protected]>
… quirks" This reverts commit fb52d17. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 4a996a8. Signed-off-by: Andrzej Głąbek <[email protected]>
… core" This reverts commit 97e7184. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 13117f6. Signed-off-by: Andrzej Głąbek <[email protected]>
…nes" This reverts commit b1e8294. Signed-off-by: Andrzej Głąbek <[email protected]>
…sable" This reverts commit 0bb05fc. Signed-off-by: Andrzej Głąbek <[email protected]>
…hread" This reverts commit 6246b5a. Signed-off-by: Andrzej Głąbek <[email protected]>
…h UDC API" This reverts commit afad960. Signed-off-by: Andrzej Głąbek <[email protected]>
…nts" This reverts commit 3fbfd62. Signed-off-by: Andrzej Głąbek <[email protected]>
This reverts commit 734839f. Signed-off-by: Andrzej Głąbek <[email protected]>
Fixes build issue when WMM AC feature is disabled. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 2350b6b)
Remove the HCI command & event emulation layer for ECDH commands and events. This means that we always do the necessary operations in the host. The existing BT_ECC Kconfig option stays, but now gets automatically enabled when necessary (e.g. based on the BT_SMP option), which is why this commit removes so many explicit assignments in prj.conf files. Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit 09e86f3)
This option only exposes internal APIs, so there should be no need to allow applications to set an explicit value. Instead, users of the API should select it through Kconfig. Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit d3c8cb4)
Mention the removed prompt for BT_ECC in the migration guide, and also add a note about the removed HCI emulation layer to the release notes. Signed-off-by: Johan Hedberg <[email protected]> (cherry picked from commit b3c6151)
…got deprecated This is a follow-up to commit 8cfad44. Replace the deprecated BT_LE_ADV_CONN macro with BT_LE_ADV_CONN_FAST_2. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit f930739)
-liblc3 added a library-internal path to zephyr_interface containing a file that collides with common.h required e.g. by Mbed TLS This commit fixes build-issues by making this include-folder PRIVATE to the named library liblc3 Signed-off-by: Frank Audun Kvamtrø <[email protected]> (cherry picked from commit 2ca64a3)
-Mbed TLS requires common.h to build. The sample provides common.h in sample folder level but included the path to zephyr_interface. This commit changes the include to be sample-specific Signed-off-by: Frank Audun Kvamtrø <[email protected]> (cherry picked from commit 627d2f9)
…rm_zeroize -Adding explicit include for mbedtls/platform_util.h to get acces to mbedtls_platform_zeroize in ITS. Somehow not visibile in Zephyr but it caused build issues in nRF Connect SDK. Signed-off-by: Frank Audun Kvamtrø <[email protected]> (cherry picked from commit 202c2c4)
…he boards Bring the change that was already in place for the qemu_cortex_m3 board to all the other ones, since also nrf52840 had a similar problem (see issue #82812). Signed-off-by: Valerio Setti <[email protected]> (cherry picked from commit 9d0b124)
The BT Host module also uses import/export PSA functions alongside the generate one, so these PSA_WANT should be added as well. Previously it happened to work only because Mbed TLS is enabling IMPORT/EXPORT internally whenever GENERATE/DERIVE is set. However the same might not be true for all PSA Crypto providers. Signed-off-by: Valerio Setti <[email protected]> (cherry picked from commit 222f8d8)
…rver The unicast server does not need GATT caching and it was likely incorrectly added earlier as a dependency. Upstream PR #: 84232 Signed-off-by: Emil Gydesen <[email protected]>
… updates -Update hostap to PR nrfconnect#76 which adds support for CONFIG_MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG instead of relying on legacy Mbed TLS ctr_drbg and entropy APIs. This change is in line with PSA crypto entropy changes in Zephyr happening with TinyCrypt deprecation and advancement of PSA crypto mechanisms -Remove defconfigs for currently sets the Kconfigs: -CONFIG_MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG -CONFIG_MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG_ALLOW_NON_CSPRNG Upstream PR #: 84517 Signed-off-by: Frank Audun Kvamtrø <[email protected]>
…endencies -PSA_WANT_ALG_CMAC must be set for PSA_WANT_ALG_PBKDF2_AES_CMAC_PRF_128 to be valid. This commit fixes this mismatch for hostap Upstream PR #: 84517 Signed-off-by: Frank Audun Kvamtrø <[email protected]>
…nfigurations
-Mbed TLS has come up with its own naming-convention with regards
to certain Mbed TLS configurations for TLS/DTLS and X.509. This
commit fixes a couple of them by depending on MBEDTLS_BUILTIN
which is set when Kconfig.tls-generic is used
1. Make MBEDTLS_PEM_CERTIFICATE_FORMAT depend on MBEDTLS_BUILTIN
The proper name for this functionaity is MBEDTL_PEM_PARSE_C and
MBEDTLS_PEM_WRITE_C
2. Make MBEDTLS_SERVER_NAME_INDICATION depend on MBEDTLS_BUILTIN
The proper name for this is MBEDTLS_SSL_SERVER_NAME_INDICATION
Upstream PR #: 84517
Signed-off-by: Frank Audun Kvamtrø <[email protected]>
This is a follow-up to commit 12eee61. Explicitly enable "PSA_WANT_ALG_ECB_NO_PADDING" to select the AES ECB mode that it is used in CMAC operation. This is done because CMAC uses AES-ECB, so both AES and ECB must be explicitly enabled. Previously it worked because Mbed TLS is not currently performing any check internally on this and it's just enabling ECB automatically. Upstream PR #: 84638 Signed-off-by: Andrzej Głąbek <[email protected]>
There's no point building the sample for platforms that do not support networking, hence add netif dependency to reduce the CI execution scope. Upstream PR #: 84857 Signed-off-by: Robert Lubos <[email protected]>
Non-secure variants for nRF7002 DK were removed from upstream in commit 10d4973. Revert these changes downstream, so that the NS variants are still available. Signed-off-by: Andrzej Głąbek <[email protected]>
…en built-in in PSA" This reverts commit ac6d834. Temporarily revert an upstream change that leads to a Kconfig dependency loop with MBEDTLS_CIPHER_AES_ENABLED. This is supposed to be replaced with a better fix later. Signed-off-by: Andrzej Głąbek <[email protected]>
This needs to be revisited but the newly added dependency on the entropy_generator doesn't work with nrf_security. Signed-off-by: Georgios Vasilakis <[email protected]>
This test cannot be executed with the SDFW Service Framework client started, so disable its initialization. Signed-off-by: Andrzej Głąbek <[email protected]>
Set CONFIG_TFM_PROFILE_TYPE_NOT_SET for boards with TF-M to enable ITS and PS. Enable also CONFIG_PSA_WANT_GENERATE_RANDOM. native_sim target needs CONFIG_PSA_WANT_ALG_ECB_NO_PADDING. Skip checking of private_reserved field in psa_key_attributes if present. Signed-off-by: Juha Ylinen <[email protected]>
Increase maximum ITS asset size. Set CONFIG_TFM_PROFILE_TYPE_NOT_SET for boards with TF-M to enable ITS. Signed-off-by: Juha Ylinen <[email protected]>
ab3e35a to
9e73d89
Compare
Set CONFIG_TFM_PROFILE_TYPE_NOT_SET for boards with TF-M to enable ITS and PS. Enable also CONFIG_PSA_WANT_GENERATE_RANDOM for persistent_key sample. Signed-off-by: Juha Ylinen <[email protected]>
native_sim requires ECB cipher to be enabled. Signed-off-by: Juha Ylinen <[email protected]>
native_sim requires ECB cipher to be enabled. Signed-off-by: Juha Ylinen <[email protected]>
92439bd to
f003ebc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Set CONFIG_TFM_PROFILE_TYPE_NOT_SET for boards with TF-M to enable ITS and PS. Enable also CONFIG_PSA_WANT_GENERATE_RANDOM.
native_sim target needs CONFIG_PSA_WANT_ALG_ECB_NO_PADDING.
Skip checking of private_reserved field in psa_key_attributes if present.