Playing a video causes app crash [Android 7 - Lenovo] - caused by NullPointer in SurfaceView.updateWindow

[0xTyagi]

Checklist

• I am using the latest version - 0.21.14
• I checked, but didn't find any duplicates (open OR closed) of this issue in the repo.
• I have read the contribution guidelines given at https://github.com/TeamNewPipe/NewPipe/blob/HEAD/.github/CONTRIBUTING.md.
• This issue contains only one bug. I will open one issue for every bug report I want to file.

Steps to reproduce the bug

1.Open any video.
2. Play video.
3.Change Orientation (either automatically or manually). Or press the back button.
4. App crash

Actual behavior

App crash

Expected behavior

Videos must be played without any carsh

Screenshots/Screen recordings

Not required.

Logs

Exception

• User Action: ui error
• Request: ACRA report
• Content Country: GB
• Content Language: en
• App Language: en_IN
• Service: none
• Version: 0.21.14
• OS: Linux Android 7.0 - 24

Crash log

java.lang.RuntimeException: Unable to stop service org.schabi.newpipe.player.MainPlayer@264a0e5: java.lang.NullPointerException: Attempt to invoke interface method 'void android.view.IWindowSession.performDeferredDestroy(android.view.IWindow)' on a null object reference
	at android.app.ActivityThread.handleStopService(ActivityThread.java:3344)
	at android.app.ActivityThread.-wrap27(ActivityThread.java)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1570)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:154)
	at android.app.ActivityThread.main(ActivityThread.java:6077)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)
Caused by: java.lang.NullPointerException: Attempt to invoke interface method 'void android.view.IWindowSession.performDeferredDestroy(android.view.IWindow)' on a null object reference
	at android.view.SurfaceView.updateWindow(SurfaceView.java:658)
	at android.view.SurfaceView.onWindowVisibilityChanged(SurfaceView.java:255)
	at android.view.View.dispatchDetachedFromWindow(View.java:15437)
	at android.view.ViewGroup.dispatchDetachedFromWindow(ViewGroup.java:3222)
	at android.view.ViewGroup.removeViewInternal(ViewGroup.java:4758)
	at android.view.ViewGroup.removeViewInternal(ViewGroup.java:4732)
	at android.view.ViewGroup.removeView(ViewGroup.java:4663)
	at org.schabi.newpipe.player.MainPlayer.removeViewFromParent(MainPlayer.java:240)
	at org.schabi.newpipe.player.MainPlayer.cleanup(MainPlayer.java:187)
	at org.schabi.newpipe.player.MainPlayer.onDestroy(MainPlayer.java:178)
	at android.app.ActivityThread.handleStopService(ActivityThread.java:3327)
	... 8 more

---

[AudricV]

What's your device and your device's ROM? It seems you removed this part when filling the template.

I couldn't reproduce this issue on my device (Honor 9X, EMUI 10.0/Android 10).

[Kariosfox]

I'm also experiencing this issue. I'm using a Lenovo P2a42 with Android 7.0.

[0xTyagi]

OS: Android 24(7.0) stock
Device: Lenovo k33

[0xTyagi]

This comment may be helpful as this issues also have same crash signature.

lipangit/JiaoZiVideoPlayer#62 (comment)

[Kariosfox]

This issue is also present in the newest version 21.15.

[litetex]

#7549 (comment) is the same exception as in the issue

#7549 (comment) is an unrelated exception

Note: It looks like the problem only happens on Android 7.0 (as seen in the crash reports) and on Lenovo devices.

Analysis

The problem occurs inside the Android framework itself (likely here) however I was unable to located any source/branch which exactly matches the line numbers. So some kind of modification was likely done inside the class by the device manufacturer.

The problematic code is no longer present in Android 8+ and was likely fixed by a refactoring, adding a null check and the catching of all exceptions with this commit:

TL;DR
The problem is not fixable by us as it's a Android problem or a problem introduced by a modification from the manufacturer of the device (likely Lenovo).
Android 8+ provides a fix so I recommend updating the OS if possible.

[litetex]

In a duplicate issue the reporter stated that s/he also uses a Lenovo device - I'm now 99% sure that there is a bug in Lenovo's Android 7 OS.

[jjakob]

I have probably the same bug on LineageOS 14.1 Android 7.1.2

java.lang.NullPointerException: Attempt to invoke interface method 'void android.view.IWindowSession.finishDrawing(android.view.IWindow)' on a null object reference
at android.view.SurfaceView.updateWindow(SurfaceView.java:652)
at android.view.SurfaceView.onWindowVisibilityChanged(SurfaceView.java:247)
at android.view.View.dispatchWindowVisibilityChanged(View.java:10358)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewGroup.dispatchWindowVisibilityChanged(ViewGroup.java:1290)
at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1544)
at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1254)
at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6344)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:874)
at android.view.Choreographer.doCallbacks(Choreographer.java:686)
at android.view.Choreographer.doFrame(Choreographer.java:621)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:860)
at android.os.Handler.handleCallback(Handler.java:751)
at android.os.Handler.dispatchMessage(Handler.java:95)\n\tat android.os.Looper.loop(Looper.java:154)
at android.app.ActivityThread.main(ActivityThread.java:6186)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:889)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:779)

[celikten]

Reply to: #7549 (comment) (Mod-note: removed quote reply because it requires massive amounts of space)

As I remember correctly, the previous versions of the NewPipe versions (before 0.21.14) have worked properly. , and I loved putting something in the background and closing the phone screen.

Is this crash only related to the android version? Can not it be another bug?

[litetex]

Is this crash only related to the android version? Can not it be another bug?

→ see above

The problem is not fixable by us as it's a Android problem or a problem introduced by a modification from the manufacturer of the device (likely Lenovo).
Android 8+ provides a fix so I recommend updating the OS if possible.