bevestiging voor een update quey

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Bart nukem

bart nukem

10/05/2011 14:34:30
Quote Anchor link
Wie kan me op weg helpen, in onderstaande code wil ik graag eerst een bevestiging van de gebruiker voordat de update query wordt doorgevoerd.

Code (php)
PHP script in nieuw venster Selecteer het PHP script
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
<?php
session_start();
if(!empty($_POST)){
include("contactdb.php");
$query = "SELECT * FROM aanmelding WHERE email='" .mysql_real_escape_string($_POST["gebruikersnaam"])  ."' AND lidnr='". mysql_real_escape_string($_POST["wachtwoord"]) . "'";  

$result = mysql_query($query);

if(mysql_num_rows($result) > 0){
$gebruiker = $_POST["gebruikersnaam"];
$wachtwoord = $_POST["wachtwoord"];

//goede login , bevestiging vragen voordat update wordt uitgevoerd
            
    // query samenstellen

    $query="UPDATE aanmelding SET status = 'A' WHERE email='$gebruiker' AND lidnr='$wachtwoord'";
    $result = mysql_query($query) or die ("FOUT1: " . mySql_error());
    echo("De volgende opdracht is uitgevoerd: <b>$query</b><br>\n");

 
exit();
}

else { //foute login
$text = "Je hebt geen goede combinatie van gebruikersnaam en wachtwoord gebruikt!";
die($text);
} }

else{
header("Location: inloggen.php");
}

?>
 
PHP hulp

PHP hulp

21/11/2024 21:41:49
 
- SanThe -

- SanThe -

10/05/2011 15:18:15
Quote Anchor link
Denk aan SQL-Injection. Script is lek.
 
Bart nukem

bart nukem

10/05/2011 15:19:51
Quote Anchor link
Dank je SanThe, wil je mij vertellen waar het lek zit?
 
Joren de Wit

Joren de Wit

10/05/2011 15:24:10
Quote Anchor link
Op regel 16 gebruik je $gebruiker en $wachtwoord ongecontroleerd en onbeveiligd in je query...
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.