Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 2 years ago

Closed 2 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@…> 2 years ago.
Backtrace
gdb.2.txt (106.0 KB) - added by Dave Badia <dbadia@…> 2 years ago.
Backtrace

Download all attachments as: .zip

Change History (6)

comment:1 Changed 2 years ago by robertm

  • Status changed from new to infoneeded_new

Backtrace forthcoming?

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

Backtrace

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

Backtrace

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

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

comment:3 Changed 2 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 2 years ago by robertm

  • Resolution set to Fixed
  • Status changed from infoneeded_new to closed

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.