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

Conversation

@jason-simmons
Copy link
Member

If we want to fix forward, this probably just needs a null check on https://github.com/flutter/engine/blob/master/shell/platform/android/io/flutter/plugin/common/BinaryCodec.java#L57

@zanderso
Copy link
Member Author

Let's go by the book and do the revert. The NPE is potentially just the first issue that the framework tree tests hit.

@zanderso
Copy link
Member Author

Landing revert over resolved infra failures.

@zanderso zanderso merged commit 8e439b1 into master May 28, 2021
@zanderso zanderso deleted the revert-26331-platform-channels-direct-buffer branch May 28, 2021 02:12
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 28, 2021
@gaaclarke
Copy link
Member

Thanks, yea missing null check. If only we had null safety in java. I'll reland with a fix/test.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants