Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Release acquired typed data before calling Dart_SetReturnValue.#17729

Merged
fluttergithubbot merged 1 commit intoflutter:masterfrom
mraleph:fix54433
Apr 15, 2020
Merged

Release acquired typed data before calling Dart_SetReturnValue.#17729
fluttergithubbot merged 1 commit intoflutter:masterfrom
mraleph:fix54433

Conversation

@mraleph
Copy link
Member

@mraleph mraleph commented Apr 15, 2020

Otherwise we are going to hit an assertion in debug builds, as
Dart API should not be used between Dart_TypedDataAcquireData and
Dart_TypedDataReleaseData.

Fixes flutter/flutter#54433

Otherwise we are going to hit an assertion in debug builds, as
Dart API should not be used between Dart_TypedDataAcquireData and
Dart_TypedDataReleaseData.
@jason-simmons jason-simmons added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 15, 2020
@fluttergithubbot fluttergithubbot merged commit 9afc6fa into flutter:master Apr 15, 2020
@mraleph mraleph deleted the fix54433 branch April 15, 2020 21:32
goderbauer pushed a commit to goderbauer/engine that referenced this pull request Apr 16, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ASSERT: ../../third_party/dart/runtime/vm/dart_api_impl.cc: 5191: error: expected: arguments->thread()->no_callback_scope_depth() == 0

4 participants

Comments