Opened 15 years ago
Closed 15 years ago
Last modified 15 years ago
#7056 closed defect (fixed)
Mythvideo crashes after deleting a video and exiting
Reported by: | Owned by: | robertm | |
---|---|---|---|
Priority: | major | Milestone: | 0.22 |
Component: | Plugin - MythVideo | Version: | head |
Severity: | high | Keywords: | |
Cc: | Ticket locked: | no |
Description
Mythvideo crashes after you delete a video and try to return to the main menu (press escape)
Attachments (1)
Change History (7)
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
Owner: | changed from Anduin Withers to robertm |
---|---|
Status: | new → assigned |
Code above isn't actually directly responsible for the crash. This segfault only occurs after having deleted the last node in a folder. For whatever reason, the MythGenericTree? still reports the subnodes as both valid and visible when backing out, even though their metadata has been deleted and they've ostensibly been marked invisible. Will look for a solution in the widget itself, otherwise maybe a MythVideo? workaround can be found.
comment:3 Changed 15 years ago by
I can reproduce this even if it's NOT the last node (video) in the folder. It's 100% reproduceable.
comment:4 Changed 15 years ago by
I am also finding this issue and the deleted item was not to my knowledge the last node, trunk r21855.
comment:5 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
(In [21886]) Fixes #7056. Make sure to delete the item from both the MythGenericTree? as well as the buttontree/buttonlist. If a folder becomes empty, reload the tree to prevent orphaned folders (and associated segfaults).
comment:6 Changed 15 years ago by
(In [21887]) Refs #7056. Reverts a part of prior commit triggers a segfault in 1/3 instances. This leaves a potential segfault if you delete the last item in a folder and then scan without leaving mythvideo, but it's a step in the right direction. I'm sure there's some mythgenerictree cleanup I should be doing that I'm not...
Just since svn changes so rapidly, this is the exact line videodlg.cpp:3017