Skip to content

Commit

Permalink
boot: Reduce repeating code in boot_decrypt_and_copy_image_to_sram
Browse files Browse the repository at this point in the history
There was not really needed repetition of code in if-else
block; common code has been moved out and the block has been
reduced.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
  • Loading branch information
de-nordic committed Jul 18, 2024
1 parent c4b89ba commit e7dfbb2
Showing 1 changed file with 4 additions and 10 deletions.
14 changes: 4 additions & 10 deletions boot/bootutil/src/loader.c
Original file line number Diff line number Diff line change
Expand Up @@ -2769,21 +2769,15 @@ boot_decrypt_and_copy_image_to_sram(struct boot_loader_state *state,
cur_dst = ram_dst + bytes_copied;
blk_sz = chunk_sz;
idx = 0;
blk_off = ((bytes_copied) - hdr->ih_hdr_size) & 0xf;
if (bytes_copied + chunk_sz > tlv_off) {
/* Going over TLV section
* Part of the chunk is encrypted payload */
blk_off = ((bytes_copied) - hdr->ih_hdr_size) & 0xf;
blk_sz = tlv_off - (bytes_copied);
boot_encrypt(BOOT_CURR_ENC(state), image_index, area_id,
(bytes_copied + idx) - hdr->ih_hdr_size, blk_sz,
blk_off, cur_dst);
} else {
/* Image encrypted payload section */
blk_off = ((bytes_copied) - hdr->ih_hdr_size) & 0xf;
boot_encrypt(BOOT_CURR_ENC(state), image_index, area_id,
(bytes_copied + idx) - hdr->ih_hdr_size, blk_sz,
blk_off, cur_dst);
}
boot_encrypt(BOOT_CURR_ENC(state), image_index, area_id,
(bytes_copied + idx) - hdr->ih_hdr_size, blk_sz,
blk_off, cur_dst);

bytes_copied += chunk_sz;
}
Expand Down

0 comments on commit e7dfbb2

Please sign in to comment.