W jakim stopniu ten skrypt jest bezpieczny, jeśli zawiera błędy to jak go poprawić ?

Daj jeszcze na początek:

[PHP] pobierz, plaintext 
<?php
$_GET['page'] = basename($_GET['page']);
?>
[PHP] pobierz, plaintext 

i powinno być ok

A czym różnił by się ten zapis:

od takiego zapisu:

Ja na niektórych serwisach mam takie zabezpieczenie.

[PHP] pobierz, plaintext 
<?php
$adres="/home/user/public_html/"."przedrostek_"."$dzial"."/"."$str".".php";
 
if ($dzial=="" && $str=="" || $dzial=="serwis" && $str=="glowna")
				  {
				  echo '<h1>HOME</h1>';
				  }
 
		  else {
 
						 if (file_exists($adres)) {
														  include "$adres";
														  } 
						 else {
								echo "<h1>Błąd 404</h1>";
								}
					   }
?>
[PHP] pobierz, plaintext 

jak juz dajesz jakis kod php daj go w znaczniki

admin22 - na bezpieczny to on nie wygląda, korzystasz z register_globals i brak zabezpieczenia przed directory transversal. Polecam zapoznać się z tablicami superglobalnymi($_GET, $_POST etc.) oraz z funkcją basename.

morphi99 - ten drugi zapis to raczej jest przestarzały i nie używany

to jak każdy daje coś od siebie to ja też dam ;-)

http://www.beldzio.com/bezpieczenstwo-dost...do-plikow.freez

Pierwsza linia oznacza odwołanie do indeksu page w tablicy $_GET
Druga linia oznacza odwołanie do indeksu w tablicy $_GET, którego wartość znajduje się w stałej page.

Drugi zapis spowoduje wygenerowanie E_WARNING (bądź E_NOTICE, nie pamiętam w tej chwili) zawsze, gdy stała page jest niezdefiniowana. Niestety PHP nie traktuje tego jako błędu krytycznego i uznaje wartość niezdefiniowanej zmiennej jako równa jej nazwie (czyli po wygenerowaniu powyższego błędu symuluje takie działanie:

[PHP] pobierz, plaintext 
<?php
define("page", "page");
?>
[PHP] pobierz, plaintext 

i kontynuuje działanie. Jeśłi masz obniżony poziom raportowania błędów, nawet tego nie zauważysz - ale jest to jeden z podstawowych błędów początkujących programistów.