Как удалять zip файл с сервера?

Question

[Петро Бойко]

Привет всем подскажите как удалять zip rar файлы из сервера ??

Использую такой код удалят все кроме zip rar

if (!isset($_GET['remove'])) die('Не задан файл для удаления');
if (@unlink("archiv/".$_GET['remove']))

Comments

[riot26]

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

GET http://site.com/script.php?remove=../../../var/log/very_important_file.log

[Дмитрий]

Добрый вечер.
Уберите из кода "@"

[Anton Mashletov]

Используйте полный путь к файлу:

$_SERVER['DOCUMENT_ROOT'] . '/archiv' . $_GET['remove'];

Answer 1

[Андрей Швед]

Попробуйте исключить вариант не хватки прав на удаление с помощью функции chmod()
Например:

if (!isset($_GET['remove'])) die('Не задан файл для удаления');
@chmod("archiv/".$_GET['remove'], 0666);
if (@unlink("archiv/".$_GET['remove']))

Что бы не гадать, попробуйте получить развернутый ответ функции unlink("archiv/".$_GET['remove']) - включить вывод ошибок на сервере.

Comments

[Sanovskiy]

?remove=/ver/www/index.php
Очень небезопасный код.

[Андрей Швед]

Sanovskiy Вы правы - код не безопасный.
Если проблему с удалением архивов мы уже решили, то следует дополни код функциями очистки получаемых данных:

if (!isset($_GET['remove'])) die('Не задан файл для удаления');
$path = $_GET['remove'];
$path = trim($path);
$path = stripslashes($path);
$path = htmlspecialchars($path);
@chmod("archiv/".$path, 0666);
if (@unlink("archiv/".$path))

[Sanovskiy]

Андрей Швед, Петро Бойко,

<?php
function sanitize_input($value){
    return htmlspecialchars(stripslashes(trim($value)));
}
if (!isset($_GET['remove'])){
    die('Не задан файл для удаления');  
} 
$path = "archiv/".sanitize_input($_GET['remove']);
if (!file_exists($path)){
    die('Файл не найден');
}
if (!is_writable($path)){
    die('Файл недоступен на запись');
}

// chmod($path, 0666); // Эта строка вообще не нужна.
unlink("archiv/".$path)
die('Файл удален');

Оптимальный вариант.

Функцию sanitize_input() стоит вынести куда-нибудь в common и использовать для очистки входных данных.