[PHP 5.2] Votre avis sur une faille de sécurité

Aspic · 06/10/2011, 22h01

Bonsoir à tous,

J'ai un ami qui a écrit ce code pour son site en activant les register global (bien que je lui ai dit que c'était une source de problèmes...) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
<?
error_reporting(E_ALL ^ E_NOTICE);
 
$adminpass="monpassadmin";
 
$connecter=0;
if ($pw == $adminpass){
Setcookie("pw",$pw);
$connecter=1;
}elseif($pw){
$erreur="Erreur de mot de passe";
}
if($HTTP_COOKIE_VARS["pw"]==$adminpass) $connecter=1;
if($deco){
Setcookie("passe");
$connecter=0;
}
 
if ($connecter==1)
{
	echo "ok connecte":
}
else
{
	?>
<html>
<body>
  <table>
    <tr>
      <td><center>Connexion</center></td>
    </tr>
    <tr>
      <td><form action="" method="POST">
          <?=$erreur?>
          <u>Mot de passe admin</u>
          <input type="password" value="" name="pw">
          <input type="submit" value="Go">
        </form></td>
    </tr>
  </table>
</body>
</html>
    <?
}
?>

A première vue, je ne vois pas de faille (et je ne suis pas spécialiste dans le hacking) mais vu que le register_global est activé, j'ai peur qu'il y ait des failles.

Qu'en pensez vous ? Y'a t-il un moyen de bypasser le mot de passe dans ce cas ?

Merci et bonne soirée

transgohan · 07/10/2011, 07h17

Il est en effet possible de renseigner n'importe quelle valeur dans l'url et ce sans avoir besoin de passer par le formulaire de connexion.
De plus ton ami a utilisé des short_tags... Remplaces les <? par des <?php

Aspic · 07/10/2011, 18h03

Citation:

Envoyé par transgohan

Il est en effet possible de renseigner n'importe quelle valeur dans l'url et ce sans avoir besoin de passer par le formulaire de connexion.
De plus ton ami a utilisé des short_tags... Remplaces les <? par des <?php

D'accord mais dans ce cas présent, l'injection de variables par l'URL n'est pas un problème ?

Merci pour les shorts tags, je lui dirais

stealth35 · 07/10/2011, 18h07

fait plutôt

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

error_reporting(-1);

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$_COOKIE

Marc3001 · 07/10/2011, 22h02

Mettre le password dans un cookie ne me paraît pas une bonne idée.

J'aurais plutot utiliser un flag (au lieu du pass) et dans une variable de session.

Aspic · 08/10/2011, 11h06

Merci à vous,.
N'hésitez pas à me faire d'autres remarques

En tout cas, bien que le code soit mauvais, il ne semble pas vulnérable

kdmbella · 08/10/2011, 11h51

voici un article datant de 2007 qui te sera certainement utile

Marc3001 · 10/10/2011, 21h21

Citation:

Envoyé par Aspic

Merci à vous,.
N'hésitez pas à me faire d'autres remarques

En tout cas, bien que le code soit mauvais, il ne semble pas vulnérable

Le code non mais le mot de passe dans un cookie c'est pas terrible niveau sécu. Un cookie ça se récupère.....

07/10/2011, 07h17	#2
transgohan Membre Expert Baptiste ROUSSEL Étudiant Inscription : janvier 2011 Messages : 808 Détails du profil Informations personnelles : Nom : Baptiste ROUSSEL Localisation : France, Territoire de Belfort (Franche Comté) Informations professionnelles : Activité : Étudiant Informations forums : Inscription : janvier 2011 Messages : 808 Points : 1 523 Points : 1 523	Il est en effet possible de renseigner n'importe quelle valeur dans l'url et ce sans avoir besoin de passer par le formulaire de connexion. De plus ton ami a utilisé des short_tags... Remplaces les <? par des <?php __________________ Toujours se souvenir que la majorité des ennuis viennent de l'espace occupé entre la chaise et l'écran de l'ordinateur.
	00

07/10/2011, 18h07	#4
stealth35 Modérateur Inscription : septembre 2010 Messages : 7 131 Détails du profil Informations forums : Inscription : septembre 2010 Messages : 7 131 Points : 8 491 Points : 8 491	fait plutôt Code : Sélectionner tout - Visualiser dans une fenêtre à part error_reporting(-1); et Code : Sélectionner tout - Visualiser dans une fenêtre à part $_COOKIE __________________ http://blog.stealth35.com/
	00

07/10/2011, 22h02	#5
Marc3001 Membre chevronné Ingénieur développement logiciels Inscription : février 2008 Messages : 430 Détails du profil Informations personnelles : Sexe : Âge : 28 Localisation : France, Ille et Vilaine (Bretagne) Informations professionnelles : Activité : Ingénieur développement logiciels Informations forums : Inscription : février 2008 Messages : 430 Points : 682 Points : 682	Mettre le password dans un cookie ne me paraît pas une bonne idée. J'aurais plutot utiliser un flag (au lieu du pass) et dans une variable de session. __________________ Le logiciel, c'est comme le sexe, c'est meilleur quand c'est libre. Linus Torvalds
	00

08/10/2011, 11h06	#6
Aspic Expert Confirmé Florent Diedler Étudiant Inscription : août 2005 Messages : 3 720 Détails du profil Informations personnelles : Nom : Florent Diedler Localisation : France, Hauts de Seine (Île de France) Informations professionnelles : Activité : Étudiant Informations forums : Inscription : août 2005 Messages : 3 720 Points : 3 392 Points : 3 392	Merci à vous,. N'hésitez pas à me faire d'autres remarques En tout cas, bien que le code soit mauvais, il ne semble pas vulnérable __________________ Qui ne tente rien n'a rien ! Ce qui ne nous tue pas nous rends plus fort !! Mon projet ZELDA en C++/Allegro http://www.tutoworld.com - Le Forum - Mes ressources Dotnet (cours, sources, tutos) -------------------------------------------- + + = Ne pas oublier le Tag !
	00

08/10/2011, 11h51	#7
kdmbella Membre Expert Demazy Mbella Développeur Web Inscription : août 2010 Messages : 620 Détails du profil Informations personnelles : Nom : Demazy Mbella Localisation : Cameroun Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Produits et services télécom et Internet Informations forums : Inscription : août 2010 Messages : 620 Points : 1 470 Points : 1 470	voici un article datant de 2007 qui te sera certainement utile __________________ Trois personnes peuvent garder un secret si deux d'entre elles sont mortes. :Benjamin Franklin L'humanité se divise en trois catégories : ceux qui ne peuvent pas bouger, ceux qui peuvent bouger, et ceux qui bougent : Benjamin Franklin Le hasard, c'est le déguisement que prend Dieu pour voyager incognito: Albert Einstein bon je m'arrête là au risque de me faire buter
	00

Outils de la discussion
Afficher une version imprimable Envoyer un lien vers cette page par email