Ik heb een fout in mijn inlogsysteem
Als ik met opzet verkeerd inlog komt er niet de gewenste foutmelding

Ziet iemand de fout?


<html>

<head>

<!--hier include hij het stijl bestandje zodat de website de gewenste stijl krijgt-->
<link rel="stylesheet" type="text/css" href="stijl.css" />

</head>

<body>

<ul class="navbar">

<script src="http://www.clocklink.com/embed.js"></script>
<script type="text/javascript" language="JavaScript">
obj=new Object;
obj.clockfile="5005-purple.swf";
obj.TimeZone="GMT0100";
obj.width=120;
obj.height=40;
obj.wmode="transparent";
showClock(obj);
</script> 
<br>
<br>

  <li><a href="index.php">Home</a>
  <li><a href="ophalen.php">De Database</a>
  <li><a href="videos.php">Trailers</a>
  <li><a href="login.php">Admin</a>
</ul>


<?php


if(isset($_POST['submit'])){

/*voeg het bestand verbinding.php toe voor contact met de database*/
		include("verbinding.php");
	        include("functions.inc.php");
	
	//Opstarten van een sessie
	
        //Ophalen en controleren van de input uit de formuliervelden.
	$username = cleanQuery($_POST['username']);
	$password = cleanQuery($_POST['password']);
	//Wachtwoord versleutelen
	$password = md5($password);
		
	$query = "SELECT * FROM users WHERE username='$username' and password='$password'";
	
	$result = mysql_query($query);
	
	if (mysql_num_rows($result) < 1) {
	$error = "Logingegevens kloppen niet";
	    include("login.php");
	
	} else {
	    $_SESSION['username'] = "$username";
	    include "index2.php";
	}
}
else {



?>

<form name="login" method="post" action="login.php" class="box1">
<fieldset><legend>Inloggen voor Admins:</legend>
<p><label for="username">Gebruikersnaam:</label><input type='text' name="username" class="txt"/></p>
<p><label for="password">Wachtwoord:</label><input type='password' name="password"  class="txt"/></p>
<p><input type='submit' name="submit" value='Login' class="btn"/></p>
</fieldset>
</form>


<?php



}

?>

</body>

</html>


de foutmelding is:
Fatal error: Cannot redeclare cleanquery() (previously declared in /home/www/filmdatabase.awardspace.info/functions.inc.php:7) in /home/www/filmdatabase.awardspace.info/functions.inc.php on line 7

mijn fuctions.inc.php =

<?php 



//Functie om SQL-Injection te voorkomen

function cleanQuery($string)
{
  if(get_magic_quotes_gpc())  // voorkomt dubbele backslashes
  {
    $string = stripslashes($string);
  }
  if (phpversion() >= '4.3.0')
  {
    $string = mysql_real_escape_string($string);
  }
  else
  {
    $string = mysql_escape_string($string);
  }
  return $string;
}

?>


Iemand een idee?
En zo?


<?php 

//Dit bestand komt uit het informaticaboek

//Functie om SQL-Injection te voorkomen

function clnQuery($string)
{
  if(get_magic_quotes_gpc())  // voorkomt dubbele backslashes
  {
    $string = stripslashes($string);
  }
  if (phpversion() >= '4.3.0')
  {
    $string = mysql_real_escape_string($string);
  }
  else
  {
    $string = mysql_escape_string($string);
  }
  return $string;
}

?> 

<html>

<head>

<!--hier include hij het stijl bestandje zodat de website de gewenste stijl krijgt-->
<link rel="stylesheet" type="text/css" href="stijl.css" />

</head>

<body>

<ul class="navbar">

<script src="http://www.clocklink.com/embed.js"></script>
<script type="text/javascript" language="JavaScript">
obj=new Object;
obj.clockfile="5005-purple.swf";
obj.TimeZone="GMT0100";
obj.width=120;
obj.height=40;
obj.wmode="transparent";
showClock(obj);
</script> 
<br>
<br>

  <li><a href="index.php">Home</a>
  <li><a href="ophalen.php">De Database</a>
  <li><a href="videos.php">Trailers</a>
  <li><a href="login.php">Admin</a>
</ul>


<?php


if(isset($_POST['submit'])){

/*voeg het bestand verbinding.php toe voor contact met de database*/
        include("verbinding.php");
    
    //Opstarten van een sessie
    
        //Ophalen en controleren van de input uit de formuliervelden.
    $username = clnQuery($_POST['username']);
    $password = clnQuery($_POST['password']);
    //Wachtwoord versleutelen
    $password = md5($password);
        
    $query = "SELECT * FROM users WHERE username='$username' and password='$password'";
    
    $result = mysql_query($query);
    
    if (mysql_num_rows($result) == 1) {
      $_SESSION['username'] = "$username";
      include "index2.php";
    } else {
      $error = "Logingegevens kloppen niet";
      include("login.php");
    }
}
else {



?>

<form name="login" method="post" action="login.php" class="box1">
<fieldset><legend>Inloggen voor Admins:</legend>
<p><label for="username">Gebruikersnaam:</label><input type='text' name="username" class="txt"/></p>
<p><label for="password">Wachtwoord:</label><input type='password' name="password"  class="txt"/></p>
<p><input type='submit' name="submit" value='Login' class="btn"/></p>
</fieldset>
</form>


<?php



}

?>

</body>

</html>
atal error: Cannot redeclare clnquery() (previously declared in /home/www/filmdatabase.awardspace.info/login.php:7) in /home/www/filmdatabase.awardspace.info/login.php on line 7
Dan weet ik het ook niet meer. Zit de pagina zelf niet in een loop?
Weet ik niet!?

Maar zie
functions.inc.php kunnen we gewoon weglaten
dat heeft geen gevolgen voor het inloggen
er zit gewoon ook nog een fout in het login script
Hahaha, ik zie de fout.

Regel 84
include("login.php");
Moet zijn
header("login.php");

Als het nu werkt kan je de rest allemaal weer terug zetten.
Warning: Cannot modify header information - headers already sent by (output started at /home/www/filmdatabase.awardspace.info/login.php:39) in /home/www/filmdatabase.awardspace.info/login.php on line 84
Gevonden!!! (althans dat denk ik)

<?php
if (mysql_num_rows($result) == 1) {
$_SESSION['username'] = "$username";
include "index2.php";
} else {
$error = "Logingegevens kloppen niet";
include("login.php");
}
?>

Je kunt login.php niet includein in login.php!!! dat gaat natuurlijk niet goed!!!
tuurlijk wel

Wat er moet gebeuren is dat er komt te staan dat de logingegevens niet kloppen en dat dan het login scherm opnieuw komt

als het goed is kan dat gewoon
je haalt de header en include functie hier door elkaar denk ik!!!
denk het ook ..
hoe los ik het op dan?

Reageren