diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2018-11-06 15:33:30 +0000 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2018-11-08 13:24:35 +0000 |
| commit | 0b57007755c2ee442740b5321ffc7072aed88714 (patch) | |
| tree | 56186f8fe222e8007119c7d0eb0b1398348570e1 /hw | |
| parent | fa27257432689e8927cb993b251d380d654dcc86 (diff) | |
| download | qemu-0b57007755c2ee442740b5321ffc7072aed88714.zip qemu-0b57007755c2ee442740b5321ffc7072aed88714.tar.gz qemu-0b57007755c2ee442740b5321ffc7072aed88714.tar.bz2 | |
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')
0 files changed, 0 insertions, 0 deletions