Fix integer overflows in BL1 FWU code
authorSandrine Bailleux <sandrine.bailleux@arm.com>
Fri, 11 Nov 2016 16:44:37 +0000 (16:44 +0000)
committerDan Handley <dan.handley@arm.com>
Tue, 20 Dec 2016 11:43:10 +0000 (11:43 +0000)
commit949a52d24eea48a58608645b6536ab7158abcbbb
tree5376794e180c176959d8e581f34dca3c88d40669
parent1bfb706851c8606aede8f9f391afde1e5339fbf3
Fix integer overflows in BL1 FWU code

Before adding a base address and a size to compute the end
address of an image to copy or authenticate, check this
won't result in an integer overflow. If it does then consider
the input arguments are invalid.

As a result, bl1_plat_mem_check() can now safely assume the
end address (computed as the sum of the base address and size
of the memory region) doesn't overflow, as the validation is
done upfront in bl1_fwu_image_copy/auth(). A debug assertion
has been added nonetheless in the ARM implementation in order
to help catching such problems, should bl1_plat_mem_check()
be called in a different context in the future.

Fixes TFV-1: Malformed Firmware Update SMC can result in copy
of unexpectedly large data into secure memory

Change-Id: I8b8f8dd4c8777705722c7bd0e8b57addcba07e25
Signed-off-by: Sandrine Bailleux <sandrine.bailleux@arm.com>
Signed-off-by: Dan Handley <dan.handley@arm.com>
bl1/bl1_fwu.c
plat/arm/common/arm_bl1_fwu.c