Czy w tym skrypcie jest jakiś błąd?

[Ormi]

Już dłuższy czas siedzę nad tym skryptem i szukam jakiegoś błędu. Coś mi mówi, że jakiś tam jest, tylko go znaleźć nie mogę Może wy coś znajdziecie.
Oto plik admin.php:

<?



  require("config.inc.php");
  

  if (isset($_REQUEST['poprawny_login']) || isset($_REQUEST['poprawne_haslo'])) {
  	die ('<img src="http://www.programuj.com/moje/comnews/pozdrawiam.gif">');
  }
    import_request_variables("GP","");

  if ($haslo==$poprawne_haslo && $login==$poprawny_login)

  {

   include("head.inc");

//różne nieistotne dla nas instrukcje :)


?>

Tu powinienem wytłumaczyć dlaczego autor skryptu wstawił fragment: if (isset($_REQUEST['poprawny_login']) || isset($_REQUEST['poprawne_haslo'])) {
die ('<img src="http://www.programuj.com/moje/comnews/pozdrawiam.gif">');
Otóż w starszej wersji tego nie było, a jakie były tego konsekwencje można zobaczyć: http://hack.pl/forum/thread4268,comboys_news_17.html
Czy można jakoś to ominąć? Może gdzie indziej znajdziecie błąd.
login.php:

<?

 require("config.inc.php");

 import_request_variables("GPC","");

 if ($login==$poprawny_login && $haslo==$poprawne_haslo)

 {

  setcookie("haslo",$haslo);

  setcookie("login",$login);

  Header("Location: admin.php");

 } else

 {

  include("head.inc");

  echo("<BR><BR><CENTER><FONT color=red><B>Błędy login i/lub hasło</B> !</FONT><BR><BR><BR>\n");

  echo("<A href='index.php'>Powrót do strony logowania</A></CENTER>");



  include("stopka.inc");

 }



?>

Oraz index.php:

 <? include("head.inc"); ?>

 <BR><BR>

 <CENTER>

 Aby mieć możliwość administracji systemem newsów, musisz się zalogować:<BR><BR>

 <FORM action="login.php" method="post">

 <TABLE border=0 >

 <TR><TD>Login: </TD><TD><INPUT type=text name=login></TD></TR>

 <TR><TD>Hasło: </TD><TD><INPUT type=password name=haslo></TD></TR>

 <TR><TD colspan=2 align=center><BR><INPUT type=submit value="       Zaloguj się       "></TD></TR>

 </TABLE>

 </CENTER>

 <? include("stopka.inc"); ?>

I to by było na tyle

[blooregard]

if (isset($_REQUEST['poprawny_login']) || isset($_REQUEST['poprawne_haslo'])){
  	die ('<img src="http://www.programuj.com/moje/comnews/pozdrawiam.gif">');
  }

Warunek sprawdza, czy są ustawione 'poprawny_login' LUB 'poprawne_haslo'. Ponieważ są (bo ustawiasz je wcześniej w login.php jako ciasteczka, a $_REQUEST odpowiada tablicom $_GET, $_POST i $_COOKIE), dla poprawnego hasla LUB loginu warunek będzie spelniony, więc wykona się die() z tym gif-em.

Poprawnie powinno więc być:

if ( !(isset($_REQUEST['poprawny_login']))  || !(isset($_REQUEST['poprawne_haslo'])) )

- czyli przy nie spełnieniu jednego z warunków pojawia sie ten gif "pozdrawiam.gif" , bo zakładam, że o to Ci chodziło?

[Ormi]

Nie, nie nie Widzę, że źle pytanie sformuławem. Ten skrypt działa bez twoich poprawek, blooregard. Mi chodziło o błąd w sensie błąd w zabezpieczeniach. Czy macie jakieś pomysły jak obejść te zabezpieczenia

[rip]

Już dłuższy czas siedzę nad tym skryptem i szukam jakiegoś błędu.

Caly skrypt to jedna wielka pomylka. Powinienes zapytac: jak to zrobic, aby bylo dobrze.

[blooregard]

Aha. No to faktycznie źle sformułowałeś.

Mnie udało się złamać to w ten sposób:

login.php?poprawny_login=a&login=a&poprawne_haslo=a&haslo=a

Spowodowało to, że w pliku login.php sprawdzenie warunku:

 if ($login==$poprawny_login && $haslo==$poprawne_haslo)

 {

było prawdziwe, bo (a==a && a==a).
A pod każdą z tych zmiennych można podstawic dowolną wartość poprzez GET-a, bo wcześniej uzyłeś f-kcji

 import_request_variables("GP","");

, co jest niemal równoznaczne z użyciem register_globals.

Nie wiem, co jest u Ciebie w pliku config.inc.php i jak odczytujesz ten prawidlowy_login i prawidlowe_haslo, ale to raczej chyba nie ma znaczenia w tym przypadku.

[Ormi]

To login.php?poprawny_login=a&login=a&poprawne_haslo= a&haslo=a też wymyśliłem, ale nie działa Po wpisaniu wyświetla się "nieprawidłowy login i/lub hasło", więc to nie to :P