suzanne.soy/racket
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

avoid undefined behavior in hashing inexacts

Browse Source 
Casting a negative floating-point number to an unsigned integer
is not ok. Corece to a signed integer, first.

Thanks to John Regehr for help.
This commit is contained in:
Matthew Flatt 2015-09-18 09:25:26 -06:00
parent b7bcd4f687
commit 0b9cda5018
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
racket/src/racket/src/hash.c
View File

@ -1117,7 +1117,7 @@ XFORM_NONGCING static uintptr_t dbl_hash_val(double d)
d = frexp(d, &e); d = frexp(d, &e);
} }
return ((uintptr_t)(d * (1 << 30))) + e; return ((uintptr_t)(intptr_t)(d * (1 << 30))) + (uintptr_t)e;
} }
XFORM_NONGCING static uintptr_t dbl_hash2_val(double d) XFORM_NONGCING static uintptr_t dbl_hash2_val(double d)