nettika/phpmyadmin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Protect against inclusion of arbitrary file and HTTP header splitting.

Browse Source
This commit is contained in:
Michal Čihař
2009-03-24 12:24:45 +00:00
parent 184934bb10
commit 69bfbf11c7
2 changed files with 52 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
bs_disp_as_mime_type.php
View File

@@ -6,16 +6,42 @@
* @package BLOBStreaming
*/
/**
* Core library.
*/
require_once './libraries/common.inc.php';
// load PMA configuration
$PMA_Config = $_SESSION['PMA_Config'];
// retrieve BS server variables from PMA configuration
$bs_server = $PMA_Config->get('BLOBSTREAMING_SERVER');
if (empty($bs_server)) die('No blob streaming server configured!');
// Check URL parameters
PMA_checkParameters(array('reference', 'c_type'));
// Increase time limit, because fetching blob might take some time
set_time_limit(0);
$filename = isset($_REQUEST['file_path']) ? $_REQUEST['file_path'] : NULL;
$c_type = isset($_REQUEST['c_type']) ? $_REQUEST['c_type'] : NULL;
$reference = $_REQUEST['reference'];
/*
* FIXME: Maybe it would be better to check MIME type against whitelist as
* this code sems to support only few MIME types (check
* function PMA_BS_CreateReferenceLink in libraries/blobstreaming.lib.php).
*/
$c_type = preg_replace('/[^A-Za-z0-9/_-]/', '_', $_REQUEST['c_type']);
$filename = 'http://' . $bs_server . '/' . $reference;
if (isset($filename) && isset($c_type))
{
$hdrs = get_headers($filename, 1);
if (is_array($hdrs))
if ($hdrs === FALSE) die('Failed to fetch headers');
$fHnd = fopen($filename, "rb");
if ($fHnd === FALSE) die('Failed to open remote URL');
$f_size = $hdrs['Content-Length'];
header("Expires: 0");
@@ -27,15 +53,10 @@ if (isset($filename) && isset($c_type))
header('Content-length: ' . $f_size);
header("Content-disposition: attachment; filename=" . basename($filename));
$fHnd = fopen($filename, "rb");
if ($fHnd)
{
$pos = 0;
$content = "";
while (!feof($fHnd))
{
while (!feof($fHnd)) {
$content .= fread($fHnd, $f_size);
$pos = strlen($content);
@@ -47,6 +68,3 @@ if (isset($filename) && isset($c_type))
flush();
fclose($fHnd);
}
}
?>

3
bs_play_media.php
View File

@@ -40,7 +40,8 @@
$bs_file_path = "http://" . $bs_server . '/' . $bsReference;
if (isset($customType) && $customType)
$bs_file_path = "bs_disp_as_mime_type.php?file_path=" . urlencode($bs_file_path) . "&c_type=" . urlencode($mediaType);
$bs_file_path = 'bs_disp_as_mime_type.php' . PMA_generate_common_url(array('reference' => $bsReference, 'c_type' => $mediaType));
?>
<html>