[mythtv-commits] Ticket #10616: mythtranscode segfaults if called with --fifoinfo

MythTV noreply at mythtv.org
Wed Apr 18 02:21:57 UTC 2012


#10616: mythtranscode segfaults if called with --fifoinfo
------------------------------------+----------------------------
 Reporter:  furti@…                 |          Owner:  beirdo
     Type:  Bug Report - Crash      |         Status:  closed
 Priority:  minor                   |      Milestone:  0.25.1
Component:  MythTV - Mythtranscode  |        Version:  0.25-fixes
 Severity:  medium                  |     Resolution:  fixed
 Keywords:                          |  Ticket locked:  0
------------------------------------+----------------------------

Comment (by Gavin Hurlbut <ghurlbut@…>):

 In [changeset:9cb904e9b9998ea94eadba4db54c0af10dc893f0/mythtv]:
 {{{
 #!CommitTicketReference repository="mythtv"
 revision="9cb904e9b9998ea94eadba4db54c0af10dc893f0"
 Convert the use of player_ctx in mythtranscode to QPointer<>

 We have far too many places in the transcode code path that delete
 player_ctx
 and return.  Unfortunately, freeing "transcode" also will try to delee
 player_ctx, but it's already deleted, and not nulled out.  Rather than
 putting
 many instances of "player_ctx = NULL", I opted to make PlayerContext a
 QObject, and then just use QPointer<PlayerContext> as this will clear
 itself
 to NULL as soon as it is destroyed.

 Fixes #10616
 (cherry picked from commit c5a9262355a51b2d2b0465234d79b527c3907e84)
 }}}

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10616#comment:8>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center


More information about the mythtv-commits mailing list