Skip to content
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

Bandaid fix #714 #786

Closed
wants to merge 3 commits into from
Closed

Bandaid fix #714 #786

wants to merge 3 commits into from

Conversation

MrCsabaToth
Copy link

@MrCsabaToth MrCsabaToth commented Mar 10, 2021

The channel is a type of MethodChannel and is explicitly nulled out in the tearDown and is not initialized before setup. If we strictly look at the crash call stack the NPE comes inside the run function after android.app.Activity.runOnUiThread. Maybe this is some race condition before the setup or happens after a tearDown. But it regularly comes from apps installed in the wild.

MrCsabaToth added a commit to TrackMyIndoorWorkout/TrackMyIndoorWorkout that referenced this pull request Mar 11, 2021
… by pauldemarco/flutter_blue#786 in my repo. Crash happens a lot out in the wild, release-only
PR from main repo for null safety
Refresh patch-1 branch with null safety newbies from master
C4ptainCrunch added a commit to hastakecorp/flutter_blue that referenced this pull request Jun 8, 2021
Squashed commit of the following:

commit bb6f0d1
Merge: 3a8c3ab 4670349
Author: Csaba Toth <csaba.toth.us@outlook.com>
Date:   Thu Mar 25 22:11:03 2021 +0100

    Merge pull request #2 from MrCsabaToth/master

    Refresh patch-1 branch with null safety newbies from master

commit 4670349
Merge: c19bb9a 7c7a86c
Author: Csaba Toth <csaba.toth.us@outlook.com>
Date:   Thu Mar 25 22:09:09 2021 +0100

    Merge pull request #1 from pauldemarco/master

    PR from main repo for null safety

commit 3a8c3ab
Author: Csaba Toth <csaba.toth.us@outlook.com>
Date:   Wed Mar 10 10:58:21 2021 -0800

    Bandaid fix #714
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant