Skip to content

Commit

Permalink
VideoLookupInfo: Fix Coverity IDs 700896,700897 Uninitialized scalar …
Browse files Browse the repository at this point in the history
…fields

In DTC::ArtworkItem::ArtworkItem(): A couple of scalar fields are not
initialized by the constructor.

In DTC::VideoLookup::VideoLookup(): Several scalar fields are not initialized
by the constructor.

Also add several fields to DTC::VideoLookup::Copy() which are missing and I
assume is an oversight.
  • Loading branch information
Paul Harrison committed Jun 15, 2013
1 parent 89b6571 commit f43a9d7
Showing 1 changed file with 15 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,8 @@ class SERVICE_PUBLIC ArtworkItem : public QObject
ArtworkItem(QObject *parent = 0)
: QObject ( parent )
{
m_Width = 0 ;
m_Height = 0 ;
}

ArtworkItem( const ArtworkItem &src )
Expand Down Expand Up @@ -131,6 +133,14 @@ class SERVICE_PUBLIC VideoLookup : public QObject
VideoLookup(QObject *parent = 0)
: QObject ( parent )
{
m_Season = 0 ;
m_Episode = 0 ;
m_Year = 0 ;
m_UserRating = 0.0 ;
m_Length = 0.0 ;
m_Popularity = 0 ;
m_Budget = 0 ;
m_Revenue = 0 ;
}

VideoLookup( const VideoLookup &src )
Expand All @@ -154,6 +164,11 @@ class SERVICE_PUBLIC VideoLookup : public QObject
m_ReleaseDate = src.m_ReleaseDate ;
m_UserRating = src.m_UserRating ;
m_Length = src.m_Length ;
m_Popularity = src.m_Popularity ;
m_Budget = src.m_Budget ;
m_Revenue = src.m_Revenue ;
m_IMDB = src.m_IMDB ;
m_TMSRef = src.m_TMSRef ;

CopyListContents< ArtworkItem >( this, m_Artwork, src.m_Artwork );
}
Expand Down

0 comments on commit f43a9d7

Please sign in to comment.