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

[blooregard]

Aha. Więc za mało kodu przytoczyłeś .
A co jest w tym config.inc.php ?

[Ormi]

Oto i on:

<?



$poprawny_login = "root";    // Tutaj wpisz swój login

$poprawne_haslo = "qwerty";  // A tutaj hasło

$katalog = "comnews/";       // Nazwa katalogu w którym znajduje się skrypt. Może to być

                             // katalog bezwzględny lub względem pliku w którym skrypt

                             // jest includowany.

                             

$ile = 3;                    // Iloœć najnowsyzch newsów które zostanš wyœwietlone.



?>

[blooregard]

No to może tak: (zakładamy, że intruz wie, że to w pliku config.inc.php jest zapisane prawidłowe hasło i login,ale nie ma do niego dostępu i nie może go odczytać, tak? )

<?php
// pliczek xploit.php 
require("config.inc.php");

setcookie("haslo",$poprawne_haslo);
setcookie("login",$poprawny_login);

header("Location:admin.php");

?>

U mnie zadziałało, na Twoim kodzie.

[Ormi]

No tak, ale tego exploita nie da się użyć na serwerze www, bo tam przecież nie umieścimy tego twojego exploita