1385738 - Crash in java.lang.SecurityException: uid 10132 does not have android.permission.UPDATE_APP_OPS_STATS. at android.os.Parcel.readException(Parcel.java)

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Nicholas Nethercote [inactive]]

This bug was filed from the Socorro interface and is 
report bp-3d8224e1-eacc-4ce5-b1d6-04a750170729.
=============================================================

Stack trace:

> Java.lang.SecurityException: uid 10132 does not have android.permission.UPDATE_APP_OPS_STATS.
>  at android.os.Parcel.readException(Parcel.java:1684)
>  at android.os.Parcel.readException(Parcel.java:1637)
>  at android.media.session.ISessionController$Stub$Proxy.getVolumeAttributes(ISessionController.java:703)
>  at android.media.session.MediaController.getPlaybackInfo(MediaController.java:261)
>  at org.mozilla.gecko.media.MediaControlService.initMediaSession(MediaControlService.java:299)
>  at org.mozilla.gecko.media.MediaControlService.initialize(MediaControlService.java:170)
>  at org.mozilla.gecko.media.MediaControlService.onCreate(MediaControlService.java:86)
>  at android.app.ActivityThread.handleCreateService(ActivityThread.java:3236)
>  at android.app.ActivityThread.-wrap5(ActivityThread.java)
>  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1610)
>  at android.os.Handler.dispatchMessage(Handler.java:102)
>  at android.os.Looper.loop(Looper.java:160)
>  at android.app.ActivityThread.main(ActivityThread.java:6270)
>  at java.lang.reflect.Method.invoke(Native Method)
>  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:898)
>  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:788)

It's showing up under multiple signatures because the uid changes.

jchen, any ideas?

Comment 1

[Alastor Wu [:alwu]]

I guess it might be caused when user disable fennec's notification, and only system app can modify the APP_OPS.

NotificationManagerCompat::areNotificationsEnabled() [1] can be used to check the status, but it's only available after support library 24.

For this issue,
Should we (1) just catch this exception and return false, or (2) update our support library and use areNotificationsEnabled() to do more appropriate checking?

Thanks!

[1] https://developer.android.com/reference/android/support/v4/app/NotificationManagerCompat.html#areNotificationsEnabled()

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

> NotificationManagerCompat::areNotificationsEnabled() [1] can be used to check the status, but it's only available after support library 24.

We should be able to update to the new support library very soon! :)

For future patches/questions, please flag :nechen

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

No crashes since October.