movie: fix some checks

Fixes invalid movie error
This commit is contained in:
Valentin Vanelslande 2018-10-05 17:20:09 -05:00
parent b163502744
commit dcd6e7f99f
1 changed files with 3 additions and 3 deletions

View File

@ -453,7 +453,7 @@ static boost::optional<CTMHeader> ReadHeader(const std::string& movie_file) {
void Movie::PrepareForPlayback(const std::string& movie_file) { void Movie::PrepareForPlayback(const std::string& movie_file) {
auto header = ReadHeader(movie_file); auto header = ReadHeader(movie_file);
if (header != boost::none) if (header == boost::none)
return; return;
init_time = header.value().clock_init_time; init_time = header.value().clock_init_time;
@ -468,7 +468,7 @@ void Movie::PrepareForRecording() {
Movie::ValidationResult Movie::ValidateMovie(const std::string& movie_file, u64 program_id) const { Movie::ValidationResult Movie::ValidateMovie(const std::string& movie_file, u64 program_id) const {
LOG_INFO(Movie, "Validating Movie file '{}'", movie_file); LOG_INFO(Movie, "Validating Movie file '{}'", movie_file);
auto header = ReadHeader(movie_file); auto header = ReadHeader(movie_file);
if (header != boost::none) if (header == boost::none)
return ValidationResult::Invalid; return ValidationResult::Invalid;
return ValidateHeader(header.value(), program_id); return ValidateHeader(header.value(), program_id);
@ -476,7 +476,7 @@ Movie::ValidationResult Movie::ValidateMovie(const std::string& movie_file, u64
u64 Movie::GetMovieProgramID(const std::string& movie_file) const { u64 Movie::GetMovieProgramID(const std::string& movie_file) const {
auto header = ReadHeader(movie_file); auto header = ReadHeader(movie_file);
if (header != boost::none) if (header == boost::none)
return 0; return 0;
return static_cast<u64>(header.value().program_id); return static_cast<u64>(header.value().program_id);