diff options
| author | Philippe Mathieu-Daudé | 2020-09-19 15:24:35 +0200 |
|---|---|---|
| committer | Peter Maydell | 2020-10-08 16:24:32 +0200 |
| commit | 1cb169b27a7e78176de2101ce7c0a577945c8dec (patch) | |
| tree | bd29a5f8f6e189920e0282a17a0be289fb3e7e89 /include/hw/arm | |
| parent | Merge remote-tracking branch 'remotes/huth-gitlab/tags/pull-request-2020-10-0... (diff) | |
| download | qemu-1cb169b27a7e78176de2101ce7c0a577945c8dec.tar.gz qemu-1cb169b27a7e78176de2101ce7c0a577945c8dec.tar.xz qemu-1cb169b27a7e78176de2101ce7c0a577945c8dec.zip | |
hw/ssi/npcm7xx_fiu: Fix handling of unsigned integer
Fix integer handling issues handling issue reported by Coverity:
hw/ssi/npcm7xx_fiu.c: 162 in npcm7xx_fiu_flash_read()
>>> CID 1432730: Integer handling issues (NEGATIVE_RETURNS)
>>> "npcm7xx_fiu_cs_index(fiu, f)" is passed to a parameter that cannot be negative.
162 npcm7xx_fiu_select(fiu, npcm7xx_fiu_cs_index(fiu, f));
hw/ssi/npcm7xx_fiu.c: 221 in npcm7xx_fiu_flash_write()
218 cs_id = npcm7xx_fiu_cs_index(fiu, f);
219 trace_npcm7xx_fiu_flash_write(DEVICE(fiu)->canonical_path, cs_id, addr,
220 size, v);
>>> CID 1432729: Integer handling issues (NEGATIVE_RETURNS)
>>> "cs_id" is passed to a parameter that cannot be negative.
221 npcm7xx_fiu_select(fiu, cs_id);
Since the index of the flash can not be negative, return an
unsigned type.
Reported-by: Coverity (CID 1432729 & 1432730: NEGATIVE_RETURNS)
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Havard Skinnemoen <hskinnemoen@google.com>
Message-id: 20200919132435.310527-1-f4bug@amsat.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'include/hw/arm')
0 files changed, 0 insertions, 0 deletions
