[MySQL][PHP]Bezpieczeństwo skryptu Opcje

[malin23]

poniżej jest mój przykładowy skrypt.
skrypt dostępny po zalogowaniu, hasła przechowuje w bazie danych

Pytanie mam na ile to jest bepieczne, albo co powinienem zrobić zmienić żeby uniknąć "nieprzyjemności"

[PHP] pobierz, plaintext 
[/php]
 
<?php require("check.php");
 
$uzytkownik = $_SESSION["uzytkownik"];
echo $uzytkownik;
// nawiazujemy polaczenie
$connection = @mysql_connect('', '', '')
// w przypadku niepowodznie wyświetlamy komunikat
or die('Brak połączenia z serwerem MySQL.<br />Błąd: '.mysql_error());
// połączenie nawiązane ;-)
//echo "Udało się połączyć z serwerem!<br />";
// nawiązujemy połączenie z bazą danych
$db = @mysql_select_db('', $connection)
// w przypadku niepowodzenia wyświetlamy komunikat
or die('Nie mogę połączyć się z bazą danych<br />Błąd: '.mysql_error());
// połączenie nawiązane ;-)
//echo "Udało się połączyć z bazą dancych!";
/* zapytanie do konkretnej tabeli */
$wynik = mysql_query("SELECT * FROM users WHERE user_login = '$uzytkownik'")
or die('Błąd zapytania');
 
$r = mysql_fetch_assoc($wynik); 
//echo $r['user_upr'];
 
$_SESSION["uprawnienia"] = $r['user_upr'];
 
mysql_close($connection);
?>
 
 
<?php require("check.php");
 
 
// nawiazujemy polaczenie
$connection = @mysql_connect('', '', '')
// w przypadku niepowodznie wyświetlamy komunikat
or die('Brak połączenia z serwerem MySQL.<br />Błąd: '.mysql_error());
// połączenie nawiązane ;-)
//echo "Udało się połączyć z serwerem!<br />";
// nawiązujemy połączenie z bazą danych
$db = @mysql_select_db('', $connection)
// w przypadku niepowodzenia wyświetlamy komunikat
or die('Nie mogę połączyć się z bazą danych<br />Błąd: '.mysql_error());
// połączenie nawiązane ;-)
//echo "Udało się połączyć z bazą dancych!";
 
 
 
/* zapytanie do konkretnej tabeli */
$wynik = mysql_query("SELECT * FROM serwis_zlecenie, klienci WHERE serwis_zlecenie.zrealizowane = 0 AND serwis_zlecenie.IDklienci = klienci.IDklienci ORDER BY `nr` ASC")
or die('Błąd zapytania');
 
/*
wyświetlamy wyniki, sprawdzamy,
czy zapytanie zwróciło wartość większą od 0
*/
 
$uprawnienia = $_SESSION["uprawnienia"];
 
if($uprawnienia==1)
{
 
//panel admina
if(mysql_num_rows($wynik) > 0) {
    /* jeżeli wynik jest pozytywny, to wyświetlamy dane */
    echo "<table cellpadding=\"2\" border=1>";
		        echo "<tr>";
        echo "<td>nr</td>";
        echo "<td>data</td>";
				echo "<td>Nazwa</td>";
        echo "<td>osoba_przyjmujaca</td>";
        echo "<td>nazwa_urzadzenia</td>";
        echo "<td>nr_fabryczny</td>";
        echo "<td>adres_klienta</td>";
        echo "<td>kontakt_klienta</td>";
        echo "<td>gwarancja</td>";
        echo "<td>opis</td>";
        echo "<td>uwagi</td>";
        echo "<td>termin_realizacji</td>";
        echo "</tr>";
 
 
    while($r = mysql_fetch_assoc($wynik)) {
        echo "<tr>";
        echo "<td>".$r['nr']."</td>";
        echo "<td>".$r['data']."</td>";
				echo "<td>".$r['Nazwa']."</td>";
        echo "<td>".$r['osoba_przyjmujaca']."</td>";
        echo "<td>".$r['nazwa_urzadzenia']."</td>";
        echo "<td>".$r['nr_fabryczny']."</td>";
        echo "<td>".$r['adres_klienta']."</td>";
        echo "<td>".$r['kontakt_klienta']."</td>";
        echo "<td>".$r['gwarancja']."</td>";
        echo "<td>".$r['opis']."</td>";
        echo "<td>".$r['uwagi']."</td>";
        echo "<td>".$r['termin_realizacji']."</td>";
        echo "<td>
       <a href=\"serwis_edit.php?a=edit&amp;nr={$r['nr']}\">EDIT</a>
       </td>";
        echo "</tr>";
    }
    echo "</table>";
}
}
 
if($uprawnienia==0)
{
//panel handlowca
if(mysql_num_rows($wynik) > 0) {
    /* jeżeli wynik jest pozytywny, to wyświetlamy dane */
    echo "<table cellpadding=\"2\" border=1>";
    while($r = mysql_fetch_assoc($wynik)) {
        echo "<tr>";
        echo "<td>".$r['nr']."</td>";
        echo "<td>".$r['data']."</td>";
        echo "<td>".$r['osoba_przyjmujaca']."</td>";
        echo "<td>".$r['nazwa_urzadzenia']."</td>";
        echo "<td>".$r['nr_fabryczny']."</td>";
        echo "<td>".$r['nazwa_klienta']."</td>";
        echo "<td>".$r['adres_klienta']."</td>";
        echo "<td>".$r['kontakt_klienta']."</td>";
        echo "<td>".$r['gwarancja']."</td>";
        echo "<td>".$r['opis']."</td>";
        echo "<td>".$r['uwagi']."</td>";
        echo "<td>".$r['termin_realizacji']."</td>";
        echo "</tr>";
    }
    echo "</table>";
}
}
mysql_close($connection);
?>
 
[php]
[PHP] pobierz, plaintext