New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GetIteratorFlattenable should check that iterator is an Object. #247
Comments
Wouldn’t GetV check that? |
@ljharb no, GetV performs a ToObject conversion, so if the returned iterator is an object-coercible primitive value and there's a callable FWIW, GetIterator has a check for this case: "4. If iterator is not an Object, throw a TypeError exception." |
aha, thanks, good catch |
michaelficarra
added a commit
that referenced
this issue
Dec 1, 2022
michaelficarra
added a commit
that referenced
this issue
Dec 1, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
GetIteratorFlattenable
apparently is missing a check that the iterator returned by Call(method, obj) (i.e.@@iterator()
or@@asyncIterator()
) is an Object, as required by Iterator Record:The text was updated successfully, but these errors were encountered: