<?php

//Dans la variable $ID se trouve l'ID du joueur



/* ******  Annulation d'une attaque: ****** */

if(isset($_GET['annuler'])) //On vérifie s'il y a eu une demande d'annulation

{

$result = mysql_query("SELECT * from attaque WHERE IDAttaquant='$ID' AND ID='".$_GET['annuler']."'");//Si oui, on sélectionne l'attaque que l'ont doit annuler dans la base de données.

$donnee = mysql_fetch_array($result);

if($donnee['timestampDepart']>(time()-2*60))//Timestamp départ contient le timestamp au moment du lancement, on vérifie donc que cela fait moins de deux minutes que le lancement de l'attaque a eu lieu.

 {

    $Armee=getSoldats($donnee);//On extrait des donnees les informations décrivant l'armée. Pour vous simplifier la vie, on considére cette fonction sans faille.

    addArmee($ID,$Armee,'2');//On remet l'armée en question au joueurs qui vient d'annuler l'attaque. Pour vous simplifier la vie, on considére cette fonction sans faille.

    mysql_query("DELETE from attaque WHERE IDAttaquant='$ID' AND ID='".$_GET['annuler']."'");//On supprime l'attaque de la base de données.

 }

}



/* ******  Affichage des attaques: ****** */

$resultat = mysql_query("SELECT * from attaque WHERE IDAttaquant='$ID'");



while ($donnees = mysql_fetch_array($resultat))

{

    $tempsRestant = $donnees['timestamp'] - time();



    echo '<p class="gras">- Vous allez attaquer '.getPseudo($donnees['IDDefenseur']).' dans '.convertiTemps($tempsrestant);//Message de l'attaque.

    if($donnees['timestampDepart']>(time()-2*60)) //Si le joueur peut encore annuler l'attaque, alors on ajoute le lien pour pouvoir annuler.

    {

        echo ' - <a href="Armee.php?annuler='.$donnees['ID'].'">Annuler</a>';

    }

    echo ' </p>';



} 

LexLxUs a écrit :Une accolade en trop à la fin ? iffle:

NicoMSEvent a écrit :tu as eu une injection de SQL dans la variable : $_GET['annuler'] ?

Pouratant, une des premiere chose qu'on apprend en sécurité, est de ne jamais faire confiance aux données entrées... faut tester la validité, faire des "escape" et tout ce qu'il faut pour éviter ça...

Si ce n'est pas ça, je te suggère quand meme de vérifier ce point 16

Kéké a écrit :Règle numéro 1 en PHP : ne jamais utiliser GET.

Est-ce là toute l'erreur de ton script ? ou bien il y a quelque chose d'encore plus profond à trouver ?

Kéké.

uriak a écrit :pas certain, mais étant donné la configuration des fonctions et leur point d'entrées, je soupçonne de l'injection SQL dans la variable $_GET['annuler']

par exemple avec un "0 OR IDAttaquant = (Id d'un autre joueur)"

on pourrait alors provoquer l'annulation d'une attaque récente de la part d'un autre joueur dont on aurait récupéré l'ID.
Bien entendu, simplement vérifier que $_GET['annuler'] est un nombre rend caduque cette attaque.

par contre comme je ne vois pas trop le lien avec la seconde partie du script je ne suis pas sûr de mon coup..

orditeck a écrit :Déjà, comme dis Lex, y'a une accolade de trop à la fin ce qui résulte à une erreur PHP.
Je vois pas comment tes joueurs peuvent afficher cette page 1

Sinon, tu fais un :
DELETE from attaque WHERE IDAttaquant='$ID' AND ID='".$_GET['annuler']."'

Ensuite, un joueur fait un petit :
Armee.php?annuler=1' DROP DATABASE 'jeu

Et le tour est joué... enfin, ça fonctionne seulement si magic_quotes_gpc est désactivé. Mais pour des raisons pratiques, je crois que c'est mieux qu'il soit activé.

S'il n'est pas activé, voici un petit bout de code utile qui arrange tout ça, bien que j'imagine que ta faille est corrigée depuis longtemps... !

Code PHP :
$get_arr = array();
foreach ($_GET as $key => $get_arr) {
$_GET[$key] = addslashes($get_arr);
}


J'ai trouvé, au moins ?

NicoMSEvent a écrit :autre point que je vois, c'est : Armee.php?annuler='.$donnees['ID']

Rien n'empeche de sauver le lien, et le lancer plus tard. A moins que tu ne vérifies si la durée de validité du lien est ok lorsque qu'il est activé (avec un message du genre : 'trop tard!' si il n'est plus valide).

LexLxUs a écrit :Bonjour 2

Le problème ne serait pas que le fait de passer l'id de l'attaque à annuler en GET permet au joueur d'annuler une attaque qui n'est pas la sienne ?

L.

keke a écrit :Donc si je comprends bien ton message, il faut qu'on creuse encore un peu ?

Je continue.

kéké.

NicoMSEvent a écrit :si tu de délogue, le $ID n'est plus initialisé... il ne suffirait pas de rajouter a n'importe quelle requete un parametre en GET &ID=xx pour usurper l'identité de n'importe quel joueur?