Thread (7 messages) 7 messages, 3 authors, 2025-05-10

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

From: Neil Armstrong <neil.armstrong@linaro.org>
Date: 2025-05-09 18:52:27
Also in: dri-devel, linux-amlogic, lkml

On 09/05/2025 20:27, Martin Blumenstingl wrote:
On Fri, May 9, 2025 at 5:35 PM [off-list ref] wrote:
quoted
On 05/05/2025 20:43, I Hsin Cheng wrote:
quoted
Coverity scan reported the usage of "mode->clock * 1000" may lead to
integer overflow. Use "1000ULL" instead of "1000"
when utilizing it to avoid potential integer overflow issue.

Link: https://scan5.scan.coverity.com/#/project-view/10074/10063?selectedIssue=1646759
Signed-off-by: I Hsin Cheng <redacted>
Could you add a Fixes tag ?
Fixes: 1017560164b6 ("drm/meson: use unsigned long long / Hz for
frequency types")

Neil, can you add this while applying or do we need a v3?
Will add while applying.

Thanks,
Neil

Best regards,
Martin
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help