mysql_real_escape_string ? funktz das hier ?

In 10 Minuten startet der nächtliche Backupvorgang! Es kann währenddessen (ca. 10 Minuten) zu Einschränkungen bei der Nutzung des Forums kommen
Weitere Infos findet ihr im Thema Backup des Forums

    hier hab ich ein script gemacht zu php sprich ein login system für mysql_Real_escape_string damit keine injection möglich ist mehr.



    <?php
    session_start(login);
    //---------------------------->
    $user = $_POST['username'];
    $pwd = $_POST['password'];
    //------------------------------>


    $user = mysql_real_escape_string($user);
    $pwd = mysql_real_escape_string($pwd);


    //-----{main}-------------->


    $con = mysql_connect('localhost','username','password');
    if($con = true){
    die('connection failed!');
    session_destroy(login);
    }
    mysql_select_db('databank');
    //-------------------------------------->
    //checken


    if(mysql_query('name','passwort') == $user && $pwd){
    echo('willkommen zurück $user');
    header('abgesicheterbereich.php');
    }
    if(!empty($pwd)){
    echo 'bitte fülle alle felder aus!';
    }
    if(!empty($user)){
    echo'bitte fülle alle felder aus!';
    }
    ?>


    wenn da fehler enthalten sind , könnte die mir wer verbessern ? , a
    außerdem ist das script richtig ?


    mfg kyle :thumbup:

    Du kannst kein mysql_real_escape_string benutzen bevor du eine Verbindung zu einem MySQL Server hast.

    PHP
    //-----{main}-------------->
$con = mysql_connect('localhost','username','password');


$user = mysql_real_escape_string($user);
$pwd = mysql_real_escape_string($pwd);


    MFG Piet

    Zitat von [PA]Piet_Meier

    Du kannst kein mysql_real_escape_string benutzen bevor du eine Verbindung zu einem MySQL Server hast.

    PHP
    //-----{main}-------------->
$con = mysql_connect('localhost','username','password');


$user = mysql_real_escape_string($user);
$pwd = mysql_real_escape_string($pwd);


    MFG Piet


    danke aber ist meine abfrage richtig überhaupt ?
    sprich das script allgemein

    Du hast außerdem geschrieben:

    PHP
    if($con = true){
die('connection failed!');
session_destroy(login);
}


    Damit überprüfst du nicht, sondern du setzt $con auf true :D
    So müsste es wenn heißen:


    PHP
    if($con == true){
die('connection failed!');
session_destroy(login);
}

    Hier hab ich mal überarbeitet(nicht getestet):


    /e
    Fehler behoben bei einer Nachricht.


    MFG Piet