ben/download-dispatcher
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add mkvinfo check for bad files before moving into place

Browse Source
This commit is contained in:
Ben Roberts
2013-05-26 21:56:39 +01:00
parent 872e2ef8d0
commit 8aa10a0746
1 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
source/lib/DownloadDispatcher/Source/Plugin/TV.class.php
View File

@@ -261,7 +261,21 @@ class DownloadDispatcher_Source_Plugin_TV extends DownloadDispatcher_Source_Plug
throw new DownloadDispatcher_Exception_UnacceptableContent($source_file); throw new DownloadDispatcher_Exception_UnacceptableContent($source_file);
} }
}
case 'mkv': {
// Verify that the file isn't a fake
$safe_source_file = escapeshellarg($source_file);
$command = "mkvinfo {$safe_source_file}";
DownloadDispatcher_LogEntry::debug($this->log, "Verifying '{$source_file}' contents with command: {$command}");
list($code, $output, $error) = DownloadDispatcher_ForegroundTask::execute($command, $source_dir);
if ($code != 0) {
DownloadDispatcher_LogEntry::warning($this->log, "Failed to parse contents of '{$source_dir}/{$source_file}'.");
throw new DownloadDispatcher_Exception_UnacceptableContent($source_file);
}
} // continue into the next case } // continue into the next case
default: { default: {
DownloadDispatcher_LogEntry::info($this->log, "Copying '{$source_file}' to '{$destination_dir}'."); DownloadDispatcher_LogEntry::info($this->log, "Copying '{$source_file}' to '{$destination_dir}'.");
$result = copy("{$source_dir}/{$source_file}", "{$destination_dir}/{$source_file}"); $result = copy("{$source_dir}/{$source_file}", "{$destination_dir}/{$source_file}");