Този код за новини защитен ли е

[phpbot]

Този код за новини защитен ли е защо имам упасение че не е :]

<?php
$tablica = $_GET['page']; //Таблицата в която ще добавя новина
include('conf.php');

$id= (int)$_GET['id'];
$query = mysql_query("SELECT * FROM $tablica where id='$id'");
if(mysql_num_rows($query) > 0)
{
while($row = mysql_fetch_array($query)){
echo'


<center><font size=5> '.$row['title'].' <br /></center></font><br />
</div>





                                	
              

<br>
<div align="center">
<a href="'.$row['picture1'].'" target="blank">';

if(file_exists(''.$row['picture1'].'')){
echo '<img src="'.$row['picture1'].'" alt="" width="200" border="0" />';
}

echo'</a><br /><br />
<a href="'.$row['file2'].'" target="blank">';

if(file_exists(''.$row['file2'].'')){
echo 'Изтегли '.$row['filedescription'].'';
}

echo'</a></div>


<div id="container">



<div id="info">
'.$row['article'].' 

</div>
</div>


';
}
}
else
{
header('Location: view');
exit;
}
?>

 

[streleca_stz]

<?php
$tablica = addslashes(htmlspecialchars($_GET['page'])); //Таблицата в която ще добавя новина
include('conf.php');

$id= (int)$_GET['id'];
$query = mysql_query("SELECT * FROM $tablica where id='$id'");
if(mysql_num_rows($query) > 0)
{
while($row = mysql_fetch_array($query)){
echo'
<center><font size=5> '.$row['title'].' <br /></center></font><br />
</div>
<br>
<div align="center">
<a href="'.$row['picture1'].'" target="blank">';

if(file_exists(''.$row['picture1'].'')){
echo '<img src="'.$row['picture1'].'" alt="" width="200" border="0" />';
}

echo'</a><br /><br />
<a href="'.$row['file2'].'" target="blank">';

if(file_exists(''.$row['file2'].'')){
echo 'Изтегли '.$row['filedescription'].'';
}
echo'</a></div>

<div id="container">
<div id="info">
'.$row['article'].'
</div>
</div>
';
}
}
else
{
header('Location: view');
exit;
}
?>

 

[slideR]

<?php
$tablica = trim(strip_tags($_GET['page'])); //Таблицата в която ще добавя новина
include('conf.php');

$id= (int)$_GET['id'];
$query = mysql_query("SELECT * FROM $tablica where id='$id'");
if(mysql_num_rows($query) > 0)
{
while($row = mysql_fetch_array($query)){
echo'
<center><font size=5> '.$row['title'].' <br /></center></font><br />
</div>
<br>
<div align="center">
<a href="'.$row['picture1'].'" target="blank">';

if(file_exists(''.$row['picture1'].'')){
echo '<img src="'.$row['picture1'].'" alt="" width="200" border="0" />';
}

echo'</a><br /><br />
<a href="'.$row['file2'].'" target="blank">';

if(file_exists(''.$row['file2'].'')){
echo 'Изтегли '.$row['filedescription'].'';
}
echo'</a></div>

<div id="container">
<div id="info">
'.$row['article'].'
</div>
</div>
';
}
}
else
{
header('Location: view');
exit;
}
?>

Така мисля, че е по-добре

 

[alex3]

Самия начин за ползване на GET за таблица и то за ползването й директно в заявката не е хубав, ползвай if или switch.

 

[phpbot]

Благодаря много trim(strip_tags и за switch ама аз ползвам него


index.php?p=newsfull Благодаря