-
Notifications
You must be signed in to change notification settings - Fork 30.8k
RenderUiKitView _handleGlobalPointerEvent is not checking for null size #83481
Copy link
Copy link
Closed
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work lista: platform-viewsEmbedding Android/iOS views in Flutter appsEmbedding Android/iOS views in Flutter appsc: crashStack traces logged to the consoleStack traces logged to the consolecustomer: googleVarious Google teamsVarious Google teamsfound in release: 3.16Found to occur in 3.16Found to occur in 3.16found in release: 3.17Found to occur in 3.17Found to occur in 3.17found in release: 3.32Found to occur in 3.32Found to occur in 3.32frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.has reproducible stepsThe issue has been confirmed reproducible and is ready to work onThe issue has been confirmed reproducible and is ready to work onplatform-iosiOS applications specificallyiOS applications specificallyr: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
Description
Metadata
Metadata
Assignees
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work lista: platform-viewsEmbedding Android/iOS views in Flutter appsEmbedding Android/iOS views in Flutter appsc: crashStack traces logged to the consoleStack traces logged to the consolecustomer: googleVarious Google teamsVarious Google teamsfound in release: 3.16Found to occur in 3.16Found to occur in 3.16found in release: 3.17Found to occur in 3.17Found to occur in 3.17found in release: 3.32Found to occur in 3.32Found to occur in 3.32frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.has reproducible stepsThe issue has been confirmed reproducible and is ready to work onThe issue has been confirmed reproducible and is ready to work onplatform-iosiOS applications specificallyiOS applications specificallyr: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
Type
Fields
No fields configured for issues without a type.
Steps to Reproduce
Sadly i have no working reproducer. Only Error reports in crashlytics.
In this function the size of the render box is used without checking if it is null:
flutter/packages/flutter/lib/src/rendering/platform_view.dart
Line 382 in 1cf492f
It looks like there is a way for RenderUiKitView to receiver touch events bevor it is layouted and the size is still null:
flutter/packages/flutter/lib/src/rendering/box.dart
Line 1951 in e905dd2
Expected results:
No error Report generated.
Actual results: