[PHP] Pomocy! Nie działa prosty skrypt Opcje

[maykee]

Witam serdecznie, mam spory problem. Otóż na swojej stronie mam taki oto skrypcik:

[PHP] pobierz, plaintext 
<? 
$rozsz=".php";
if(file_exists("$id$rozsz")){include("$id$rozsz");
}
else {
include("main$rozsz");
}
?>
[PHP] pobierz, plaintext 

który do tej pory działał bez problemu pod PHP 5, ostatnio jednak byłem zmuszony przeniesc stronę na inny serwer gdzie również jest PHP5 (a konkretnie wersja 5.2.12) i skrypt ten przestał działać, a mianowicie za każdym razem, gdy jest wywolywany za pomoca linka np. index.php?id=inne/kontakt skrypt za kazdym razem wyswietla plik main.php tak jakby plik inne/kontakt.php nie istnial.

Pilnie prosze o pomoc, moze ktos z Was ma jakis pomysl na rozwiazanie problemu?

[adam882]

[PHP] pobierz, plaintext 
<? 
$rozsz=".php";
if(file_exists("$id.$rozsz")){include("$id.$rozsz");
}
else {
include("main.$rozsz");
}
?>
[PHP] pobierz, plaintext 

btw: jeśli to moduł do podstron to lepiej użyć switcha

Ten post edytował adam882 3.06.2010, 12:26:57

[Spawnm]

Odbieraj dane przez $_GET
czyli
$id=$_GET['id'];

I poczytaj o bezpieczeństwie skryptów.
Np. o LFI.

[Fifi209]

[PHP] pobierz, plaintext 
<? 
$rozsz=".php";
if(file_exists("$id.$rozsz")){include("$id.$rozsz");
}
else {
include("main.$rozsz");
}
?>
[PHP] pobierz, plaintext 

Wiesz sam co za bzdety napisałeś?

@topic
Poczytaj o register_globals - wyłączone i dobrze, tylko kod musisz przerobić.

[maykee]

[PHP] pobierz, plaintext 
<? 
$rozsz=".php";
if(file_exists("$id.$rozsz")){include("$id.$rozsz");
}
else {
include("main.$rozsz");
}
?>
[PHP] pobierz, plaintext 

btw: jeśli to moduł do podstron to lepiej użyć switcha

teraz juz nawet main.php nie wyswietla, tylko warning.

Zgadza sie to ma być moduł do podstron.

[Fifi209]

teraz juz nawet main.php nie wyswietla, tylko warning.

Zgadza sie to ma być moduł do podstron.

Było przeczytać mój post.

[maykee]

Było przeczytać mój post.

Przeczytałem, poczytałem i zmieniłem kod na cos takiego:

[PHP] pobierz, plaintext 
<? 
$rozsz=".php";
$strona=$_GET['id'];
if(file_exists("$id$rozsz")){include("$id$rozsz");
}
else {
include("main$rozsz");
}
?>
[PHP] pobierz, plaintext 

ale niestety nic to nie dalo, efekt jest dokladnie ten sam

[croc]

maykee -> twój kod jest nie do przyjęcia. Aż prosi się o atak. Posłucham adama882 i użyj switcha do wyznaczenia dozwolonych podstron.

[Fifi209]

[PHP] pobierz, plaintext 
 
<?php
$rozsz=".php";
$id=$_GET['id'];
 
if(file_exists("$id$rozsz")){include("$id$rozsz");
}
else {
include("main$rozsz");
}
?>
[PHP] pobierz, plaintext 

tak trudno przeczytać posty?

[maykee]

maykee -> twój kod jest nie do przyjęcia. Aż prosi się o atak. Posłucham adama882 i użyj switcha do wyznaczenia dozwolonych podstron.

[PHP] pobierz, plaintext 
 
<?php
$rozsz=".php";
$id=$_GET['id'];
 
if(file_exists("$id$rozsz")){include("$id$rozsz");
}
else {
include("main$rozsz");
}
?>
[PHP] pobierz, plaintext 

tak trudno przeczytać posty?

Dziekuje serdecznie. Przepraszam za klopot ale jestem totalnym laikiem jesli chodzi o php, posty czytalem, ale najwidoczniej nie do konca zrozumialem...

pozdrawiam wszystkich uzytkownikow forum!

[adam882]

Wiesz sam co za bzdety napisałeś?

@topic
Poczytaj o register_globals - wyłączone i dobrze, tylko kod musisz przerobić.

Bzdety? Sam napisałeś to samo. Nie uwzględniałem $_GET bo myślałem, że to fragment ze skryptu.

[Fifi209]

Bzdety? Sam napisałeś to samo. Nie uwzględniałem $_GET bo myślałem, że to fragment ze skryptu.

Nie kompromituj się, chyba sam do końca nie wiesz co napisałeś i jaki byłby wynik z połączenia stringów w Twoim kodzie.