diff options
author | Peter Maydell | 2018-11-06 16:33:30 +0100 |
---|---|---|
committer | Peter Maydell | 2018-11-08 14:24:35 +0100 |
commit | 0b57007755c2ee442740b5321ffc7072aed88714 (patch) | |
tree | 56186f8fe222e8007119c7d0eb0b1398348570e1 /hw/pci-host/ppce500.c | |
parent | Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (diff) | |
download | qemu-0b57007755c2ee442740b5321ffc7072aed88714.tar.gz qemu-0b57007755c2ee442740b5321ffc7072aed88714.tar.xz qemu-0b57007755c2ee442740b5321ffc7072aed88714.zip |
replay: Exit on errors reading from replay log
Currently replay_get_byte() does not check for an error
from getc(). Coverity points out (CID 1390622) that this
could result in unexpected behaviour (such as looping
forever, if we use the replay_get_dword() return value
for a loop count). We don't expect reads from the replay
log to fail, and if they do there is no way we can
continue. So make them fatal errors.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru>
Message-id: 20181106153330.5139-1-peter.maydell@linaro.org
Diffstat (limited to 'hw/pci-host/ppce500.c')
0 files changed, 0 insertions, 0 deletions