Opened 14 years ago
Closed 14 years ago
#8494 closed defect (fixed)
Segfault since program info refactor
| Reported by: | Stuart Auchterlonie | Owned by: | danielk |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.24 |
| Component: | MythTV - General | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Going into
Manage Recordings -> Previous Recordings
then deleting info about a previous recording causes a segfault. Backtrace attached.
Stuart
Attachments (2)
Change History (5)
Changed 14 years ago by
| Attachment: | program-info-segfault.txt added |
|---|
Changed 14 years ago by
| Attachment: | 8494-v1.patch added |
|---|
comment:1 Changed 14 years ago by
There is one "TODO FIXME" in the patch, otherwise it just needs further testing (the fixme as to do with restoring your position in the list after a reload where an item may have been deleted from the list.)
comment:3 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [25090]) Fixes #8494. Cleanup of ProgList? code.
I had tried to do a minimal fix of a large memory leak found in this class during the ProgramInfo? refactor, but a high degree of bitrot meant that a bit of reorg was necessary to fix the memory leak safely. This does a full once over of the ProgList? code fixing not just the memory leak but all problems seen. Note: The helper classes in proglist_helpers.{h,cpp} were not cleaned up, only moved to out.
Fixes a number of problems with proglist