From 3f054530e46651e83a64b59f82c62d32ddb4ebfb Mon Sep 17 00:00:00 2001
From: Matthias Ringwald <matthias@ringwald.ch>
Date: Fri, 15 May 2020 16:06:33 +0200
Subject: [PATCH] btstack_hal_flash_memory: fix write of 0xff bytes to
 simulated flash, fix #283

---
 CHANGELOG.md                              | 1 +
 platform/embedded/hal_flash_bank_memory.c | 5 ++++-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 6dbc2804f..220b13614 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - pbap_client: fix PBAP UUID len on connect message
 - sm: fix secure connection pairing as peripheral when local user confirmation happens after remote one
 - A2DP Source: only connect to remote sink stream endpoints
+- btstack_hal_flash_memory: fix write of 0xff bytes to simulated flash
 
 ### Added
 - example/le_mitm: MITM implementation that forwards ATT PDUs and allows for pairing
diff --git a/platform/embedded/hal_flash_bank_memory.c b/platform/embedded/hal_flash_bank_memory.c
index 70926e5e3..2315375b1 100644
--- a/platform/embedded/hal_flash_bank_memory.c
+++ b/platform/embedded/hal_flash_bank_memory.c
@@ -89,7 +89,10 @@ static void hal_flash_bank_memory_write(void * context, int bank, uint32_t offse
 	int i;
 	for (i=0;i<size;i++){
 		// write 0xff doesn't change anything
-		if (data[i] == 0xff) continue;
+		if (data[i] == 0xff) {
+		    offset++;
+		    continue;
+		}
 		// writing something other than 0x00 is only allowed once
 		if (self->banks[bank][offset] != 0xff && data[i] != 0x00){
 			log_error("Error: offset %u written twice. Data: 0x%02x!", offset+i, data[i]);