php - sikkerhedstjek virker ikke? - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

Bruger #14545 @ 08.01.09 16:32

Hej alle

Jeg er igang med at lave et script hvor man kan opdatere sit password. Det virker også fint...sådan da. Men mit sikkerhedstjek virker ikke hvis man efterlader et tomt felt, eller det vil sige at fejl beskeden kommer op som den skal, men ændringerne sker alligevel i min database? Jeg har nu selv prøvet i meget lang tid at finde fejlen, men har nu givet op

Her er mit script:

Kode

<?
session_start();
$_SESSION['brugernavn'] = $brugernavn;
$_SESSION['password'] = $password;

?>
<html><head><LINK href="/style.css" type="text/css" rel=stylesheet>
</head>
<body>
<?php
if ($_SESSION['logget_in'] != 1)
{
?>
<table class="artikel">
<tr>
 <TD ALIGN="CENTER" VALIGN="TOP"><b>Du skal være logget ind for at få vist denne side</b></td>
</tr>
</table>
<?php
}
else
{


{
?>

<table align="center"; style="color: #FFFFFF;
width: 500px; 
height=500px; 
padding:10px; 
border:1px solid black; 
background-color:transparent; 
border-color:#FFFFFF;">
<tr> 
 <TD ALIGN="CENTER" VALIGN="TOP"><p style="color=#990000;"><b>Opdater din profil</p><hr><p>Du har her mulighed for at ændre dit password og din email</b></p> <form name="post" method="post" action=""> <input name="action" type="hidden" value="update">

<p align="left" style="margin-left:10px;">
Nuværende password: 
 <input type="password" name="password2"></p>
<p align="left" style="margin-left:65px;">
Nyt password: 
 <input type="password" name="nytpassword"></p>
<p align="left" style="margin-left:39px;">
Gentag password: 
 <input type="password" name="nytpassword2"></p>
<p align="center">
<input type="submit" name="opdater1" value="Opdater">
</p>
<hr>
</td>
</form>
</tr>
<?php
if ($_POST[action] == "update"){
	$_POST['password2'] = $password2;
	$_POST[nytpassword] = $nytpassword;
	$_POST['nytpassword2'] = $nytpassword2;
require("config.php");
mysql_connect($mysql_host, $mysql_user, $mysql_pw);
mysql_select_db($mysql_db) or die(mysql_error());
$resultat = mysql_query("SELECT * FROM `brugere` WHERE brugernavn = '$brugernavn' AND password = '$password2'");//Spørger efter ID
$number = mysql_num_rows($resultat);//Tæller antaller af resultater	


	if(!$password2) {

     echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at udfylde dit nuværende password</p>';

} elseif(!$nytpassword) {

echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at udfylde dit nye password</p>';

} elseif(!$nytpassword2) {

echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at bekræfte dit nye password</p>';
}
elseif($nytpassword != $nytpassword2) {
echo	'<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Indholdet i nyt password og gentag password skal være identisk </p>';
}

if($number != 1) {

echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Det er vidst det forkerte password du har indtastet</p>';

}
else {
mysql_select_db($mysql_db) or die(mysql_error()); 
mysql_query("UPDATE brugere SET
password = '$nytpassword'
WHERE brugernavn = '$brugernavn'")or die(mysql_error());

echo '<p style="position:absolute; left:400px; color=#FFFFFF">Diw password er nu ændret. <a href="index.php?side=forside">Tilbage til Forsiden</a></p>';

}
}

?>


</table>
<?php
}
}
?>
</body>
</html>

8 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 3 karma

Sorter efter stemmer Sorter efter dato

Bruger #3427 @ 08.01.09 18:09

1.520

nogen af de fejl jeg lige kunne se, skulle være rettet her:

Kode

<?
session_start();
$brugernavn = $_SESSION['brugernavn'];
$password = $_SESSION['password'];

?>
<html><head><LINK href="/style.css" type="text/css" rel=stylesheet>
</head>
<body>
<?php
if ($_SESSION['logget_in'] != 1)
{
?>
<table class="artikel">
<tr>
 <TD ALIGN="CENTER" VALIGN="TOP"><b>Du skal være logget ind for at få vist denne side</b></td>
</tr>
</table>
<?php
}
else
{

?>

<table align="center"; style="color: #FFFFFF;
width: 500px; 
height=500px; 
padding:10px; 
border:1px solid black; 
background-color:transparent; 
border-color:#FFFFFF;">
<tr> 
 <TD ALIGN="CENTER" VALIGN="TOP"><p style="color=#990000;"><b>Opdater din profil</p><hr><p>Du har her mulighed for at ændre dit password og din email</b></p> <form name="post" method="post" action=""> <input name="action" type="hidden" value="update">

<p align="left" style="margin-left:10px;">
Nuværende password: 
 <input type="password" name="password2"></p>
<p align="left" style="margin-left:65px;">
Nyt password: 
 <input type="password" name="nytpassword"></p>
<p align="left" style="margin-left:39px;">
Gentag password: 
 <input type="password" name="nytpassword2"></p>
<p align="center">
<input type="submit" name="opdater1" value="Opdater">
</p>
<hr>
</td>
</form>
</tr>
<?php
if ($_POST[action] == "update"){

	$password2 		= $_POST['password2'];
	$nytpassword 	= $_POST['nytpassword'];
	$nytpassword2 = $_POST['nytpassword2'];
	
	require("config.php");
	mysql_connect($mysql_host, $mysql_user, $mysql_pw);
	mysql_select_db($mysql_db) or die(mysql_error());
	$resultat = mysql_query("SELECT * FROM `brugere` WHERE brugernavn = '$brugernavn' AND password = '$password2'");//Spørger efter ID
	$number = mysql_num_rows($resultat);//Tæller antaller af resultater	

	if(($password2 == $password) && ($nytpassword == $nytpassword2)){
		$updateDataOk = true;
	}else{
		$updateDataOk = false;
	}
		
		/* Error Msg */
	if(strlen($password2) == 0) {

		echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at udfylde dit nuværende password</p>';
			 
	} 
	elseif(strlen($nytpassword)) == 0) {

		echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at udfylde dit nye password</p>';
		
	} 
	elseif(strlen($nytpassword2)) == 0) {

		echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Du mangler vidst at bekræfte dit nye password</p>';
		
	}
	elseif($nytpassword != $nytpassword2) {

		echo	'<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Indholdet i nyt password og gentag password skal være identisk </p>';
		
	}
	elseif($number != 1) {

		echo '<p style="position:absolute; left:370px; color=#FFFFFF">Oops! Det er vidst det forkerte password du har indtastet</p>';

	}

	if ($updateDataOk){
		mysql_select_db($mysql_db) or die(mysql_error()); 
		mysql_query("UPDATE brugere SET password = '$nytpassword' WHERE brugernavn = '$brugernavn'")or die(mysql_error());

		echo '<p style="position:absolute; left:400px; color=#FFFFFF">Diw password er nu ændret. <a href="index.php?side=forside">Tilbage til Forsiden</a></p>';
	}
}

?>


</table>
<?php
}
?>
</body>
</html>

Edit: rettet lidt i koden.

Indlæg senest redigeret d. 08.01.2009 18:29 af Bruger #3427

Bruger #5620 @ 08.01.09 17:24

1.500

du kan også bare prøve at skrive
elseif($number != 1) {
i stedet for
if($number != 1) {

Bruger #5789 @ 08.01.09 16:47

640

if(!$password2) {

Spørger du ikke bare om variablen eksisterer der, ikke om den er tom.

php kan have noget indbygget, men prøv at sammenligne variablen med en tom streng i stedet.

måske sådan her:

($password2 == '')

Vh

Bruger #5789 @ 08.01.09 16:55

640

som du sikkert kan læse er jeg ikke så meget inde i php, men jeg tror du har stor risiko for at folk laver lidt sql injection på det site. Især når du poster kildekoden offentligt.

Dog kan man ikke umiddelbart se hvor det afvikles, så hold da i det mindste det hemmeligt

Bruger #3427 @ 08.01.09 17:07

1.520

du kan ikke bruge if(!$password2) { da det vil betyde at indholdet af $password2 skulle være en boolsk værdi og ikke en streng

du kan bruge if(strlen($password2) == 0){

Eidt:
kunne være du skulle kigge på MD5 eller SHA1 til passwords

Indlæg senest redigeret d. 08.01.2009 17:09 af Bruger #3427

Bruger #14545 @ 08.01.09 17:38

Nu har jeg prøvet at udskifte if(!$password2) { } med if(strlen($password2)==0){ } men med samme resultat, jeg har også prøvet tidligere med if(strlen($password2)==""){ } det virker heller ikke

Bruger #14545 @ 08.01.09 20:49

Årh nu virker det! jeg gennelæser dine rettelser om lidt! men faktum er at nu virker sikkerheden som den skal

mange tak!

Bruger #2695 @ 09.01.09 11:19

1.963

Årh nu virker det! jeg gennelæser dine rettelser om lidt! men faktum er at nu virker sikkerheden som den skal mange tak!

Det er jo som man tager det. Alle variable, der bruges i SQL bør lige sendes en tur igennem mysql_real_escape_string:

Kode

$password = mysql_real_escape_string($_POST['password']);

...og magic quotes bør slåes fra (det er ikke engang understøttet i PHP 6).

sikkerhedstjek virker ikke?

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler