mgmt/mcumgr/lib: Remove redundant offset check

The offset is checked, for correctness, before the function
img_mgmt_impl_write_image_data is called, so it is redundant to
do the same check within it.

Signed-off-by: Dominik Ermel <[email protected]>

Author: de-nordic

subsys/mgmt/mcumgr/lib/cmd/img_mgmt/src/zephyr_img_mgmt.c

@@ -370,11 +370,6 @@ img_mgmt_impl_write_image_data(unsigned int offset, const void *data, unsigned i
 		}
 	}
 
-	if (offset != flash_img_bytes_written(ctx) + ctx->stream.buf_bytes) {
-		rc = MGMT_ERR_EUNKNOWN;
-		goto out;
-	}
-
 	if (flash_img_buffered_write(ctx, data, num_bytes, last) != 0) {
 		rc = MGMT_ERR_EUNKNOWN;
 		goto out;
@@ -402,10 +397,6 @@ img_mgmt_impl_write_image_data(unsigned int offset, const void *data, unsigned i
 		}
 	}
 
-	if (offset != ctx.stream.bytes_written + ctx.stream.buf_bytes) {
-		return MGMT_ERR_EUNKNOWN;
-	}
-
 	if (flash_img_buffered_write(&ctx, data, num_bytes, last) != 0) {
 		return MGMT_ERR_EUNKNOWN;
 	}