Wątek: [SQL Injection] account.php

1. Opis błędu

Brak escape’owania przy $_POST[’ce’] i $_POST[’ne’].

2. Zagrożenie

Błąd umożliwiający atak SQL Injection.

3. Rozwiązanie

[OTWÓRZ]
account.php
[ZNAJDŹ]

Kod php:

        $_POST['ne'] = strip_tags($_POST['ne']);
        $_POST['ce'] = strip_tags($_POST['ce']); 


[ZAMIEŃ NA]

Kod php:

        $_POST['ne'] = mysql_escape_string(strip_tags($_POST['ne']));
        $_POST['ce'] = mysql_escape_string(strip_tags($_POST['ce'])); 


a dlaczego pg_ a nie mysql_ ??

o dzięki sazian, że zuważyłeś. pg jest do PostgreSQL'a.

Nawet bez escape’owania nie udało mi się użyć sql injection, chociaż pewnie zrobiłem to nieumiejętnie

powstawiaj sobie do emaila bez escapowania pazurki tzn. ' '

Te wszystkie bugi na kodzie 1.3 naprawiasz ?

Tak. Jak naprawię większość postaram się wydać czysty silnik vallheru, ale bez bugów.

A moglbys mi powiedziec czy tutaj mam cos wyescapowac ?

Kod php:

if (isset($_GET['step']) && $_GET['step'] == "ce") 
    {
        if (empty($_POST["ne"]) || empty($_POST['ce'])) 
        {
            error (EMPTY_FIELDS);
        }
        require_once('includes/verifymail.php');
        if (MailVal($_POST['ne'], 2)) 
        {
            error(BAD_EMAIL);
        }       
        $query = $db -> Execute("SELECT id FROM players WHERE email='".$_POST['ne']."'");
        $dupe = $query -> RecordCount();
        $query -> Close();      
        if ($dupe > 0) 
        {
            error (EMAIL_BLOCK);
        }
        $db -> Execute("UPDATE players SET email = '".$_POST['ne']."' WHERE email = '".$_POST['ce']."' AND id = '".$player -> id."'") or error (BAD_EMAIL);
        error (YOU_CHANGE." ".$_POST['ce']." ".ON." ".$_POST['ne'].". ".CLOSE_WINDOW);
    } 


Bo tutaj nie ma tego co ty kazales znalezc.

te wszystkie poradniki sa do 1.3 vall, a tam oryginalnie jest tak:

Kod php:

    /**
     * Change email
     */
    if (isset($_GET['step']) && $_GET['step'] == "ce") 
    {
        if (empty($_POST["ne"]) || empty($_POST['ce'])) 
        {
            error(EMPTY_FIELDS);
        }
        $_POST['ne'] = strip_tags($_POST['ne']);
        $_POST['ce'] = strip_tags($_POST['ce']);
        require_once('includes/verifymail.php');
        if (MailVal($_POST['ne'], 2)) 
        {
            error(BAD_EMAIL);
        }       
        $query = $db -> Execute("SELECT `id` FROM `players` WHERE `email`='".$_POST['ne']."'");    
        if ($query -> fields['id']) 
        {
            error(EMAIL_BLOCK);
        }
        $query -> Close();
        $intNumber = substr(md5(uniqid(rand(), true)), 3, 9);
        $strLink = $gameadress."/index.php?step=newemail&code=".$intNumber."&email=".$_POST['ne'];
        $adress = $_POST['ne'];
        $message = MESSAGE_PART1.$gamename."\n".MESSAGE_PART2."\n".$strLink."\n".MESSAGE_PART3." ".$gamename."\n".$adminname;
        $subject = MESSAGE_SUBJECT.$gamename;
        require_once('mailer/mailerconfig.php');
        if (!$mail -> Send()) 
        {
            $smarty -> assign ("Error", MESSAGE_NOT_SEND." ".$mail -> ErrorInfo);
            $smarty -> display ('error.tpl');
            exit;
        }
        $db -> Execute("INSERT INTO `lost_pass` (`number`, `email`, `id`, `newemail`) VALUES('".$intNumber."', '".$_POST['ce']."', ".$player -> id.", '".$_POST['ne']."')") or $db -> ErrorMsg();
        error(YOU_CHANGE);
    }