[php][mysql]Prosty skrypt logowania wysypuje Apache'a Opcje

[petra999]

Witajcie!
Zacząłem się bawić z php i doszło do skryptu logowania.
Napisałem sobie prosty skrypt i odpaliłem go na lokalnym serwerze stworzonym za pomocą Wampserver'a.
I za każdym razem jak testuje ten skrypt to wysypuje mi się Apache. Czy w moim skrypcie jest błąd? I przy okazji czy mój skrypt jest w miarę bezpieczny?
Pozdrawiam i z góry dziękuje za pomoc!

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['login']) || empty($_POST['password']))
{
	echo 'bład. nie podałeś loginu/hasła<br />';
	echo "<form action=\"index.php\" method=\"post\">
			<input type=\"text\" name=\"login\" /><br />
			<input type=\"password\" name=\"password\" /><br />
			<input type=\"submit\" value=\"Loguj\" /><br /><br />
		</form>";
}
else
{
$login  = $_POST['login'];
$password = sha1($_POST['password']);
 
 
require_once('connect.php');
 
session_start();
if(isset($_SESSION['access']))
{
	echo 'zalogowany';
}
else
{
		$query = "SELECT username, access FROM user WHERE username='$login' AND pass='$password'";
		$result = mysql_query($query);
        $row = mysql_fetch_array ($result, MYSQL_ASSOC);
		if($row)
		{
			$_SESSION['access'] = $row['access'];
		}
		else
		{
			echo 'błąd. podałeś błędne dane';
			echo '<a href="index.php"><br />Spróbój jeszcze raz.</a>';
			exit;
		}
}
mysql_close();
}
?>
[PHP] pobierz, plaintext 

[Kszyhuu]

I przy okazji czy mój skrypt jest w miarę bezpieczny?

Lepiej trzymać hasła w MD5. (IMG:style_emoticons/default/winksmiley.jpg)
I wygodniej zrobić to na osobnym pliku np. logowanie.php

To dodajesz w dowolnym pliku:

[HTML] pobierz, plaintext 
<form method="post" action="logowanie.php">
        Login:</b> <input type="text" name="login" size="10" class="input"><br>    
        Pass:</b> <input type="password" name="pass" size="10" class="input"><br>
        <input class="input" type="submit" name="submit" value="Zaloguj">
        </form>
[HTML] pobierz, plaintext 

A to plik logowanie.php:

[PHP] pobierz, plaintext 
<?php
session_start(); 
ob_start();  
 
mysql_connect('HOST','BAZA','PASS');  
mysql_select_db('BAZA');
 
 
$login = mysql_escape_string($_POST['login']);  
$pass = mysql_escape_string(md5($_POST['pass'])); 
 
$zapytanie = mysql_query("SELECT * FROM user WHERE login = '$login'");  
$dane = mysql_fetch_array($zapytanie);
 
if(isset($_POST['login']) && isset($_POST['pass'])) {
if($login ==''.$dane['login'].'' && $pass ==''.$dane['pass'].'')
{
 
    $_SESSION['admin']='ok';  
    $_SESSION['login']=''.$dane['login'].'';
 
/*TUTAJ MOŻESZ WALNĄĆ SESJĘ, CIACHO I PRZEKIEROWANIE
 
setcookie("log" , "log", time()+3600, "/","", 0);  
 header("Location: index.php"); 
ALBO
echo 'Zalogowano, kliknij <A HREF="XXX">tutaj</A>
*/
}
 
else {
 echo 'Nie zalogowano.'; }  
}
 
?>
[PHP] pobierz, plaintext 

I później w plikach dodajesz tylko:

[PHP] pobierz, plaintext 
include("logowanie.php");
if($_SESSION['admin'] == 'ok' && isset($_COOKIE['log'])) {
KOD DLA ZALOGOWANYCH
}
else {
echo 'Nie masz prawa tu przebywać.';
}
[PHP] pobierz, plaintext 

Ten post edytował Kszyhuu 26.08.2009, 16:15:00

[Fifi209]

Lepiej trzymać hasła w MD5. (IMG:style_emoticons/default/winksmiley.jpg)

Gdyż? Uzasadnij swoją wypowiedź...
Ja czytałem, że sha1 jest bezpieczniejsze...