[Coverity CID :215243] Unintentional integer overflow in drivers/timer/stm32_lptim_timer.c

[zephyrbot]

Static code scan issues found in file:

https://github.com/zephyrproject-rtos/zephyr/tree/c9a2a5e7fb0194bfb168f5aa1a0a83c73f62acb3/drivers/timer/stm32_lptim_timer.c#L313

Category: Integer handling issues
Function: z_timer_cycle_get_32
Component: Drivers
CID: 215243

Details:

307             lp_time += LL_LPTIM_GetAutoReload(LPTIM1) + 1;
308         }
309    
310         lp_time += accumulated_lptim_cnt;
311    
312         /* convert lptim count in a nb of hw cycles with precision */
>>>     CID 215243:  Integer handling issues  (OVERFLOW_BEFORE_WIDEN)
>>>     Potentially overflowing expression "lp_time * (sys_clock_hw_cycles_per_sec() / 32768)" with type "unsigned int" (32 bits, unsigned) is evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type "uint64_t" (64 bits, unsigned).
313         uint64_t ret = lp_time * (sys_clock_hw_cycles_per_sec() / LPTIM_CLOCK);
314    
315         k_spin_unlock(&lock, key);
316    
317         /* convert in hw cycles (keeping 32bit value) */
318         return (uint32_t)(ret);

Please fix or provide comments in coverity using the link:

https://scan9.coverity.com/reports.htm#v32951/p12996.

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.

[github-actions]

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.