Commit 42d75f1a254653cbd4d441b300248c37d5f8d5a2 | zig

author	ominitay <37453713+ominitay@users.noreply.github.com> 2022-03-10 19:53:28+00:00
committer	Veikka Tuominen <git@vexu.eu> 2022-03-12 10:23:57+02:00
log	42d75f1a254653cbd4d441b300248c37d5f8d5a2
tree	92bdfd7f377304e91b72fff0dd3785ae57528039
parent	b3259b47ad03592a156841c87e69421da05e37f3

Fixes integer overflow caused by cast from maxInt(u32) as an f32 to u32.

1 files changed, 5 insertions(+), 4 deletions(-)

lib/std/math.zig+5-4

 @@ -1190,18 +1190,18 @@ pub fn lossyCast(comptime T: type, value: anytype) T {
         .Int => {
             switch (@typeInfo(@TypeOf(value))) {
                 .Int, .ComptimeInt => {
-                    if (value > maxInt(T)) {
+                    if (value >= maxInt(T)) {
                         return @as(T, maxInt(T));
-                    } else if (value < minInt(T)) {
+                    } else if (value <= minInt(T)) {
                         return @as(T, minInt(T));
                     } else {
                         return @intCast(T, value);
                     }
                 },
                 .Float, .ComptimeFloat => {
-                    if (value > maxInt(T)) {
+                    if (value >= maxInt(T)) {
                         return @as(T, maxInt(T));
-                    } else if (value < minInt(T)) {
+                    } else if (value <= minInt(T)) {
                         return @as(T, minInt(T));
                     } else {
                         return @floatToInt(T, value);
 @@ -1218,6 +1218,7 @@ test "lossyCast" {
     try testing.expect(lossyCast(i16, 70000.0) == @as(i16, 32767));
     try testing.expect(lossyCast(u32, @as(i16, -255)) == @as(u32, 0));
     try testing.expect(lossyCast(i9, @as(u32, 200)) == @as(i9, 200));
+    try testing.expect(lossyCast(u32, @as(f32, maxInt(u32))) == maxInt(u32));
 }
 test "f64_min" {