Forum PHP.pl > [MySQL][PHP]Bezpiczeństwo mojego skryptu

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [MySQL][PHP]Bezpiczeństwo mojego skryptu

Vion

17.02.2011, 17:52:56

Witam
O to mój skrypt, jakie popełniłem błędy przy bezpieczeństwie. Jak powinienem się zabezpieczyć.

[PHP] pobierz, plaintext 
$wynik = "SELECT * FROM shot WHERE shot_is_published ='1' and shot_id='".$_GET['id']."' ORDER BY RAND()  LIMIT 1 ";
$results=mysql_query($wynik);
while($row = mysql_fetch_array($results)) {
  $image_id= $row['shot_id'];
  $image_filename= "shot/" . $image_id . ".jpg"; ?>
   <div class="post">
     <div class="entry">
       <p class="meta">Data dodania: <?php echo $row['shot_date_submitted'];?></p><br>
       <img src='<?php echo $image_filename; ?>'>
     </div>
     <div class="byline">
    <br>
     <p class="meta"><a type="button_count" share_url="epicshot.pl/view.php?id=<?php echo $row['shot_id']; ?>"name="fb_share">Share</a> 
     <script      src="http://static.ak.fbcdn.net/connect.php/js/FB.Share" type="text/javascript"></script></p>
     </div>
  </div>
<?php }
[PHP] pobierz, plaintext

Piogola

17.02.2011, 18:57:17

Nie filtrujesz zmiennej $_GET['id']. Strona podatna jest na SQL Injection

Vion

17.02.2011, 20:00:37

trochę poczytałem i zmieniłem na coś takiego:

[PHP] pobierz, plaintext 
$id=(int)$_GET['id'];
$id = trim($id);
$id = htmlspecialchars($id);
$wynik = "SELECT * FROM shot WHERE shot_is_published ='1' and shot_id='$id' ORDER BY RAND()  LIMIT 1 ";
$results=mysql_query($wynik);
while($row = mysql_fetch_array($results)) {
	$image_id= $row['shot_id'];
	$image_filename= "shot/" . $image_id . ".jpg"; ?>
	<div class="post">
		<div class="entry">
			<p class="meta">Data dodania: <?php echo $row['shot_date_submitted'];?></p><br>
			<img src='<?php echo $image_filename; ?>'>
		</div>
		<div class="byline">
			<br>
			<p class="meta"><a type="button_count" share_url="epicshot.pl/view.php?id=<?php echo $row['shot_id']; ?>"name="fb_share">Share</a> <script src="http://static.ak.fbcdn.net/connect.php/js/FB.Share" type="text/javascript"></script></p>
		</div>
	</div>
<?php }
[PHP] pobierz, plaintext

hyhyhy

17.02.2011, 20:27:10

w zasadzie jestem początkujący i moge pocisnąć głupote, ale... może dorzuć mysql_real_escape_string() ?

Piogola

17.02.2011, 20:33:29

@Vion
Jeśli rzutujesz typ na zmienną $id to dalej nie musisz filtrować zmiennej, bo zmienna $id może być tylko liczbą.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.