From 11d212a83f3f52cb562d2d7b782511e6ce059648 Mon Sep 17 00:00:00 2001 From: "Paul S. Owen" Date: Sun, 27 Jan 2002 14:13:59 +0000 Subject: Fixed content-type/length bugs git-svn-id: file:///svn/phpbb/trunk@1970 89ea8834-ac86-4346-8a33-228a782c2dd0 --- phpBB/admin/admin_users.php | 20 +++++++++++--------- phpBB/profile.php | 7 ++++--- 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/phpBB/admin/admin_users.php b/phpBB/admin/admin_users.php index 491c99706f..5e7d2ede71 100644 --- a/phpBB/admin/admin_users.php +++ b/phpBB/admin/admin_users.php @@ -367,6 +367,7 @@ if( $mode == "edit" || $mode == "save" && ( isset($HTTP_POST_VARS['username']) | { case "jpeg": case "pjpeg": + case "jpg": $imgtype = '.jpg'; break; case "gif": @@ -393,12 +394,12 @@ if( $mode == "edit" || $mode == "save" && ( isset($HTTP_POST_VARS['username']) | if( $this_userdata['user_avatar_type'] == USER_AVATAR_UPLOAD && $this_userdata['user_avatar'] != "" ) { - if( @file_exists("./" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']) ) + if( @file_exists("./../" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']) ) { - @unlink("./" . $board_config['avatar_path'] . "/". $this_userdata['user_avatar']); + @unlink("./../" . $board_config['avatar_path'] . "/". $this_userdata['user_avatar']); } } - @copy($user_avatar_loc, "./" . $board_config['avatar_path'] . "/$avatar_filename"); + @copy($user_avatar_loc, "./../" . $board_config['avatar_path'] . "/$avatar_filename"); $avatar_sql = ", user_avatar = '$avatar_filename', user_avatar_type = " . USER_AVATAR_UPLOAD; } @@ -457,15 +458,16 @@ if( $mode == "edit" || $mode == "save" && ( isset($HTTP_POST_VARS['username']) | } @fclose($fsock); - if( preg_match("/Content-Length\: ([0-9]+)[^\/]+Content-Type\: image\/[x\-]*([a-z]+)[\s]+/i", $avatar_data, $file_data) ) + if( preg_match("/Content-Length\: ([0-9]+)[^\/ ][\s]+/i", $avatar_data, $file_data1) && preg_match("/Content-Type\: image\/[x\-]*([a-z]+)[\s]+/i", $avatar_data, $file_data2) ) { - $file_size = $file_data[1]; - $file_type = $file_data[2]; + $file_size = $file_data1[1]; + $file_type = $file_data2[1]; switch( $file_type ) { case "jpeg": case "pjpeg": + case "jpg": $imgtype = '.jpg'; break; case "gif": @@ -501,12 +503,12 @@ if( $mode == "edit" || $mode == "save" && ( isset($HTTP_POST_VARS['username']) | if( $this_userdata['user_avatar_type'] == USER_AVATAR_UPLOAD && $this_userdata['user_avatar'] != "") { - if( file_exists("./" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']) ) + if( file_exists("./../" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']) ) { - @unlink("./" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']); + @unlink("./../" . $board_config['avatar_path'] . "/" . $this_userdata['user_avatar']); } } - @copy($tmp_filename, "./" . $board_config['avatar_path'] . "/$avatar_filename"); + @copy($tmp_filename, "./../" . $board_config['avatar_path'] . "/$avatar_filename"); @unlink($tmp_filename); $avatar_sql = ", user_avatar = '$avatar_filename', user_avatar_type = " . USER_AVATAR_UPLOAD; diff --git a/phpBB/profile.php b/phpBB/profile.php index 42be984914..fa4ce327bc 100644 --- a/phpBB/profile.php +++ b/phpBB/profile.php @@ -881,15 +881,16 @@ if( isset($HTTP_GET_VARS['mode']) || isset($HTTP_POST_VARS['mode']) ) } @fclose($fsock); - if( preg_match("/Content-Length\: ([0-9]+)[^\/]+Content-Type\: image\/[x\-]*([a-z]+)[\s]+/i", $avatar_data, $file_data) ) + if( preg_match("/Content-Length\: ([0-9]+)[^\/ ][\s]+/i", $avatar_data, $file_data1) && preg_match("/Content-Type\: image\/[x\-]*([a-z]+)[\s]+/i", $avatar_data, $file_data2) ) { - $file_size = $file_data[1]; - $file_type = $file_data[2]; + $file_size = $file_data1[1]; + $file_type = $file_data2[1]; switch( $file_type ) { case "jpeg": case "pjpeg": + case "jpg": $imgtype = '.jpg'; break; case "gif": -- cgit v1.2.1