Skip to content

Commit 9252099

Browse files
chakrihackerfacebook-github-bot
authored andcommitted
Catch NullPointerException for onRequestPermissionsResult (facebook#39715)
Summary: On Android 13 Devices, we are seeing `NullPointerException`, which should be handled with this ## Changelog: [ANDROID] [FIXED] - Handle Crash for onRequestPermissionsResult Pull Request resolved: facebook#39715 Reviewed By: NickGerleman Differential Revision: D49965583 Pulled By: javache fbshipit-source-id: 8a39049675510f9cca8141c893d93fdb04ba0e25
1 parent 6355703 commit 9252099

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/modules/permissions/PermissionsModule.java

+7-2
Original file line numberDiff line numberDiff line change
@@ -172,8 +172,13 @@ public void invoke(Object... args) {
172172
public boolean onRequestPermissionsResult(
173173
int requestCode, String[] permissions, int[] grantResults) {
174174
try {
175-
mCallbacks.get(requestCode).invoke(grantResults, getPermissionAwareActivity());
176-
mCallbacks.remove(requestCode);
175+
Callback callback = mCallbacks.get(requestCode);
176+
if (callback != null) {
177+
callback.invoke(grantResults, getPermissionAwareActivity());
178+
mCallbacks.remove(requestCode);
179+
} else {
180+
FLog.w("PermissionsModule", "Unable to find callback with requestCode %d", requestCode);
181+
}
177182
return mCallbacks.size() == 0;
178183
} catch (IllegalStateException e) {
179184
FLog.e(

0 commit comments

Comments
 (0)