diff options
| author | José Fonseca <[email protected]> | 2014-11-07 21:20:25 +0000 |
|---|---|---|
| committer | José Fonseca <[email protected]> | 2014-11-08 10:32:39 +0000 |
| commit | b238c756da5b8c54437e58c2878ecae7467079eb (patch) | |
| tree | 146327aeb91ac81e88e0f06f52e46d77b3026399 /docs/relnotes/10.3.3.html | |
| parent | d268eac3a9c81febc5efcae7fe6854045bfc5144 (diff) | |
util/format: Fix clamping to 32bit integers.
Use clamping constants that guarantee no integer overflows.
As spotted by Chris Forbes.
This causes the code to change as:
- value |= (uint32_t)CLAMP(src[0], 0.0f, 4294967295.0f);
+ value |= (uint32_t)CLAMP(src[0], 0.0f, 4294967040.0f);
- value |= (uint32_t)((int32_t)CLAMP(src[0], -2147483648.0f, 2147483647.0f));
+ value |= (uint32_t)((int32_t)CLAMP(src[0], -2147483648.0f, 2147483520.0f));
Reviewed-by: Roland Scheidegger <[email protected]>
Diffstat (limited to 'docs/relnotes/10.3.3.html')
0 files changed, 0 insertions, 0 deletions