Commit 03bc6e0d authored by Dominik Ermel's avatar Dominik Ermel Committed by Henrik Brix Andersen
Browse files

storage/stream_flash: Fix range check in stream_flash_erase_page 

Added check where stream_flash_erase_page checks if requested
offset is actually within stream flash designated area.

Fixes #79800

Signed-off-by: default avatarDominik Ermel <dominik.ermel@nordicsemi.no>
(cherry picked from commit 8714c172)
parent 6aeb7a2b

include/zephyr/storage/stream_flash.h

+2 −1
Changes for include/zephyr/storage/stream_flash.h: 2 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -141,7 +141,8 @@ int stream_flash_erase_page(struct stream_flash_ctx *ctx, off_t off); 
 * @param settings_key key to use with the settings module for loading

 *                     the stream write progress

 *

 * @return non-negative on success, negative errno code on fail

 * @return non-negative on success, -ERANGE in case when @p off is out

 * of area designated for stream or negative errno code on fail

 */

int stream_flash_progress_load(struct stream_flash_ctx *ctx,

			       const char *settings_key);

subsys/storage/stream/stream_flash.c

+5 −0
Changes for subsys/storage/stream/stream_flash.c: 5 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -79,6 +79,11 @@ int stream_flash_erase_page(struct stream_flash_ctx *ctx, off_t off) 
	int rc;

	struct flash_pages_info page;



	if (off < ctx->offset || (off - ctx->offset) >= ctx->available) {

		LOG_ERR("Offset out of designated range");

		return -ERANGE;

	}



	rc = flash_get_page_info_by_offs(ctx->fdev, off, &page);

	if (rc != 0) {

		LOG_ERR("Error %d while getting page info", rc);

CRA Git | Maintained and supported by SUSTech CRA and CCSE