Discussion :

[backnight]

Bonjour à toutes et à tous,

Je viens de me poser une question sur l'un de mes scripts qui est l'une des plus basique^^: est ce qu'il est sécurisé ?

Voici son contenu:

Code :

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

1
2
3
4
5
6
7
8
9
 
<?php
if (!isset($_GET['id']) && !is_numeric($_GET['id'])) {
exit;
}
 
actions en tout genre notament une insertion mysql du type:
mysql_query("Update membre set id='".$_GET['id']."' ...");
?>

En faite, je voulais savoir si la simple vérification ISSET et IS_NUMERIC suffisait à empêcher les attaques notament injection mysql etc et si ce 'est pas le cas merci de me fournir un exemple

J'ai toutefois remarqué qu'un id du type 6.5646 passait la sécurité mais je n'ai rien trouvé d'autre de plus affolant.

PS: Je sais qu'il y a de meilleures protections complémentaires mais si je ne les ai pas mises c'est tout simplement pour tester l'efficacité du IS_NUMERIC.

Merci et bonne continuation !

[Unkof]

Tu peux faire simplement :

Code :

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

1
2
3
4
5
6
 
 if(isset($_GET['id']))
{
 $id=mysql_real_escape_string($_GET['id']);
 mysql_query("Update membre set id='$id' ...");
}

Dans ce cas, peut pas y avoir d'injection, et au pire, si l'id a été trafiqué et qu'il n'est pas numérique, il ne fera pas l'update parce qu'il ne le trouvera pas.

N'utilise surtout pas is_numeric pour ca.

[Doksuri]

salut,
pour ma part, j'utilise aussi la fonction intval();
par exemple :

Code :

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

1
2
$id = intval($_GET['id']);
echo $id;

ainsi, si un malin essaye dans l'url ta_page.php?id=toto
bah au final, si $_GET['id'] n'est pas un entier, il vaudra zero, sinon, il le recupere normalement

[backnight]

Bonsoir,

Merci de vos réponses mais elles ne correspondent pas exactement à mes attentes (Je reconnais que j'ai aussi mal formulé la question^^).

Je cherche plus particulièrement à trouver des failles pour la fonction IS_NUMERIC et non à sécuriser mes scripts.

Puisque si des fonctions complémentaires comme mysql_real_escape_string() ont été ajouté, c'est que les fonctions du type IS_NUMERIC n'assurent pas la totalité de la protection et c'est ce problème que je souhaîte soulever:

- Les limites en sécurité pour cette fonction.

Merci

PS: Si vous trouvez une faille merci de donner un exemple ou un type de valeur qui répondrait TRUE alors qu'elle n'est pas numérique.

[Unkof]

C'est simple, y en a pas. Si ta variable est is_numeric, c'est que c'est une valeur numérique.

D'un autre côté, elle est faite pour ca la fonction ... ^^

Manuel php:

bool is_numeric ( mixed var )
Détermine si la variable donnée est de type numérique. Les chaînes numériques sont composées optionnellement de signes, de n'importe quel nombre de chiffres, optionnellement d'une partie décimale ainsi qu'une partie exponentielle. +0123.45e6 est une valeur numérique valide. La notation hexadécimale ( 0xFF ) est autorisée également, mais uniquement sans un signe, sans décimale et sans partie exponentielle.

Difficile d'être plus clair.

6.5646 est aussi numérique ... hein ... ^^ .... ou tu confonds peut être avec les entiers, dans ce cas il te faut utiliser intval, pas is_numeric.

[backnight]

Oki merci.

Donc d'après tes dires la fonction is_numeric n'a pas de points faibles niveau sécurité informatique.

Merci.