Google Git
Sign in
qemu / qemu / 7baa9c39fc3d525216f0cedcfda5374c26d50e80
commit7baa9c39fc3d525216f0cedcfda5374c26d50e80[log] [tgz]
authorPhilippe Mathieu-Daudé <philmd@linaro.org>Mon Aug 11 11:31:08 2025 +0200
committerPhilippe Mathieu-Daudé <philmd@linaro.org>Tue Sep 02 17:57:05 2025 +0200
treed7d3411ad7643b88cc4d7aaf7d09fc9fdb7a530b
parentc5ade4f9d289fed61df7a04950f8f607e26353e6 [diff]
hw/scsi/mptsas: Avoid silent integer truncation in MPI_FUNC_IOC_INIT

For the MaxDevices 8-bit field of the request / response structures
of the MPI_FUNCTION_IOC_INIT command, the 0x00 value means "max 256
devices". This is not a problem because when max_devices=256, its
value (0x100), being casted to a uint8_t, is truncated to 0x00.
However Coverity complains for an "Overflowed constant". Fix by
re-using the request fields in the response, since they are not
modified and use the same types.

Fix: Coverity 1547736 (Overflowed constant)
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20250811095550.93655-1-philmd@linaro.org>
1 file changed
tree: d7d3411ad7643b88cc4d7aaf7d09fc9fdb7a530b
  1. .github/
  2. .gitlab/
  3. .gitlab-ci.d/
  4. accel/
  5. audio/
  6. authz/
  7. backends/
  8. block/
  9. bsd-user/
  10. chardev/
  11. common-user/
  12. configs/
  13. contrib/
  14. crypto/
  15. disas/
  16. docs/
  17. dump/
  18. ebpf/
  19. fpu/
  20. fsdev/
  21. gdb-xml/
  22. gdbstub/
  23. host/
  24. hw/
  25. include/
  26. io/
  27. libdecnumber/
  28. linux-headers/
  29. linux-user/
  30. migration/
  31. monitor/
  32. nbd/
  33. net/
  34. pc-bios/
  35. plugins/
  36. po/
  37. python/
  38. qapi/
  39. qga/
  40. qobject/
  41. qom/
  42. replay/
  43. roms/
  44. rust/
  45. scripts/
  46. scsi/
  47. semihosting/
  48. stats/
  49. storage-daemon/
  50. stubs/
  51. subprojects/
  52. system/
  53. target/
  54. tcg/
  55. tests/
  56. tools/
  57. trace/
  58. ui/
  59. util/
  60. .b4-config
  61. .dir-locals.el
  62. .editorconfig
  63. .exrc
  64. .gdbinit
  65. .git-blame-ignore-revs
  66. .gitattributes
  67. .gitignore
  68. .gitlab-ci.yml
  69. .gitmodules
  70. .gitpublish
  71. .mailmap
  72. .patchew.yml
  73. .readthedocs.yml
  74. .travis.yml
  75. block.c
  76. blockdev-nbd.c
  77. blockdev.c
  78. blockjob.c
  79. clippy.toml
  80. configure
  81. COPYING
  82. COPYING.LIB
  83. cpu-common.c
  84. cpu-target.c
  85. event-loop-base.c
  86. gitdm.config
  87. hmp-commands-info.hx
  88. hmp-commands.hx
  89. iothread.c
  90. job-qmp.c
  91. job.c
  92. Kconfig
  93. Kconfig.host
  94. LICENSE
  95. MAINTAINERS
  96. Makefile
  97. meson.build
  98. meson_options.txt
  99. module-common.c
  100. os-posix.c
  101. os-wasm.c
  102. os-win32.c
  103. page-vary-common.c
  104. page-vary-target.c
  105. pythondeps.toml
  106. qemu-bridge-helper.c
  107. qemu-edid.c
  108. qemu-img-cmds.hx
  109. qemu-img.c
  110. qemu-io-cmds.c
  111. qemu-io.c
  112. qemu-keymap.c
  113. qemu-nbd.c
  114. qemu-options.hx
  115. qemu.nsi
  116. qemu.sasl
  117. README.rst
  118. replication.c
  119. target-info-stub.c
  120. target-info.c
  121. trace-events
  122. VERSION
  123. version.rc