Opened 14 years ago

Last modified 13 years ago

#6497 closed bug

MediaConverter does not handle errors gracefully when sniffing files (easy) — at Initial Version

Reported by: yourpalal Owned by: yourpalal
Priority: normal Milestone: R1
Component: Applications/MediaConverter Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

MediaFileInfo::LoadInfo() ignores many return codes and can end up with a GIGO situation. Some examples are:

*should check track->InitCheck() after getting the track from TrackAt()

(MediaFileInfo.cpp:37)

*should check return of track->EncodedFormat()

(MediaFileInfo.cpp:39)

*should check return of track->DecodedFormat()

(MediaFileInfo.cpp:43)

*should check return of track->GetCodecInfo()

(MediaFileInfo.cpp:46)

There are similar problems when sniffing audio tracks (MediaFileInfo.cpp:58-88)

Once those are fixed, LoadInfo() should also return a status_t. And this should be checked in MediaFileInfoView::Update().

Change History (0)

Note: See TracTickets for help on using tickets.