wachtwoord check

1. Hoe is je inlogsysteem gemaakt, dus maakt hij een sessie aan of een cookie e.d.?
2. Laat eens het inloggedeelte van je script zien.

Ik neem trouwens aan als je een "inlogsysteem" kan maken dat je dat dan ook wel kan? Maar maak eerst je vraag even compleet!

dit is een stukje van m'n inlogscript
hij maakt sessie aan

<?php
ob_start();

$gb=$_POST['gb'];
$ww=$_POST['ww'];
$host="localhost";
$username="root";
$password="";
$db_name="campagne";

mysql_connect("$host", "$username", "$password")or die("kan niet connecten met de database");
mysql_select_db("$db_name")or die("Kan de database niet selecteren");

$sql="select * FROM toevoegen WHERE Gebruikersnaam='$gb' AND Password='$ww'";
$sel=mysql_query($sql);

$aantal=mysql_num_rows($sel);

if($aantal == "1" )

{
session_start();
$_SESSION['gebruiker']=$gb;

echo "Je bent nu ingelogd <br>";
if ($_SESSION['Type'] == 'Admin') {
session_start();
else
{
header( 'refresh:2 url=./inloggen.php' );
}

if

($_SESSION['gebruiker'] != "" && isset($_SESSION['gebruiker']))
{session_start();


}
else
{
echo "Probeer opnieuw";
"header( 'refresh:3 url=./inloggen.php' )";
}
echo "<br />";
while ($volgende=mysql_fetch_array($sel)){
$Type = $volgende[0];
$ID = $volgende[1];
$gebruikersnaam = $volgende[2];
$Password = $volgende[3];
}
{
mysql_query($sql)or die (msql_error());


if(!empty($Type)){
session_start();
$gb= addslashes($_POST['gb']);
$ww= addslashes($_POST['ww']);
$_SESSION['gebruiker']=$gebruikersnaam;
$_SESSION['Password']=$Password;
$_SESSION['Type']=$Type;
$_SESSION['ID']=$ID;
}



}

ob_end_flush();
?>

alvast bedankt

Je moet een inlogsysteem met sessies gebruiken en zet dan boven elke pagina:

<?php
session_start();
if(!isset($_SESSION['login']))
{
  // niet ingelogd
}
else
{
  // wel ingelogd
}

1. Gebruik geen or die maar een nette foutafhandeling.
2. 4x session_start? 1 op lijn 22, lijn 36, lijn 57 en lijn 27? session_start() hoort maar 1x in je pagina te staan en wel helemaal bovenaan.
3. Gebruik geen mysql_fetch_array maar mysql_fetch_assoc.
4. Laat nooit mysql_error aan de gebruikers zien! (Mag wel als je zelf debugged).
5. Wat probeer je hiermee?

echo "Probeer opnieuw";
    "header( 'refresh:3 url=./inloggen.php' )";

header is toch echt een functie, dus het zou zo moeten:

header( 'refresh:3 url=./inloggen.php' );
echo "Probeer opnieuw";

De echo moet eronder omdat je voor een header functie geen html output mag hebbben.

6. Op lijn 35 heb je staan: "if ($_SESSION['gebruiker'] //Nogwat)" daar moet je of empty() of isset() voor gebruiken anders krijg je een notice error wanneer je dit probeerd en de sessie bestaat niet. Dit heb je overigens vaker in je script staan.
7. DENK AAN SQL INJECTIONS!! Je script is enorm onveilig, kijk in ieder geval eens naar: http://php.net/manual/en/function.mysql-real-escape-string.php (of gebruik mysqli of PDO).
8. Probeer je script wat duidelijker te maken, beetje commentaar e.d.

Pas dat eerst eens even aan!

9. niet onnodig vars kopieeren

Oja en:

10. Gebruik geen ob_start() e.d. die zijn er (volgens mij) niet voor bedoeld om header fouten op te lossen, je moet gewoon zorgen dat je geen output voor session_start(), setCookie() en header() hebt staan.

iedereen bedankt voor de reactie

ik ga het veranderen

thnx

is het gelukt?

ik doe het zo:
<formulier>

<?php
@session_start();
$postnaam= $_POST['naam'] ;
$postwachtwoord= $_POST['wachtwoord'] ;

if($_SERVER['REQUEST_METHOD'] == 'POST')
{
   include"database.php";

if($postnaam=="$naam")
{

if($postwachtwoord=="$wachtwoord")
{

if($ban =="ja")
{echo'<font color="red"> <b> JIJ BENT VERBANNEN <br> 
omdat: '; echo"$banrede"; echo'</b></font>';
} else{
?>
<?php
if($VIP =="ja"){
include"inloggovip.php";}else{ include"inloggo.php";}
?>

<?php

if($rang =="0"){echo"jij bent een lid!";}elseif($rang =="1"){echo"jij bent VIP!!!!!";}elseif($rang=="2"){echo"jij bent beheerder";}
}
}else{
echo'<font color="red"><b>wachtwoord is niet goed!</b></font>';}}else{echo'<font color="red"><b>naam bestaat niet!</b></font>';}}else{?>
<form method="post">
je gebruiksnaam: <input type="text" name="naam"

<?php
if($ledensysteem=="uit"){?> disabled="disabled" <?php }else{}; ?>

 ><br>
wachtwoord: <input type="password" name="wachtwoord" <?php
if($ledensysteem=="uit"){?> disabled="disabled" <?php }else{}; ?>

 ><br>
<input type="submit" name="mysubmit" value="klikkie" <?php
if($ledensysteem=="uit"){?> disabled="disabled" <?php }else{}; ?>

 >
</form> 
<?php
};
?>

@Maarten: Geen net script.