zabezpieczenia Opcje

[madalena]

Ostatnio dużo czytałem o zabezpieczeniech danych przesyłanych drogą post i get
Nawet w prowadzonym przez mnie serwisie stosując stare techniki ktoś odczytał plik d.php gdzie przechowywane sa hasła

i zaproponował mi zmainą poniższego kodu na następujący

[php:1:38b804c758]<?php
if ((!isset($plik)) || ($plik==""))
{
include("aktualnosci.php");
}
else
{
include($plik);
}
?>[/php:1:38b804c758]

na następujący

[php:1:38b804c758]<?php
$plik = $HTTP_GET_VARS['plik'];
if ($plik||!file_exists($plik))
{
include("aktualnosci.php");
}
else
{
include($plik);
}
?>[/php:1:38b804c758]

Czy to jest poprawne i nie ma zadnego haka

i jak bylo to mozliew ze sprawdzili moje hasła

___
[scanner] Zmoderowane. Prosze używac BBCode

[spenalzo]

W takim przypadku:
[php:1:d4636a2cf5]<?php
$plik = '/etc/passwd';
if(!file_exists($plik))
{
include("404.php");
}
else
{
// wykona sie ten warunek
include("./etc/passwd");
}
?>[/php:1:d4636a2cf5]
I skrypt się wysypie - bo nie ma takiego pliku w bieżącym (./) katalogu.

Ale poprawnie skrypt powinien wyglądać tak:
[php:1:d4636a2cf5]<?php
if(!file_exists("./".$plik))
{
include("404.php");
}
else
{

include("./".$plik);
}
?>[/php:1:d4636a2cf5]