PHP中的文件上传问题

Question

嗨，我试图上传一个图像使用PHP脚本。和什么很奇怪的是我只在Internet Explorer中收到以下错误地方有什么脚本正常工作：

Warning: move_uploaded_file(pictures/) [function.move-uploaded-file]: failed to open stream: Is a directory in /home/tntauto1/public_html/admin_add1.php on line 59 

Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move '/tmp/phpcJnHZE' to 'pictures/' in /home/tntauto1/public_html/admin_add1.php on line 59 

Warning: copy() [function.copy]: The first argument to copy() function cannot be a directory in /home/tntauto1/public_html/admin_add1.php on line 60 

下面是脚本：

if(is_uploaded_file($_FILES['image']['tmp_name'])){ 
    if($_FILES['image']['type'] == 'image/jpeg'){ 
     $original = 'original_'.$v_id.'.jpg'; 
     $large = 'large_'.$v_id.'.jpg'; 
     $small = 'small_'.$v_id.'.jpg'; 

    }elseif($_FILES['image']['type'] == 'image/gif'){ 
     $original = 'original_'.$v_id.'.gif'; 
     $large = 'large_'.$v_id.'.gif'; 
     $small = 'small_'.$v_id.'.gif'; 
    }else{ 
     $error = 'Error: The image could not be uploaded. It must be in .jpg, .jpeg or .gif format.'; 
    } 
    if(move_uploaded_file($_FILES['image']['tmp_name'],'pictures/'.$large)){} 
     copy('pictures/'.$large,'pictures/'.$small); 

    $imgsize = getimagesize('pictures/'.$large); //>>>>>>>>>>>>>>>>>>>>>>>>>>>>---- Resize to 480 X 360 
    $width = $imgsize[0]; 
    $height = $imgsize[1]; 
    if(($width > 480) || ($height > 360)){//resize the image 
     $ratio = $width/$height; 
     if(100/$ratio >= 80){//calculates if height of uploaded image is too large 
      $new_width = floor(360 * $ratio); 
      $new_height = 360; 
     }elseif(150 * $ratio > 100){// calculate if width of uploaded image is too large 
      $new_width = 480; 
      $new_height = floor(480/$ratio); 
     } 
     if($_FILES['image']['type'] == 'image/jpeg'){ 
      $img = imagecreatefromjpeg('pictures/'.$large); 
      $img_copy = imagecreatetruecolor($new_width,$new_height); 
      imagecopyresampled($img_copy,$img,0,0,0,0,$new_width,$new_height,$width,$height); 
      imagejpeg($img_copy,'pictures/'.$large,100);  
     } 
     if($_FILES['image']['type'] == 'image/gif'){ 
      $img = imagecreatefromjpeg('pictures/'.$large); 
      $img_copy = imagecreatetruecolor($new_width,$new_height); 
      imagecopyresampled($img_copy,$img,0,0,0,0,$new_width,$new_height,$width,$height); 
      imagejpeg($img_copy,'pictures/'.$large,100);  
     } 
    } 

Answer 1

if($_FILES['image']['type'] == 'image/jpeg'){ 

切勿依赖浏览器提交的MIME类型。

在这种情况下，您的问题是大卫暗示：IE通常（错误地）为JPEG提供image/pjpeg，所以您正在检测未知的文件类型并将$ error设置为Error: The image could not be uploaded. It must be in .jpg, .jpeg or .gif format. ...但尽管如此，您仍尝试无论如何移动文件，尽管没有设置$ small或$ large。

但更重要的是，浏览器提交的类型可能是完全错误的。你不能相信上传的文件名或媒体类型是合适的，所以甚至不要去检查它们。相反，在您调用getimagesize以查明PHP认为图像是什么类型之后，请查看$imgsize[2]。

而且，如果您接受来自一般用户的图片上传，则会出现安全问题。完全可以创建包含HTML标签的有效GIF（或其他文件类型）。然后，当血腥愚蠢的IE浏览器自动访问GIF作为一个页面时，它会检测到HTML标签，决定您告诉它的内容类型是否是错误的，并将其解释为HTML页面，包括任何那里的JavaScript，然后在您的网站的安全上下文中执行。

如果您必须允许从不受信任的来源进行文件上传，并且您没有自己处理图片（这通常会有删除不需要的HTML的副作用），您通常必须从不同的主机名避免他们脚本进入您的网站。

Answer 2

不能移动目录，因为$大没有价值，或被重置。

Answer 3

if($FILES['image']['type'] == 'image/jpeg'){ 

保存文件上传数据的变量应为$_FILES。由于$FILES是一个空的（刚刚使用的）变量，因此您的$large变量也是空的，因此您正在将文件移动到pictures/这是一个目录，就像PHP告诉您的一样。你的$error也应该包含错误消息，因为它们之前都没有if。

避免这种错误的一种方法是用error_reporting设置为E_ALL开发，该应用程序会显示一个通知，指出您的$FILES变量（错字）未定义。