1613871 - Attempt to invoke virtual method 'void android.media.MediaDrm.provideProvisionResponse(byte[])' on a null object reference

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

Just saw that in a migration build:

java.lang.NullPointerException: Attempt to invoke virtual method 'void android.media.MediaDrm.provideProvisionResponse(byte[])' on a null object reference
    at org.mozilla.gecko.media.GeckoMediaDrmBridgeV21$PostRequestTask.onPostExecute(GeckoMediaDrmBridgeV21.java:5)
    at android.os.AsyncTask.finish(AsyncTask.java:695)
    at android.os.AsyncTask.-wrap1
    at android.os.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:712)
    at android.os.Handler.dispatchMessage(Handler.java:106)
    at android.os.Looper.loop(Looper.java:164)
    at android.app.ActivityThread.main(ActivityThread.java:6651)
    at java.lang.reflect.Method.invoke(Method.java)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:810)

https://sentry.prod.mozaws.net/operations/fenix-fennec-nightly/issues/7220235

Comment 1

[Sebastian Kaspari (:sebastian; :pocmo)]

https://sentry.prod.mozaws.net/operations/fenix-fennec-nightly/issues/7250072/?query=is:unresolved

Comment 2

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1613871 - cancel background task when releasing DRM object. r?snorp

The provision request is executed in background through AsyncTask and will
post the result once finished. However, when the DRM object is released
before onPostExecute() is called, there will be NPE. Use AsyncTask.cancel()
to avoid that.

Comment 3

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1613871 - cancel background task when releasing DRM object. r?snorp

The provision request is executed in background through AsyncTask and will
post the result once finished. However, when the DRM object is released
before onPostExecute() is called, there will be NPE. Use AsyncTask.cancel()
to avoid that.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/410c9a491935
cancel background task when releasing DRM object. r=snorp

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/410c9a491935