aboutsummaryrefslogtreecommitdiff
path: root/plugins/video_ffmpeg_helper
diff options
context:
space:
mode:
authorFabio Varesano <fax8@13637.no-reply.drupal.org>2007-01-08 00:45:03 +0000
committerFabio Varesano <fax8@13637.no-reply.drupal.org>2007-01-08 00:45:03 +0000
commitd2c9046205bd5c9a61ffe1792470defbfa0250ce (patch)
treeb5828254b36090168a46f47d29577116eb100235 /plugins/video_ffmpeg_helper
parent6e1dfe1dbe8c70a04234df5eadbdf9b79b612139 (diff)
downloadvideo-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.module6
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();