Skip to content
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

Backport 79830 to v2.7 branch: Add stream_flash_erase_page range check #80615

Open
wants to merge 1 commit into
base: v2.7-branch
Choose a base branch
from

Conversation

de-nordic
Copy link
Collaborator

@de-nordic de-nordic commented Oct 30, 2024

Fixes a bug where: stream_flash: stream_flash_erase_page does not check whether requested offset is in range of stream flash owned area (#79800)

Fixes #80460
Failed backport #80460

@github-actions github-actions bot added area: Tests Issues related to a particular existing or missing test area: Storage Storage subsystem area: API Changes to public APIs labels Oct 30, 2024
@de-nordic
Copy link
Collaborator Author

@nashif I should drop the test commit, right?

@de-nordic de-nordic force-pushed the backport-79830-to-v2.7-branch branch from 9735411 to a5471aa Compare October 30, 2024 15:32
@github-actions github-actions bot removed the area: Tests Issues related to a particular existing or missing test label Oct 30, 2024
@de-nordic de-nordic self-assigned this Oct 30, 2024
Added check where stream_flash_erase_page checks if requested
offset is actually within stream flash designated area.

Fixes #79800

Signed-off-by: Dominik Ermel <[email protected]>
(cherry picked from commit 8714c17)
@de-nordic de-nordic force-pushed the backport-79830-to-v2.7-branch branch from a5471aa to f67cbe0 Compare October 30, 2024 16:04
@github-actions github-actions bot added the area: Tests Issues related to a particular existing or missing test label Oct 30, 2024
@de-nordic de-nordic assigned nashif and unassigned de-nordic Oct 30, 2024
@de-nordic de-nordic force-pushed the backport-79830-to-v2.7-branch branch from f67cbe0 to f173c1e Compare November 4, 2024 20:26
@github-actions github-actions bot removed the area: Tests Issues related to a particular existing or missing test label Nov 4, 2024
@nashif
Copy link
Member

nashif commented Nov 4, 2024

please put something useful in the PR title, PR numbers do not tell me anything.

@de-nordic de-nordic changed the title Backport 79830 to v2.7 branch Backport 79830 to v2.7 branch: Add stream_flash_erase_page range check Nov 5, 2024
@de-nordic
Copy link
Collaborator Author

@nashif I am sorry, but it seems that CI for this will always fail, due to missing runners or out of date configuration. I can not do anything about this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: API Changes to public APIs area: Storage Storage subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants