Bug 268271

Summary:	[JSC] Array destructuring assignment should close the iterator if PutValue or initializer evaluation throws
Product:	WebKit	Reporter:	Alexey Shvayka <ashvayka>
Component:	JavaScriptCore	Assignee:	Alexey Shvayka <ashvayka>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	All
OS:	All
See Also:	https://bugs.webkit.org/show_bug.cgi?id=268272
Bug Depends on:
Bug Blocks:	268026

Alexey Shvayka

Reported 2024-01-28 21:26:30 PST

Related test262 failures: * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-lref-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close.js * test/language/statements/for-of/dstr/array-rest-lref-err.js

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2024-01-30 14:38:49 PST

<rdar://problem/121960887>

Alexey Shvayka

Comment 2 2024-08-27 14:11:56 PDT

Pull request: https://github.com/WebKit/WebKit/pull/32778

EWS

Comment 3 2024-09-10 15:33:17 PDT

Committed 283443@main (863e4a5d13af): <https://commits.webkit.org/283443@main> Reviewed commits have been landed. Closing PR #32778 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.