Skip to content

nrf_securit: cracen: Fix input validation ordering - #28636

Merged
rlubos merged 1 commit into
nrfconnect:mainfrom
degjorva:lm20-ctr-aes-fix
May 26, 2026
Merged

nrf_securit: cracen: Fix input validation ordering#28636
rlubos merged 1 commit into
nrfconnect:mainfrom
degjorva:lm20-ctr-aes-fix

Conversation

@degjorva

@degjorva degjorva commented May 8, 2026

Copy link
Copy Markdown
Contributor

AES-CTR should not check for unprocessed_input
as it can handle individual blocks smaller than
block size.

@degjorva
degjorva requested a review from a team as a code owner May 8, 2026 12:04
AES-CTR should not check for unprocessed_input as it
can handle individual blocks smaller than block size.

Signed-off-by: Dag Erik Gjørvad <dag.erik.gjorvad@nordicsemi.no>
@degjorva
degjorva force-pushed the lm20-ctr-aes-fix branch from 052e375 to b20236c Compare May 8, 2026 12:12
@NordicBuilder

NordicBuilder commented May 8, 2026

Copy link
Copy Markdown
Contributor

CI Information

To view the history of this post, click the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: b20236c32825841a88648b3c3fcdef93ba82db40

more details

sdk-nrf:

PR head: b20236c32825841a88648b3c3fcdef93ba82db40
merge base: df915ed1ce0e433a9cdbbc8fc2bc96567b63967e
target head (main): c83efed9d47475ff0e285b3c8a6bc119d773575c
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1)
subsys
│  ├── nrf_security
│  │  ├── src
│  │  │  ├── drivers
│  │  │  │  ├── cracen
│  │  │  │  │  ├── cracen_sw
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │ cracen_sw_cipher.c

Outputs:

Toolchain

Version: eb11c27a58
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:eb11c27a58_5ea73affbf

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 2531
  • ✅ Integration tests
    • ✅ test_ble_nrf_config
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf_crypto
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-tfm
    • ✅ test-sdk-find-my
    • ✅ test-sdk-mcuboot
    • ✅ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@degjorva degjorva added the bugfix Fixes a known bug label May 11, 2026
@degjorva degjorva added this to the 3.4.0 milestone May 11, 2026

@frkv frkv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@frkv frkv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shanthanordic shanthanordic left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved for scope

@rlubos
rlubos merged commit 9fa8979 into nrfconnect:main May 26, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a known bug no-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants