Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash on ui error #4546

Closed
4 tasks done
goyalyashpal opened this issue Oct 17, 2020 · 16 comments
Closed
4 tasks done

Crash on ui error #4546

goyalyashpal opened this issue Oct 17, 2020 · 16 comments
Labels
bug Issue is related to a bug

Comments

@goyalyashpal
Copy link
Contributor

goyalyashpal commented Oct 17, 2020

Checklist

Steps to reproduce the bug

  1. The background queue was playing with auto-play on,
  2. i searched for smth (in newpipe ofc) and clicked on one video to go to details page.
  3. it crashed.

Actual behaviour

Expected behavior

Screenshots/Screen recordings

Logs

Have a running playlist in background, and open a new video from search results.

Exception

  • User Action: ui error
  • Request: App crash, UI failure
  • Content Country: DE
  • Content Language: en-US
  • App Language: en
  • Service: none
  • Version: 0.20.0
  • OS: Linux Android 7.0 - 24
Crash log

java.lang.NullPointerException: Attempt to invoke virtual method 'int org.schabi.newpipe.player.playqueue.PlayQueueItem.getServiceId()' on a null object reference
	at org.schabi.newpipe.fragments.detail.VideoDetailFragment.onQueueUpdate(VideoDetailFragment.java:1736)
	at org.schabi.newpipe.player.helper.PlayerHolder$2.onQueueUpdate(PlayerHolder.java:181)
	at org.schabi.newpipe.player.VideoPlayerImpl.updateQueue(VideoPlayerImpl.java:1982)
	at org.schabi.newpipe.player.VideoPlayerImpl.setFragmentListener(VideoPlayerImpl.java:1955)
	at org.schabi.newpipe.player.helper.PlayerHolder.startPlayerListener(PlayerHolder.java:130)
	at org.schabi.newpipe.player.helper.PlayerHolder.setListener(PlayerHolder.java:39)
	at org.schabi.newpipe.player.helper.PlayerHolder.startService(PlayerHolder.java:51)
	at org.schabi.newpipe.fragments.detail.VideoDetailFragment.initListeners(VideoDetailFragment.java:712)
	at org.schabi.newpipe.BaseFragment.onViewCreated(BaseFragment.java:72)
	at org.schabi.newpipe.fragments.BaseStateFragment.onViewCreated(BaseStateFragment.java:56)
	at androidx.fragment.app.FragmentStateManager.createView(FragmentStateManager.java:332)
	at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1187)
	at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1356)
	at androidx.fragment.app.FragmentManager.moveFragmentToExpectedState(FragmentManager.java:1434)
	at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1497)
	at androidx.fragment.app.BackStackRecord.executeOps(BackStackRecord.java:447)
	at androidx.fragment.app.FragmentManager.executeOps(FragmentManager.java:2169)
	at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:1992)
	at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:1947)
	at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:1849)
	at androidx.fragment.app.FragmentManager$4.run(FragmentManager.java:413)
	at android.os.Handler.handleCallback(Handler.java:751)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at android.os.Looper.loop(Looper.java:154)
	at android.app.ActivityThread.main(ActivityThread.java:6146)
	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)


Device info

i think it was

  • Android version/Custom ROM version: 7.0
  • Device model: Prime P30 (jivi)

source of the package: fdroid repo

@goyalyashpal goyalyashpal added the bug Issue is related to a bug label Oct 17, 2020
@opusforlife2
Copy link
Collaborator

That checklist isn't optional, you know. You should read the contribution guidelines and always check for duplicates.

@goyalyashpal
Copy link
Contributor Author

i do, but when i have at least some clue of either the thing/issue, or the project. But since i dont know either of these (it just crashed & am new to the project) - i couldnt figure out how to search for it. Plus i got sidetracked many times already while browsing through existing issues and comments.

I have read the guidelines now, and there was nothing new/non obvious.

@opusforlife2
Copy link
Collaborator

Different STR but same crash: #3440 (comment)

@goyalyashpal
Copy link
Contributor Author

STR!!?

@opusforlife2
Copy link
Collaborator

Special Theory of Relativity, or... Steps To Reproduce.

Take your pick. :P

@goyalyashpal goyalyashpal changed the title Crash Crash on ui error Oct 17, 2020
@goyalyashpal
Copy link
Contributor Author

also, any comment on if u were able to reproduce the crash @opusforlife2 {& the related info as usual} ?!

@opusforlife2
Copy link
Collaborator

Good point. I don't get a crash.

Can you try the same steps once again just to confirm? If you can still reproduce the crash, a screen record might be helpful.

@UserX404
Copy link
Contributor

Maybe a (the) similar problem:
Backgroud player crashes/locks my phone. I have to insert my unlock pattern but afterwards it gets locked again. I'm not able to use my phone and have to reboot it.
So i'm also not able to submit a log.
Sony Xperia XZ2c
Android 10
Tested with 0.20.0 and 0.20.1.
0.19.8 is the last functional version.

@nonremittal
Copy link

Same deal as above, Xiaomi Mi A2.

When using the background player and unlocking my phone, it either re-locks, displays a black screen, or (in one instance) got rid of the notification bar for my entire phone!

A few times it has caused my phone to crash completely and restart in a terminal mode which was very scary.

I really hope 0.19 continues to work for the foreseeable future. I'm not a fan of the unified player at all (back button doesn't work how I expect, takes way more taps to do the same things, etc.)

@opusforlife2
Copy link
Collaborator

@UserX404 @nonremittal Please try the test APK in #4555.

@nonremittal Check out #4569.

@UserX404
Copy link
Contributor

UserX404 commented Oct 20, 2020

@opusforlife2 Thanks for response.
I've tested the debug edition. First it seems backgroundplayer is working but after some attemps to start different videos in fore- and background while a video is running (fore- or background) my phone get locked and i run into the same situation which i mentioned above.
Is there a possibility to submit debug logs or something else that can help you?
Heap dump -> 2020-10-20_19-33-42_541_heap.zip

Heap dump 2 -> 2020-10-20_19-43-15_738.zip

@goyalyashpal
Copy link
Contributor Author

goyalyashpal commented Oct 20, 2020

i also tried 2-3 times on same version as reported in description (0.20.0, 954) but cant reproduce. so, when i cant reproduce it in this version itself, how can i test if its fixed or not in newer one 😃 - so, am unable to check that debug one too for the time being.

~"needs reproducibility"

@opusforlife2
Copy link
Collaborator

@UserX404 Thank you for the logs. When you test an APK you're testing that PR's code, so it is more appropriate to post your findings in the PR thread.

@Stypox Logs for you.

@nonremittal
Copy link

@opusforlife2 Tried it, same issue. Constant black screen, can't unlock my phone until I restart.

Seems like a really serious bug so I hope it's fixed soon!

@litetex
Copy link
Member

litetex commented Oct 22, 2021

Is this still a problem?

@litetex litetex added the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Oct 22, 2021
@goyalyashpal
Copy link
Contributor Author

hi, i think it can be closed. 'cz this issue was not reproducible with me

@github-actions github-actions bot removed the waiting for author If the author doesn't respond, the issue will be auto-closed. Otherwise the label will be removed. label Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug
Projects
None yet
Development

No branches or pull requests

5 participants