[riot-notifications] [RIOT-OS/RIOT] periph/rtt: fix overflow in tick conversion macros (#15431)

Alexandre Abadie notifications at github.com
Mon Nov 16 09:20:35 CET 2020


@aabadie commented on this pull request.



>  
 /**
  * @brief       Convert minutes to rtt ticks
  * @param[in]   min     number of minutes
  * @return              rtt ticks
  */
-#define RTT_MIN_TO_TICKS(min)   ( RTT_SEC_TO_TICKS((min) * 60) )
+#define RTT_MIN_TO_TICKS(min)   (RTT_SEC_TO_TICKS((min) * 60) )

```suggestion
#define RTT_MIN_TO_TICKS(min)   (RTT_SEC_TO_TICKS((min) * 60))
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15431#pullrequestreview-531059572
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201116/fe3662f1/attachment.htm>


More information about the notifications mailing list