Float-cast-overflow in clampTo |
||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=6240122966900736 Fuzzer: inferno_twister Job Type: linux_ubsan_chrome Platform Id: linux Crash Type: Float-cast-overflow Crash Address: Crash State: clampTo clampTo clampTo<int, Sanitizer: undefined (UBSAN) Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=551565:563900 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6240122966900736 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Jul 26
,
Jul 26
So maybe we have to check for nan in the clamp code?
,
Jul 26
If we invert one of the min/max conditionals it would catch/clamp NaN also. But it's unclear to me how we could end up with NaN in this case, so maybe there's an issue higher up the stack. I guess we should either decide NaNs are valid input for clamping, and handle them, or assert the inputs are not NaN.
,
Jul 26
I agree we should fix this up the stack. Leaving it open for someone to look into that.
,
Aug 15
|
||||
►
Sign in to add a comment |
||||
Comment 1 by ClusterFuzz
, Jul 26Labels: Test-Predator-Auto-Components