From 63f1b8fb6265e2776b9f3ca98ad11fabe136b7bb Mon Sep 17 00:00:00 2001
From: David Martos <davidmartos96@gmail.com>
Date: Mon, 5 Dec 2022 18:36:19 +0100
Subject: [PATCH] [camera] Handle empty grantResults on permission request
 (#6758)

* [camera] Handle empty grantResults on permission request

* update changelog and add test

* fix typo

* format

* Update packages/camera/camera_android/CHANGELOG.md

Co-authored-by: Camille Simon <43054281+camsim99@users.noreply.github.com>

Co-authored-by: Camille Simon <43054281+camsim99@users.noreply.github.com>
---
 packages/camera/camera_android/CHANGELOG.md        |  4 ++++
 .../flutter/plugins/camera/CameraPermissions.java  |  4 +++-
 .../plugins/camera/CameraPermissionsTest.java      | 14 ++++++++++++++
 packages/camera/camera_android/pubspec.yaml        |  2 +-
 4 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/packages/camera/camera_android/CHANGELOG.md b/packages/camera/camera_android/CHANGELOG.md
index a62d3169e409..750145ea1cb4 100644
--- a/packages/camera/camera_android/CHANGELOG.md
+++ b/packages/camera/camera_android/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.10.0+5
+
+* Fixes `ArrayIndexOutOfBoundsException` when the permission request is interrupted.
+
 ## 0.10.0+4
 
 * Upgrades `androidx.annotation` version to 1.5.0.
diff --git a/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraPermissions.java b/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraPermissions.java
index 4441751e19cf..ee8fa5a71a16 100644
--- a/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraPermissions.java
+++ b/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/CameraPermissions.java
@@ -105,7 +105,9 @@ public boolean onRequestPermissionsResult(int id, String[] permissions, int[] gr
       }
 
       alreadyCalled = true;
-      if (grantResults[0] != PackageManager.PERMISSION_GRANTED) {
+      // grantResults could be empty if the permissions request with the user is interrupted
+      // https://developer.android.com/reference/android/app/Activity#onRequestPermissionsResult(int,%20java.lang.String[],%20int[])
+      if (grantResults.length == 0 || grantResults[0] != PackageManager.PERMISSION_GRANTED) {
         callback.onResult(CAMERA_ACCESS_DENIED, CAMERA_ACCESS_DENIED_MESSAGE);
       } else if (grantResults.length > 1 && grantResults[1] != PackageManager.PERMISSION_GRANTED) {
         callback.onResult(AUDIO_ACCESS_DENIED, AUDIO_ACCESS_DENIED_MESSAGE);
diff --git a/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraPermissionsTest.java b/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraPermissionsTest.java
index d734a63b15ca..575ec8c1caad 100644
--- a/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraPermissionsTest.java
+++ b/packages/camera/camera_android/android/src/test/java/io/flutter/plugins/camera/CameraPermissionsTest.java
@@ -72,4 +72,18 @@ public void callback_doesNotRespond() {
     verify(fakeResultCallback, never())
         .onResult("AudioAccessDenied", "Audio access permission was denied.");
   }
+
+  @Test
+  public void callback_respondsWithCameraAccessDeniedWhenEmptyResult() {
+    // Handles the case where the grantResults array is empty
+
+    ResultCallback fakeResultCallback = mock(ResultCallback.class);
+    CameraRequestPermissionsListener permissionsListener =
+        new CameraRequestPermissionsListener(fakeResultCallback);
+
+    permissionsListener.onRequestPermissionsResult(9796, null, new int[] {});
+
+    verify(fakeResultCallback)
+        .onResult("CameraAccessDenied", "Camera access permission was denied.");
+  }
 }
diff --git a/packages/camera/camera_android/pubspec.yaml b/packages/camera/camera_android/pubspec.yaml
index 6f1b667670e8..6aa78268cb00 100644
--- a/packages/camera/camera_android/pubspec.yaml
+++ b/packages/camera/camera_android/pubspec.yaml
@@ -2,7 +2,7 @@ name: camera_android
 description: Android implementation of the camera plugin.
 repository: https://github.com/flutter/plugins/tree/main/packages/camera/camera_android
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+camera%22
-version: 0.10.0+4
+version: 0.10.0+5
 
 environment:
   sdk: ">=2.14.0 <3.0.0"