Bug 248784 - Avoid uint64_t overflow in Decimal::operator/()
Summary: Avoid uint64_t overflow in Decimal::operator/()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
: 247514 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-12-05 14:06 PST by Ahmad Saleem
Modified: 2024-01-23 16:52 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2022-12-05 14:06:16 PST
Hi Team,

While going through Blink's commit, I found another failing test case:

Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=191294

Failing Test Case - https://jsfiddle.net/rerich/pn1o3wvn/

(^ First range is at 'zero' compared to Chrome / Blink).

Just wanted to raise this bug so it can be merged or fixed.

Unfortunately, I will not be able to do it because the test is in C++ and I am not able to do it in this bug 247514, appreciate if someone can fix both in one go.

Thanks!
Comment 1 Sam Sneddon [:gsnedders] 2022-12-08 06:45:52 PST
> Unfortunately, I will not be able to do it because the test is in C++

While it's not ideal, importing the failing test case linked above (with input type=range), converting it to a reftest, would be better than nothing. Unless this ends up exposed through the DOM API?
Comment 2 Radar WebKit Bug Importer 2022-12-12 14:07:16 PST
<rdar://problem/103269252>
Comment 3 Ahmad Saleem 2022-12-20 08:40:32 PST
(In reply to Sam Sneddon [:gsnedders] from comment #1)
> > Unfortunately, I will not be able to do it because the test is in C++
> 
> While it's not ideal, importing the failing test case linked above (with
> input type=range), converting it to a reftest, would be better than nothing.
> Unless this ends up exposed through the DOM API?

Managed to figure out the ref-test.

PR - https://github.com/WebKit/WebKit/pull/7908
Comment 4 EWS 2022-12-20 18:49:32 PST
Committed 258174@main (d16c8e20b19e): <https://commits.webkit.org/258174@main>

Reviewed commits have been landed. Closing PR #7908 and removing active labels.
Comment 5 Brent Fulgham 2024-01-23 16:52:16 PST
*** Bug 247514 has been marked as a duplicate of this bug. ***