Skrypt includowania PHP Opcje

[Mlodycompany]

Witam. Wpadłem znów na genialny pomyśł opracowania skryptu includującego strone. Oto jego Skrypt:

[PHP] pobierz, plaintext 
<?php
$get_name = addslashes($_GET['name']);
$dir = "./";
if(!($fd = opendir($dir)))
exit("Bład");
 
if($get_name != "" && $get_name == $get_name){
while($file = readdir($fd)){
if($get_name == $file){
include($file);
}
}
}
?>
[PHP] pobierz, plaintext 

I teraz zaczoł się problem. A mianowicie. URL wygląda tak :index.php?name=home
zmienna $get_name = home
zmienna $file nie jest równa zmiennej $get_name ponieważ zmienna $file powinna przybrac wartosc home.php a home != home.php

I wpadłem na kolejny genialny pomysł. Oto on: dodac do $get_name koncówke .php i wtedy $get_name bedzie rowna $file. Problem jest w tym, że nie wiem jak można dodac tą końcówkę i czy wogóle się da to zrobic. Czy macie jakieś propozycje?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

[marcio]

Może skorzystaj z tego:

[PHP] pobierz, plaintext 
<?php
$page = (isset($_GET['page'])) ? $_GET['page'] : 'index';
 
$file = './pages/'.$page.'.php';
 
if (file_exists($file)) {
	include($file);
}
else {
	include('./pages/index.php');
}
?>
[PHP] pobierz, plaintext 

Niepotrzebnie babrasz się z addslashes oraz kompletnie niepotrzebnie otwierasz katalog i szukasz w nim konkretnego pliku. Lepiej sprawdzić czy takowy file_exists czyli czy plik_istnieje.

Jak juz chcesz tak robic to dawaj dozwolone pliki do tablicy i przy includowaniu sprawdzaj czy plik znajduje sie w tablicy bo to jest przyklad fajnego LFI (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) a jak nie zrobc na switch'u

[phpion]

Jak juz chcesz tak robic to dawaj dozwolone pliki do tablicy i przy includowaniu sprawdzaj czy plik znajduje sie w tablicy bo to jest przyklad fajnego LFI (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) a jak nie zrobc na switch'u

Racja. Ale w sumie wystarczy przelecieć $_GET['page'] funkcją basename() i już uniemożliwisz wyjście poza wskazany katalog. Szkoda babrać się z tablicami czy switchami.