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

[BUG] LE settings addon causes crash when Bluetooth is updating the window #5398

Closed
heitbaum opened this issue May 25, 2021 · 8 comments
Closed
Assignees
Labels

Comments

@heitbaum
Copy link
Contributor

heitbaum commented May 25, 2021

Crash caused during Bluetooth updating the window in LE Settings:

http://ix.io/3nNd

2021-05-23 15:34:24.405 T:692     ERROR <general>: Control 1000 in window 13001 has been asked to focus, but it can't
2021-05-23 15:34:24.406 T:795      INFO <general>: SETTINGS: run # Waiting
2021-05-23 15:34:50.948 T:692   WARNING <general>: CGUIWindowManager - HandleAction - ignoring action 92, because topmost modal dialog closing animation is running
2021-05-23 15:34:51.149 T:900     ERROR <general>: SETTINGS: bluetooth.menu_connections # AttributeError("module 'oe' has no attribute 'winOeMain'")
2021-05-23 15:34:51.150 T:900     ERROR <general>: Traceback (most recent call last):
                                                     File "/usr/share/kodi/addons/service.libreelec.settings/resources/lib/log.py", line 45, in _log_function_2
                                                     File "/usr/share/kodi/addons/service.libreelec.settings/resources/lib/modules/bluetooth.py", line 338, in menu_connections
                                                   AttributeError: module 'oe' has no attribute 'winOeMain'
@jernejsk
Copy link
Member

jernejsk commented Jun 4, 2021

I don't think this is really Allwinner specific?

@heitbaum
Copy link
Contributor Author

heitbaum commented Jun 5, 2021

No. I think it is across all. Though I haven’t tested it further on any other images.

@romanvm
Copy link

romanvm commented Jun 19, 2021

@heitbaum Could you try to reproduce this with the current master and paste the log here? It includes extended error logging that should give us more info. Please attach your log to the post or upload it to some pastebin site.

Also it would be good if you describe the steps to reproduce this.

@heitbaum
Copy link
Contributor Author

I have just done the pre testing now.

was able to get the crash before updating. I went to the BT menu in settings, and watched it refresh (jerky) I did now press in to any BT device, I then moved out of the BT menu, out of le menu and was about to press enter on the system information settings and it crashed.

LibreELEC:/.kodi/temp # pastebinit kodi.old.log
http://ix.io/3qrx
LibreELEC:
/.kodi/temp # pastebinit kodi_crashlog_20210619125905.log
http://ix.io/3qry

Next message will have the testing after reboot into master.

@heitbaum
Copy link
Contributor Author

@romanvm Whilst I haven't received a crash.

tx6:~/.kodi/temp # pastebinit kodi.log
http://ix.io/3qvT

I did receive a hang, then it went back to the main menu - see the first image, as you can see the thread doing the update to the right pane is "locked" I did add my iPhone as a BT device.

SH1

On entering into the BT menu - It always comes up with this message before it starts searching for BT devices.

SH2

@heitbaum
Copy link
Contributor Author

The follow short video also shows that when a refresh occurs the pane on the right looses focus and takes you back to "Bluetooth" thus the "down arrow" takes you to "Services"

Img.6700-1.mp4

@romanvm
Copy link

romanvm commented Jun 20, 2021

@heitbaum From the latest log I see that you are not using the latest master of the settings addon with improved diagnostics. I really want to see extended error logs. Also this comment:

On entering into the BT menu - It always comes up with this message before it starts searching for BT devices.

makes me thing that you are using the old version. One of my changes was to display this label only when necessary. Please make sure you are testing the latest version from here: #4547

@heitbaum
Copy link
Contributor Author

Closing. Not seen recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants