[mythtv-commits] Ticket #12782: backend SegFault

[MythTV]

#12782: backend SegFault
---------------------------------------------+----------------------------
 Reporter:  Eric Fontaine <ericrfontaine@…>  |          Owner:  stuarta
     Type:  Bug Report - Crash               |         Status:  infoneeded
 Priority:  minor                            |      Milestone:  0.28.1
Component:  MythTV - General                 |        Version:  0.28.0
 Severity:  medium                           |     Resolution:
 Keywords:                                   |  Ticket locked:  0
---------------------------------------------+----------------------------
Changes (by bmeek):

 * status:  accepted => infoneeded


Comment:

 Eric, PlanetEater,

 Please revert the patch and use this. Try to cause the
 failure with the command in the previous comment. The output
 should have <String>yourHostName</String> it it. With the
 existing patch, you'll see <String/>.

 {{{
 diff --git a/mythtv/libs/libmythupnp/serializers/xmlSerializer.cpp
 b/mythtv/libs/libmythupnp/serializers/xmlSerializer.cpp
 index 9851e61..eeacd61 100644
 --- a/mythtv/libs/libmythupnp/serializers/xmlSerializer.cpp
 +++ b/mythtv/libs/libmythupnp/serializers/xmlSerializer.cpp
 @@ -324,9 +324,12 @@ QString XmlSerializer::GetContentName( const QString
 &sName,
  {
      // Try to read Name or TypeName from classinfo metadata.

 -    int nClassIdx = pMetaObject->indexOfClassInfo( sName.toLatin1() );
 +    int nClassIdx = -1;

 -    if (nClassIdx >=0)
 +    if ( pMetaObject )
 +        nClassIdx = pMetaObject->indexOfClassInfo( sName.toLatin1() );
 +
 +    if (nClassIdx >=0 )
      {
          QString     sOptionData = pMetaObject->classInfo( nClassIdx
 ).value();
          QStringList sOptions    = sOptionData.split( ';' );
 }}}

 pMetaObject->indexOfClassInfo( sName.toLatin1() returns -1 "normally".

--
Ticket URL: <https://code.mythtv.org/trac/ticket/12782#comment:31>
MythTV <http://www.mythtv.org>
MythTV Media Center