Opened 11 years ago

Closed 8 years ago

#5468 closed Developer Task (Won't Fix)

[TASK] Remove abort() calls in code

Reported by: danielk Owned by: danielk
Priority: trivial Milestone: unknown
Component: MythTV - General Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by danielk)

There are 12 abort) calls in mythtv, these errors should be handled more gracefully:

libs/libavcodec/mpegvideo.c:    abort();
libs/libavcodec/shorten.c:            abort();
libs/libmythdvdnav/decoder.c:    abort();
libs/libmythdvdnav/decoder.c:   abort();
libs/libmythdvdnav/decoder.c:      abort();
libs/libmythdvdnav/nav_read.c:    abort();
libs/libmythdvdnav/nav_read.c:  if (!getbits_init(&state, buffer)) abort(); /* Passed NULL pointers */
libs/libmythdvdnav/nav_read.c:  if (!getbits_init(&state, buffer)) abort(); /* Passed NULL pointers */
libs/libmythdvdnav/vm.c:    abort();
libs/libmythdvdnav/vm.c:    abort();
libs/libmythdvdnav/vm.c:    abort();
libs/libmythdvdnav/vm.c:    abort();

Change History (7)

comment:1 Changed 11 years ago by danielk

Description: modified (diff)

comment:2 Changed 11 years ago by Dibblah

Status: newassigned

comment:3 Changed 10 years ago by stuartm

Component: mythtvMythTV - General
Type: defecttask

comment:4 Changed 9 years ago by danielk

ugh, there are now 22 abort calls in the code. We're regressing...

comment:5 Changed 9 years ago by beirdo

Summary: Remove abort() calls in code[TASK] Remove abort() calls in code
Type: taskDeveloper Task

comment:6 Changed 9 years ago by beirdo

Please add assert() to this list, as these also cause very inelegant crashing. Great for debugging, not so great for a release build.

comment:7 Changed 8 years ago by danielk

Resolution: Won't Fix
Status: assignedclosed

I'm just not going to find the time to deal with the problems in the DVD libs & the ones in FFmpeg are unlikely to cause MythTV exits.

Note: See TracTickets for help on using tickets.