проблема с загрузкой изображений с загрузкой php файла - PullRequest
1 голос
/ 15 ноября 2010

Привет, ребята, я загружаю изображения, используя метод загрузки файла @PhP @ Если я загружаю 10 изображений за раз (Каждое изображение имеет размер 2000/3000). тогда функция сохранения по клику не работает. если я загружаю 5 или менее пяти изображений, то все работает нормально с моим кодированием, я просто включаю свой php-код в этот пост <input value="Save" type="submit" name="SubSave" id="SubSave" onClick="return changes();">

 if($_POST['SubSave'] == "Save"){
    $aid = $_GET['rid'];
 $updcount = $_POST['theValue'];


if($_SESSION["almgtype"]==1 || (GetUserNoPhoto($_SESSION["almgid"]))>(GetTotalPhotoCount1($_SESSION["almgid"],$aid))) {


  $uid = $_SESSION["almgid"];

 for($k=1;$k<=$updcount;$k++) { 
        //echo $k;
   echo $_FILES["uploadfile"]["type"];

if($_FILES["uploadfile".$k]["name"]!="") {

if(($_FILES["uploadfile".$k]["type"] == "image/gif") || ($_FILES["uploadfile".$k]["type"] == "image/jpeg")|| ($_FILES["uploadfile".$k]["type"] == "image/pjpeg") || ($_FILES["uploadfile".$k]["type"] == "image/png")) {

 if ($_FILES["uploadfile".$k]["error"] > 0)
  {
  echo "Error: " . $_FILES["uploadfile".$k]["error"] . "<br />";
  }
else
  {  
       move_uploaded_file($_FILES["uploadfile".$k]["tmp_name"],
      "photoalbum/" . $_FILES["uploadfile".$k]["name"]);
      $uploadfile =  "photoalbum/" . $_FILES["uploadfile".$k]["name"];
  } 
  $path  = $uploadfile;
  $checklist = "select * from amt_photos1 where aid = '".trim($aid)."' and uid = '".trim($uid)."' and path = '".trim($path)."'";
  $chkresult  = mysql_query($checklist);
  if(mysql_num_rows($chkresult) == 0) {
  $i = 0;
  $path =$uploadfile;
  $result = "insert into amt_photos1 set uid = '".trim($uid)."',
                                     aid = '".trim($aid)."',
                                     path = '".trim($path)."',
                                     status = '0',
                                     createdby = '".$_SESSION["almgid"]."',
                                     createddate = now()";

  $rowlist = mysql_query($result) or die("Error:(".mysql_error().")".mysql_error());


                } 
                /**********************  if file already exist means ******************************************/
                else {
                $err= "The Uploaded file name ".$path." Is already exisit in the Album. Rename It or try to add Any other Photos";

                    }
                /**********************  if file already exist means ******************************************/
                $path ="";
                $uploadfile = "";
                $i  = "";
                }  // file extention
                     else {
        $err= "Unable To Upload The File Please Check The File Extention.Try Again Later";
     }

                }
                }
                }



                } // if save close

Ответы [ 3 ]

2 голосов
/ 15 ноября 2010

Возможно, вам потребуется изменить максимальный размер POST в вашем файле конфигурации php.ini (post_max_size).

0 голосов
/ 15 ноября 2010

Так же, как предупреждение: ваш скрипт обработки загрузки сделает полную подрыв вашего сервера совершенно тривиальным:

  1. Вы слепо верите, что значение $_FILES[...]['type'] установлено правильно - это значение полностьюпользовательский контроль, и они могут заполнить «image / jpeg» и загрузить любой тип файла, который они хотят
  2. Вы слепо верите, что значение $_FILES[...]['filename'] установлено правильно - опять же, это значение полностью находится под пользователемконтроль, и они могут заполнить "hackme.php", если они хотят
  3. . Вы слепо записываете файл в свой каталог фотоальбома, но не проверяете, содержит ли предоставленное пользователем имя файла данные пути

Итак, что произойдет, если кто-то загрузит следующий файл:

$_FILES['uploadfile0']['type'] = 'image/gif';
$_FILES['uploadfile0']['filename'] = '../pwn_me.php';

Теперь вы с радостью поместили предоставленный пользователем PHP-скрипт НА ВАШ ВЕБ-СЕРВЕР и теперь можетеделайте все, что захотите.

Кроме того, ваши запросы к базе данных слепо вставляют в запросы одни и те же данные, оставляя вас широко открытыми для атак с использованием SQL-инъекций.Кроме того, вы не проверяете наличие конфликтов имен файлов, пока не переместите файл.Таким образом, кто-то может загрузить вредоносный скрипт, но сделать это только один раз для этого конкретного имени файла.Поздравляем, вы реализовали версионные атаки на своем сервере.Вы будете иметь «pwn_me.php», «pwn_me2.php», «pwn_me3.php», «my_little_pwnme.php» и т. Д.

0 голосов
/ 15 ноября 2010

Вы можете использовать команду phpinfo() для выгрузки вашей конфигурации. Вероятно, как уже говорили другие, вам нужно увеличить размер загрузки и время выполнения.

Их можно изменить с помощью файла .htaccess .

php_value upload_max_filesize 20M
php_value post_max_size 20M
php_value max_execution_time 200
php_value max_input_time 200
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...