[mythtv-commits] Ticket #12782: backend SegFault

[MythTV]

#12782: backend SegFault
---------------------------------------------+-------------------------
 Reporter:  Eric Fontaine <ericrfontaine@…>  |          Owner:
     Type:  Bug Report - Crash               |         Status:  new
 Priority:  minor                            |      Milestone:  unknown
Component:  MythTV - General                 |        Version:  0.28.0
 Severity:  medium                           |     Resolution:
 Keywords:                                   |  Ticket locked:  0
---------------------------------------------+-------------------------

Comment (by Jonathan Isom <jeisom@…>):

 I am pulling from:
 git://github.com/MythTV/mythtv.git#branch=fixes/0.28

 Let me know if this is the wrong repo.

 Here is the code in that repo and below it from  the pull request. I got
 the crash from the above repo. Obviously the code looks different, but
 should not change how it functions. If there is a crash with one and not
 the other then it would be a compiler error.

 {{{
 void XmlSerializer::AddProperty( const QString       &sName,
                                  const QVariant      &vValue,
                                  const QMetaObject   *pMetaParent,
                                  const QMetaProperty *pMetaProp )
 {
     m_pXmlWriter->writeStartElement( sName );

     if ((pMetaProp != NULL) &&
         (pMetaProp->isEnumType() || pMetaProp->isFlagType()))
     {
         RenderEnum ( sName, vValue, pMetaProp );
     }
     else
         RenderValue( GetContentName( sName, pMetaParent, pMetaProp ),
 vValue );

     m_pXmlWriter->writeEndElement();
 }
 }}}
 From pull request:
 {{{
 void XmlSerializer::AddProperty( const QString       &sName,
                                  const QVariant      &vValue,
                                  const QMetaObject   *pMetaParent,
                                  const QMetaProperty *pMetaProp )
 {
     m_pXmlWriter->writeStartElement( sName );

     if (pMetaProp == NULL)
     {
         // Do nothing
     }
     else if (pMetaProp->isEnumType() || pMetaProp->isFlagType())
     {
         RenderEnum ( sName, vValue, pMetaProp );
     }
     else
         RenderValue( GetContentName( sName, pMetaParent, pMetaProp ),
 vValue );

     m_pXmlWriter->writeEndElement();
 }
 }}}

 I will try the pull request version, but I don't expect a difference. It
 also may be that Qt 5.7 from arch testing introduced another bug relate to
 the compiler.

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