Bug 262046 - Treat huge 'repeatCount' values as invalid
Summary: Treat huge 'repeatCount' values as invalid
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://wpt.live/svg/animations/repeat...
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-09-25 08:28 PDT by Ahmad Saleem
Modified: 2023-09-25 16:43 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-09-25 08:28:05 PDT 
Hi Team,

Came across another failing WPT test and also Blink merge:

Blink Commit: https://chromium.googlesource.com/chromium/src/+/8fcb33ec493a697c5973fed3fa3afaf77bf626db

WebKit Source Code: https://searchfox.org/wubkat/source/Source/WebCore/svg/animation/SVGSMILElement.cpp#722

^ Just adding '&& std::isfinite(result)'.

Test Case: In URL (Timeout in Safari / WebKit ToT)

Local build confirmed that it make us pass the test case.

Just raising so can fix it.
Comment 1 EWS 2023-09-25 16:42:03 PDT 
Committed 268420@main (67ce0e567581): <https://commits.webkit.org/268420@main>

Reviewed commits have been landed. Closing PR #18165 and removing active labels.
Comment 2 Radar WebKit Bug Importer 2023-09-25 16:43:13 PDT 
<rdar://problem/116023138>