Opened 12 years ago

Closed 12 years ago

#10176 closed Bug Report - Crash (Fixed)

[CRASH] MythVideo next video doesn't exist

Reported by: Dave Badia <dbadia@…> Owned by: robertm
Priority: minor Milestone: unknown
Component: Plugin - MythVideo Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If the childid doesn't exist, myth segafults

Steps to reproduce:

  1. Setup 2 videos, foo and bar. Set foo's next video (childid in the db) to bar
  2. Delete bar and rescan the videos
  3. Play foo, myth will segfault after when it tries to play bar which no longer exists in the DB

Backtrace attached from v24-fixes (full info below) I haven't tried this with master but the logic appears to be the same

MythTV Version : v0.24.1-128-g7d04143 MythTV Branch : 241f Network Protocol : 63 Library API : 0.24.20110505-1 QT Version : 4.6.2 Options compiled in:

linux profile using_alsa using_oss using_backend using_bindings_perl using_bindings_python using_dvb using_frontend using_hdhomerun using_hdpvr using_iptv using_ivtv using_joystick_menu using_lirc using_mheg using_opengl_video using_opengl_vsync using_qtdbus using_qtwebkit using_v4l using_x11 using_xrandr using_xv using_bindings_perl using_bindings_python using_mythtranscode using_opengl using_vdpau using_ffmpeg_threads using_live using_mheg

Attachments (2)

gdb.txt (106.0 KB) - added by Dave Badia <dbadia@…> 12 years ago.
Backtrace
gdb.2.txt (106.0 KB) - added by Dave Badia <dbadia@…> 12 years ago.
Backtrace

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by robertm

Status: newinfoneeded_new

Backtrace forthcoming?

Changed 12 years ago by Dave Badia <dbadia@…>

Attachment: gdb.txt added

Backtrace

Changed 12 years ago by Dave Badia <dbadia@…>

Attachment: gdb.2.txt added

Backtrace

comment:2 Changed 12 years ago by Dave Badia <dbadia@…>

Backtrace added. Not sure how I added it twice, but both files are the same

comment:3 Changed 12 years ago by Github

Video Library: Avoid crashing if child item no longer exists.

Check validity of child metadata before attempting to play it.

Refs #10176.

Branch: master Changeset: 2472d67085b25bedcee1cedbf2b5f9111384e48f

comment:4 Changed 12 years ago by robertm

Resolution: Fixed
Status: infoneeded_newclosed
Note: See TracTickets for help on using tickets.