[MySQL][PHP] Czy ten kod jest bezpieczny? Opcje

[Michal2451]

Cześć! Zrobiłem stronę z logowaniem, rejestracją i... rangami/uprawnieniami.
Przy każdym użytkowniku jest kolumna ranga.
1 = user
5 = mod
9 = admin

I teraz mam np. lewe menu:

[PHP] pobierz, plaintext 
<?php include('config1.php');
 
if( ("$t[1]") == 1) {
echo ' <div class="box">
                <h3>Panel</h3>
                <ul>
                    <li><a href="#">Dodaj</a></li>
                    <li><a href="#">Usuń</a></li>
					<li><a href="#">Zmień</a></li>
                    <li><a href="#">Mój profil</a></li>
					<li><a href="logi.php">Logs/Newsy</a></li>
                </ul>
            </div>';
} elseif( ("$t[1]") == 9) {
echo ' <div class="box">
                <h3>Panel</h3>
                <ul>
                    <li><a href="login.php">Logowanie</a></li>
                    <li><a href="register.php">Rejestracja</a></li>
					<li><a href="logi.php">Logs/Newsy</a></li>
					<li><a href="#">Dodaj</a></li>
                    <li><a href="#">Usuń</a></li>
					<li><a href="#">Zmień</a></li>
                    <li><a href="#">Mój profil</a></li>
                </ul>
            </div>';
			echo ' <div class="box">
                <h3>Użytkownicy</h3>
                <ul>
                    <li><a href="users.php">Zarejestrowani</a></li>
                    <li><a href="znajdz.php">ZnajdĽ</a></li>
                    <li><a href="usun.php">Usuń</a></li>
					<li><a href="edytuj.php">Ranga</a></li>
                </ul>
            </div>';
	} elseif( ("$t[1]") == 5) {
echo ' <div class="box">
                <h3>Panel</h3>
                <ul>
                    <li><a href="#">Strefa moderatora</a></li>
					<li><a href="#">Dodaj</a></li>
                    <li><a href="#">Usuń</a></li>
					<li><a href="#">Zmień</a></li>
                    <li><a href="#">Mój profil</a></li>
					<li><a href="logi.php">Logs/Newsy</a></li>
                </ul>
            </div>';
		} else {
		echo ' <div class="box">
                <h3>Panel</h3>
                <ul>
                    <li><a href="login.php">Logowanie</a></li>
                    <li><a href="register.php">Rejestracja</a></li>
					<li><a href="logi.php">Logs/Newsy</a></li>
                </ul>
            </div>'; 
			};
 
 
 
 
 
?>
[PHP] pobierz, plaintext 

I plik config1.php:

[PHP] pobierz, plaintext 
<?php include('login.php');
$sql_conn = mysql_connect('xxx', 'xxx', 'xxx')
or die('Błąd podczas łączenia z bazą danych');
 
mysql_select_db('xxx');
 
$zapytanie = "SELECT `login`,`ranga` FROM `users` WHERE `login`='$nazwa'";
$idzapytania = mysql_query($zapytanie);
$t = mysql_fetch_row($idzapytania);
?>
[PHP] pobierz, plaintext 

I kod zakładek dla odpowiednich osób:

[PHP] pobierz, plaintext 
<?php include('config1.php');
 
if( ("$t[1]") == 9) {
 
Treść dla administratora
 
					} else {
					echo '<center><BR><BR><BR>To nie miejsce dla ciebie...';
					echo '<meta http-equiv="refresh" content="1; url=index.php">';
					};
?>
[PHP] pobierz, plaintext 

Czy to jest bezpieczne? Tak żeby jakiś zwykły user nie mógł zobaczyć zakładek dla administratora oraz do nich wejść.

ps. ja jestem autorem kodu, żeby nie było, że kopiuję od kogoś (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował Michal2451 3.01.2011, 20:38:19

[Mephistofeles]

PDO + prepared statements, tak na przyszłość. Do tego wywal te echa, zamknij tag PHP i normalnie pisz HTML.