xyz-modem: Fix crash after cancelling transfer
authorPali Rohár <pali@kernel.org>
Tue, 3 Aug 2021 14:28:38 +0000 (16:28 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 3 Sep 2021 18:32:40 +0000 (14:32 -0400)
commit1e7478461bb4e8842f1ca8e5ffb5a441041b0753
tree2760a8ed96bc8cbd1c9325c51d39eb659680a542
parent00179319714fd2076cf81f49de357ee699672f31
xyz-modem: Fix crash after cancelling transfer

Variable xyz.len is set to -1 on error. At the end xyzModem_stream_read()
function calls memcpy() with length from variable xyz.len. If this variable
is set to -1 then value passed to memcpy is casted to unsigned value, which
means to copy whole address space. Which then cause U-Boot crash. E.g. on
arm64 it cause CPU crash: "Synchronous Abort" handler, esr 0x96000006

Fix this issue by checking that value stored in xyz.len is valid prior
trying to use it.

Signed-off-by: Pali Rohár <pali@kernel.org>
Acked-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
common/xyzModem.c