Android 9.0 NotificationManager.notify () выбрасывает java.lang.SecurityException - PullRequest
0 голосов
/ 10 января 2019

Мне не удалось воспроизвести эту проблему самостоятельно, но пока 5 пользователей сообщили об этом. Недавно я опубликовал обновление приложения, которое изменило целевой SDK с 27 на 28, и я уверен, что это сыграло свою роль. Все 5 пользователей работают под управлением Android 9 на каком-то устройстве Pixel. Как и я.

Приложение реагирует на ситуацию оповещения, вызывая настройку уведомления и вызывая NotificationManager.notify (). Это уведомление ссылается на канал уведомления, который пытается воспроизвести аудиофайл, расположенный на внешнем хранилище. Мое приложение содержит разрешение READ_EXTERNAL_STORAGE в манифесте. Но поскольку он сам не имеет доступа к чему-либо во внешнем хранилище, он не просил пользователя предоставить ему такое разрешение.

Когда я делаю это на моем пикселе, это работает просто отлично. Но 5 пользователей сообщили, что выбрасывают исключение типа

java.lang.RuntimeException: Unable to start activity ComponentInfo{net.anei.cadpage/net.anei.cadpage.CadPageActivity}: java.lang.SecurityException: UID 10132 does not have permission to content://media/external/audio/media/145 [user 0]
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2914)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3049)
at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:78)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1809)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:193)
at android.app.ActivityThread.main(ActivityThread.java:6680)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
Caused by: java.lang.SecurityException: UID 10132 does not have permission to content://media/external/audio/media/145 [user 0]
at android.os.Parcel.createException(Parcel.java:1950)
at android.os.Parcel.readException(Parcel.java:1918)
at android.os.Parcel.readException(Parcel.java:1868)
at android.app.INotificationManager$Stub$Proxy.enqueueNotificationWithTag(INotificationManager.java:1559)
at android.app.NotificationManager.notifyAsUser(NotificationManager.java:405)
at android.app.NotificationManager.notify(NotificationManager.java:370)
at android.app.NotificationManager.notify(NotificationManager.java:346)
at net.anei.cadpage.ManageNotification.show(ManageNotification.java:186)
at net.anei.cadpage.ReminderReceiver.scheduleNotification(ReminderReceiver.java:46)
at net.anei.cadpage.ManageNotification.show(ManageNotification.java:161)
at net.anei.cadpage.CadPageActivity.startup(CadPageActivity.java:211)
at net.anei.cadpage.CadPageActivity.onCreate(CadPageActivity.java:93)
at android.app.Activity.performCreate(Activity.java:7144)
at android.app.Activity.performCreate(Activity.java:7135)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1271)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2894)
... 11 more
Caused by: android.os.RemoteException: Remote stack trace:
at com.android.server.am.ActivityManagerService.checkGrantUriPermissionLocked(ActivityManagerService.java:9752)
at com.android.server.am.ActivityManagerService.checkGrantUriPermission(ActivityManagerService.java:9769)
at com.android.server.notification.NotificationRecord.visitGrantableUri(NotificationRecord.java:1096)
at com.android.server.notification.NotificationRecord.calculateGrantableUris(NotificationRecord.java:1072)
at com.android.server.notification.NotificationRecord.<init>(NotificationRecord.java:201)

Я велел всем 4 пользователям вручную предоставить разрешение «Хранилище» и AFAIK, который решает проблему. Но зачем это нужно? Мой сам не получил доступ к внешнему хранилищу или настроил конфигурацию канала так, чтобы это требовалось. Если требуется разрешение READ_EXTERNAL_STORAGE, диспетчер уведомлений должен управлять этим.

Проблема с сообщениями пользователей выполнялась следующим образом: Google / таймень / таймень: 9 / PQ1A.190105.004 / 5148680: пользователь / релиз-ключи Google / штриховки / штриховки: 9 / PQ1A.190105.004 / 5148680: пользователь / релиз-ключи Google / марлин / марлин: 9 / PQ1A.181205.002.A1 / 5129870: пользователь / Release-клавиши Google / парусник / парусник: 9 / PQ1A.181205.002.A1 / 5129870: пользователь / Release-клавиши Google / судак / судак: 9 / PQ1A.181205.002 / 5086253: пользователь / Release-клавиши

Я бегу Google / таймень / таймень: 9 / PQ1A.181205.002 / 5086253: пользователь / релиз-ключи который, кажется, отстает от всех остальных, обновляя до Google / таймень / таймень: 9 / PQ1A.190105.004 / 5148680: пользователь / релиз-ключи ничего не меняет Все еще отлично работает на моем устройстве.

Вот весь код с некоторыми подсказками относительно того, какие ветви взяты. Трассировка стека довольно ясна, что исключение было сгенерировано в вызове notify (). И это было прервано, потому что у приложения не было безопасного доступа к аудиофайлу, указанному каналом.

// Build and launch the notification
Notification n = buildNotification(context, message);

NotificationManager myNM = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
assert myNM != null;

// Seems this is needed for the number value to take effect on the Notification
activeNotice = true;
myNM.cancel(NOTIFICATION_ALERT);
myNM.notify(NOTIFICATION_ALERT, n);

........

private static Notification buildNotification(Context context, SmsMmsMessage message) {

/*
 * Ok, let's create our Notification object and set up all its parameters.
 */
NotificationCompat.Builder nbuild = new NotificationCompat.Builder(context, ALERT_CHANNEL_ID);

// Set auto-cancel flag
nbuild.setAutoCancel(true);

// Set display icon
nbuild.setSmallIcon(R.drawable.ic_stat_notify);

// From Oreo on, these are set at the notification channel level
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) {  // False

  // Maximum priority
  nbuild.setPriority(NotificationCompat.PRIORITY_MAX);

  // Message category
  nbuild.setCategory(NotificationCompat.CATEGORY_CALL);

  // Set public visibility
  nbuild.setVisibility(NotificationCompat.VISIBILITY_PUBLIC);

  // Set up LED pattern and color
  if (ManagePreferences.flashLED()) {
    /*
     * Set up LED blinking pattern
     */
    int col = getLEDColor(context);
    int[] led_pattern = getLEDPattern(context);
    nbuild.setLights(col, led_pattern[0], led_pattern[1]);
  }

  /*
   * Set up vibrate pattern
   */
  // If vibrate is ON, or if phone is set to vibrate
  AudioManager AM = (AudioManager) context.getSystemService(Context.AUDIO_SERVICE);
  assert AM != null;
  if ((ManagePreferences.vibrate() || AudioManager.RINGER_MODE_VIBRATE == AM.getRingerMode())) {
    long[] vibrate_pattern = getVibratePattern(context);
    if (vibrate_pattern != null) {
      nbuild.setVibrate(vibrate_pattern);
    } else {
      nbuild.setDefaults(Notification.DEFAULT_VIBRATE);
    }
  }
}

if ( ManagePreferences.notifyEnabled()) {  // false

  // Are we doing are own alert sound?
  if (ManagePreferences.notifyOverride()) {

    // Save previous volume and set volume to max
    overrideVolumeControl(context);

    // Start Media Player
    startMediaPlayer(context, 0);
  } else if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O){
    Uri alarmSoundURI = Uri.parse(ManagePreferences.notifySound());
    nbuild.setSound(alarmSoundURI);
  }
}

String call = message.getTitle();
nbuild.setContentTitle(context.getString(R.string.cadpage_alert));
nbuild.setContentText(call);
nbuild.setStyle(new NotificationCompat.InboxStyle().addLine(call).addLine(message.getAddress()));
nbuild.setWhen(message.getIncidentDate().getTime());

// The default intent when the notification is clicked (Inbox)
Intent smsIntent = CadPageActivity.getLaunchIntent(context, true);
PendingIntent notifIntent = PendingIntent.getActivity(context, 0, smsIntent, 0);
nbuild.setContentIntent(notifIntent);

// Set intent to execute if the "clear all" notifications button is pressed -
// basically stop any future reminders.
Intent deleteIntent = new Intent(new Intent(context, ReminderReceiver.class));
deleteIntent.setAction(Intent.ACTION_DELETE);
PendingIntent pendingDeleteIntent = PendingIntent.getBroadcast(context, 0, deleteIntent, 0);
nbuild.setDeleteIntent(pendingDeleteIntent);

return nbuild.build();
}

Последние новости. Вчера вечером я опубликовал обновление, поддерживающее целевой SDK до 27 из 28. За ночь еще 2 пользователя сообщили об этом конкретном сбое на телефонах Pixel под управлением Android 9. Оба работали с версией, ориентированной на SDK 28. Один из них вернулся ко мне и подтвердил, что проблема исчезли, когда они установили версию приложения SDK 27. Это подтверждает, что это проблема с приложениями, ориентированными на SDK 28, возможно, связана с изменением, запрещающим приложениям использовать разрешения файловой системы доступа к миру для преодоления ограничений изолированной программной среды приложения.

До сих пор остается загадкой, почему это влияет на одних пользователей, но не на других. Конкретно я. Когда у меня будет время, я попытаюсь воспроизвести проблему на моем телефоне. Две теории 1) Поражает только тех, кто никогда не давал разрешения READ_EXTERNAL_STORAGE. Первоначально мое разрешение было предоставлено, и я отозвал его при попытке воспроизвести проблему. 2) Это происходит только тогда, когда приложение изначально установило канал уведомления с использованием внешнего аудиофайла. Это было бы верно для большинства пользователей, но в моем случае звуковой файл был настроен вручную.

1 Ответ

0 голосов
/ 13 мая 2019

У меня была эта проблема. Оказывается, что создание канала уведомления было ошибкой.

Неправильный путь:

val notifictionChannel = NotificationChannel(...)
notificationChannel.setSound(
    RingtoneManager.getActualDefaultRingtoneUri(context, RingtoneManager.TYPE_NOTIFICATION),
    AudioAttributes.Builder().setUsage(AudioAttributes.USAGE_NOTIFICATION).build()
)

notificationManager.createNotificationChannel(notificationChannel)

Правильный путь:

...
notificationChannel.setSound(
    Settings.System.DEFAULT_NOTIFICATION_URI,
    AudioAttributes.Builder().setUsage(AudioAttributes.USAGE_NOTIFICATION).build()
)
...
...