Fix potential panic (#2755)

* Fix potential panic * Update CHANGELOG.md * Use checked_div --------- Co-authored-by: Xiaopeng Li <[email protected]>
rust-windowing · May 4, 2023 · d37dac8 · d37dac8
1 parent 64a97c1
commit d37dac8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,8 @@ And please only add new entries to the top of this list, right below the `# Unre
 
 # Unreleased
 
+- On macOS, fixed potential panic when getting refresh rate.
+
 # 0.28.3
 
 - Fix macOS memory leaks.

diff --git a/src/platform_impl/macos/monitor.rs b/src/platform_impl/macos/monitor.rs
@@ -230,7 +230,9 @@ impl MonitorHandle {
                 return None;
             }
 
-            Some((time.time_scale as i64 / time.time_value * 1000) as u32)
+            (time.time_scale as i64)
+                .checked_div(time.time_value)
+                .map(|v| (v * 1000) as u32)
         }
     }