diff options
| author | Fabio Varesano <fax8@13637.no-reply.drupal.org> | 2007-01-08 00:45:03 +0000 |
|---|---|---|
| committer | Fabio Varesano <fax8@13637.no-reply.drupal.org> | 2007-01-08 00:45:03 +0000 |
| commit | d2c9046205bd5c9a61ffe1792470defbfa0250ce (patch) | |
| tree | b5828254b36090168a46f47d29577116eb100235 /plugins/video_ffmpeg_helper | |
| parent | 6e1dfe1dbe8c70a04234df5eadbdf9b79b612139 (diff) | |
| download | video-d2c9046205bd5c9a61ffe1792470defbfa0250ce.tar.gz video-d2c9046205bd5c9a61ffe1792470defbfa0250ce.tar.bz2 | |
* added escaping to filename in ffmpeg_helper which caused filenames with spaces not to work
* fixed some incorrect usage of boolean operators
* fixed some node links
Diffstat (limited to 'plugins/video_ffmpeg_helper')
| -rw-r--r-- | plugins/video_ffmpeg_helper/video_ffmpeg_helper.module | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/plugins/video_ffmpeg_helper/video_ffmpeg_helper.module b/plugins/video_ffmpeg_helper/video_ffmpeg_helper.module index 176979a..96e16dc 100644 --- a/plugins/video_ffmpeg_helper/video_ffmpeg_helper.module +++ b/plugins/video_ffmpeg_helper/video_ffmpeg_helper.module @@ -203,10 +203,12 @@ function video_ffmpeg_helper_nodeapi(&$node, $op, $teaser) { function _video_ffmpeg_helper_get_video_info(&$node) { + // escape file name for safety + $file = escapeshellarg($_SESSION['video_upload_file']->filepath); // create the full command to execute - $command = variable_get('video_ffmpeg_helper_ffmpeg_path', '/usr/bin/ffmpeg') . ' -i ' . $_SESSION['video_upload_file']->filepath; + $command = variable_get('video_ffmpeg_helper_ffmpeg_path', '/usr/bin/ffmpeg') . ' -i ' . $file; - print $command; + drupal_set_message('executing' . $command); //execute the command ob_start(); |