diff options
| author | Cédric Le Goater <clg@redhat.com> | 2024-07-12 17:13:15 +0200 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2024-07-21 07:46:38 +0200 |
| commit | 13951ccfcdf0f31902a93859506ccf8c0ef66583 (patch) | |
| tree | 6d56e4a109b1d559f59f84a202546dd6bd74b1c3 /hw/i2c/imx_i2c.c | |
| parent | a87a7c449e532130d4fa8faa391ff7e1f04ed660 (diff) | |
| download | focaccia-qemu-13951ccfcdf0f31902a93859506ccf8c0ef66583.tar.gz focaccia-qemu-13951ccfcdf0f31902a93859506ccf8c0ef66583.zip | |
aspeed/smc: Fix possible integer overflow
Coverity reports a possible integer overflow because routine aspeeed_smc_hclk_divisor() has a codepath returning 0, which could lead to an integer overflow when computing variable 'hclk_shift' in the caller aspeed_smc_dma_calibration(). The value passed to aspeed_smc_hclk_divisor() is always between 0 and 15 and, in this case, there is always a matching hclk divisor. Remove the return 0 and use g_assert_not_reached() instead. Fixes: Coverity CID 1547822 Suggested-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Cédric Le Goater <clg@redhat.com> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'hw/i2c/imx_i2c.c')
0 files changed, 0 insertions, 0 deletions