pdf php upload - Запрос в Google

<form action="<?php print $PHP_SELF?>" enctype="multipart/form-data" method="post">
   Last Name:<br /> <input type="text" name="name" value="" /><br />
   Class Notes:<br /> <input type="file" name="classnotes" value="" /><br />
   <p><input type="submit" name="submit" value="Submit Notes" /></p>
</form>

<?php
   define ("FILEREPOSITORY","./");

   if (is_uploaded_file($_FILES['classnotes']['tmp_name'])) {

      if ($_FILES['classnotes']['type'] != "application/pdf") {
         echo "<p>Class notes must be uploaded in PDF format.</p>";
      } else {
         $name = $_POST['name'];
         $result = move_uploaded_file($_FILES['classnotes']['tmp_name'], FILEREPOSITORY."/$name.pdf");
         if ($result == 1) echo "<p>File successfully uploaded.</p>";
         else echo "<p>There was a problem uploading the file.</p>";
      } #endIF
   } #endIF
?>

Работоспособность не проверял

Kusss
Есть тут баг один! Маленький баг но есть.
Если взять картинку, и сменить разрешение с jpg=>pdf, то скрипт благополучно пропустит его творение. про этот способ я знал, пришлось вернуться к нему!

---

Спустя 5 минут, 9 секунд m4a1fox написал(а):

---

Ха! Нашел выход! Поделиться???

m4a1fox

Invis1ble
Вот ты меня убиваешь! Думаешь я ни на что не годен?

Короче дело такое. методы. Ловите

public $allowType = array('image/jpeg', 'image/jpg', 'image/png', 'image/gif');
public function TypeOfFile(){
        if($_FILES["filename"]["type"]=="application/pdf" && $this->GetImageSize() === TRUE){
            return true;
        }        
    }
    
    public function GetImageSize() {
        if(!empty($_FILES["filename"]["name"])){
            $type = getimagesize($_FILES['filename']['tmp_name']);
                if(in_array($type['mime'], $this->allowType)){
                        return false;
                    }else{
                        return true;
                    }
            }    
    }

Скажите где не прав, знать то хоть буду!

m4a1fox
Нееееет!!!
Я пощутил )

Какое это имеет отношение к pdf?

ахаха
это же проверка для картинок, и то, далеко - не панацея

Invis1ble
Ваше предложение?

Invis1ble, getimagesize вернет false, если "изображение" - не изображение. И уж если вернуло массив, то можно просто пересоздать изображение, избавившись тем самым от потенциальных скриптов.

neadekvat

m4a1fox
Ты этот код хоть пробовал запустить? Это же бред полный

---

Спустя 1 минута, 18 секунд Invis1ble написал(а):

---

neadekvat

Invis1ble

m4a1fox
покажи, как ты пользуешься этим добром

Invis1ble
Всмысле показать? Не догоняю! Если файл не картинка и с типом aplication/pdf то перемещаю его в папку, иначе false.

В смысле как объект создаешь, какие методы вызываешь. Т.е. код полностью приведи

Invis1ble
Не вопрос

class downloadPdf {
    private $path = 'file/'; // path for load on the folder 
    private static $table = 'journal_numbers'; // table in the DB
    private $allowType = array('image/jpeg', 'image/jpg', 'image/png', 'image/gif');
    private $ok = '<div style="background-color: #AEF9F8; 
            width: 110px; padding: 5px;"><font color=green>
            file download. Ok</font></div>';
    private $error = '<div style="background-color: grey; 
            width: 450px; padding: 5px;"><font color=red>
            Some error isset. Please chenge format of dowload file or download file</font></div>';
    
    
    public function downloadFile(){
        if($this->CheckIsUploadedFile() === true && $this->TypeOfFile() === true){
            $this->MoveFile();
            $this->chengePdfLoad();
            return $this->ok;
        }else{
            return $this->error;
        }
    }
    
    
    
    /**
     * check the file is isset 
     */
    private function CheckIsUploadedFile() {
        if(is_uploaded_file($_FILES["filename"]["tmp_name"])) {
            return true;
        }
    }
    
    /**
     * method for moove file in the folder 
     */
    public function MoveFile() {
        move_uploaded_file($_FILES["filename"]["tmp_name"], "".$this->path."".$_POST['pdf_number'].".pdf");
            return true;
        }
    
    
    /**
     * method for chanche the field 'lpdf_load'=>1 
     */ 
    public function chengePdfLoad(){
        $sql = "UPDATE `".self::$table."` SET 
                `pdf_load` = '1'
                WHERE `number` = '".$_POST['pdf_number']."'
                ";
        return connectDb::sql($sql);
    }
    
    public function TypeOfFile(){
        if($_FILES["filename"]["type"]=="application/pdf" && $this->GetImageSize() === TRUE){
            return true;
        }        
    }
    
    public function GetImageSize() {
        if(!empty($_FILES["filename"]["name"])){
            $type = getimagesize($_FILES['filename']['tmp_name']);
                if(in_array($type['mime'], $this->allowType)){
                        return false;
                    }else{
                        return true;
                    }
            }    
    }
 }

Подскажите, если что!

1. при загрузке любой не-картинки getimagesize() вернет false
2. индекса 'mime' у булева значения быть не может (как и любого другого индекса) - здесь получаем Notice
3. булево false не содержится в массиве запрещенных mime-типов
и..... бинго! ф-ция GetImageSize() возвращает true!

А теперь попробуй загрузить тотже exe-шник

Invis1ble

m4a1fox

Invis1ble
Ну дык картинки не проходят, не проходят и картинки с расширение pdf, а вот pdf проходит! Мне кажется, это нормально! Но! Ваше предложения какие? Что бы вы изминили?

m4a1fox
Мое предложение - выкинуть ВЕСЬ этот УГ в топку и оставить условие

if ($_FILES["filename"]["type"]=="application/pdf") {
    // move_uploaded_file();
}

и предупреждать юзверей о возможности того, что файлы содержат вирусы. Ну либо все таки заливать их на сервер и проверять антивирусом, но тоже не панацея.

Invis1ble
Считаете? Хм. Но это будет админка! :)! Окей. Я ваше предложение понял! Еще вопрос не по теме ветки.
Есть подключение к БД

class connectDb extends defaultVar {
    
    private $connection;
    
    function __construct() {
        $this->open_connection();
        //echo "Connection is set";
    }
    
    private function open_connection() {
        $this->connection = mysql_connect($this->DB_HOST, $this->DB_USER, $this->DB_PASS);
        if(!$this->connection) {
            die("Database connection faild: ".mysql_error());
        }
        else {
            $db_select = mysql_select_db($this->DB_NAME);
            if(!$db_select) {
                die ("Database selected faild: ".mysql_error());
            }
        }
        mysql_query("set names utf8") or die ("set names utf8 faild");
    }
    
    static function sql ($query) {
        $result = mysql_query($query) or die(mysql_error());
        if(!$result) {
            die("Database query failed: ".mysql_error());
        }
        return $result;
    }
}

Говорят что там не хватает

Если свой хостинг, то можно использовать никсовую утилитку file

waldicom
ты о проверке mime? но ведь по идее и file можно "обмануть"...

Invis1ble

m4a1fox
А чтоб я так знал
Наверное да.

Invis1ble
Лады! Вот и поговорили! Что то вы сударь не в настроении сегодня!

m4a1fox

Invis1ble
Да так... кажется просто мне!