مشکل در اپدیت جداول دیتابیس

من تو ی اپدیت کردن اطلاعات جداول دیتا بیس مشکل دارم
[PHP]<?php
$data=$_REQUEST['categori'];
$name=$_REQUEST['name'];
$mahsol=$_REQUEST['mahsol'];
$pric=$_REQUEST['pric'];
$email=$_REQUEST['email'];
$pass1=$_REQUEST['pass1'];
$pass2=$_REQUEST['pass2'];
if($pass1!==$pass2){
echo 'رمزعبور اول و دوم با هم یکسان نیستند. لطفا برگردید و آنها را اصلاح کنید';
exit;}
$pass1=sha1($pass1);
$mobil=$_REQUEST['mobil'];
$tele=$_REQUEST['tele'];
$adres1=$_REQUEST['addres1'];
$adres2=$_REQUEST['addres2'];
$sit=$_REQUEST['site'];
if($_FILES["userfile"]['size']){
$upload_dir="./upload/";
$userfile = $_FILES["userfile"]['tmp_name'];
$userfile_name = $_FILES["userfile"]['name'];
$userfile_size = $_FILES["userfile"]['size'];
$filename = basename($userfile_name);
if(! copy( $userfile, $upload_dir.$filename )){
echo "error in upload";
return false;
}
}
else
{
$userfile_name="close.gif";}
$describ=$_REQUEST['describ'];
if(empty($data)||empty($name)||empty($mahsol)||emp ty($mobil)||empty($tele)||empty($adres1)||empty($d escrib)||empty($pass1)){
echo '<p>اطلاعاتتون کامل نیست لطفا به صفحه سفارشات باز گردید و سفارش خود را کامل کنید</p>';
exit;}
@$db=new mysqli('localhost','omom','varedi','marja');
$query="update ".$data." set name=".$name.", mahsol=".$mahsol.", price=".$pric.", email=".$email.", mobil=".$mobil.", tele=".$tele.", addres1=".$adres1.", addres2=".$adres2.", pic=".$userfile_name.", describ=".$describ.", where email=".$email." and pass=".$pass1."";
$result=$db->query($query);
$db->close();
?>[/PHP]

با این کد اطلاعات واسه اپدیت رو وارد می کنم. هیچ اروری نمی ده ولی اطلاعات اپدیت نمی شه. کجاش مشکل داره

اینکه اروری رو نشون نمیده دلیل عدم وجود ارور نیست
باید ارورش رو ازش بگیرید با این:
[php]$db->error;[/php]

البته تا حدی هم مشخصه که مشکلش چیه
وقتی نوع فیلدی در دیتابیس string هست (char, varchar, text...) توی کوئری باید مقدارش رو در کوتیشن بزاری یعنی اینجوری:
[php]
$query="update ".$data." set name='".$name."', mahsol='".$mahsol."'...[/php]

بازم درست نشد
:41:

اگه یه کم مرتب تر بنویسی راحتتر میشه اشکالش رو پیدا کرد.
من چک کردم چند تا ارور داشت که درست کردم:
[php]
<?php
$data=$_REQUEST['categori'];
$name=$_REQUEST['name'];
$mahsol=$_REQUEST['mahsol'];
$pric=$_REQUEST['pric'];
$email=$_REQUEST['email'];
$pass1=$_REQUEST['pass1'];
$pass2=$_REQUEST['pass2'];

if($pass1!==$pass2){
echo 'رمزعبور اول و دوم با هم یکسان نیستند. لطفا برگردید و آنها را اصلاح کنید';
exit;
}

$pass1=sha1($pass1);
$mobil=$_REQUEST['mobil'];
$tele=$_REQUEST['tele'];
$adres1=$_REQUEST['addres1'];
$adres2=$_REQUEST['addres2'];
$sit=$_REQUEST['site'];

if($_FILES["userfile"]['size']) {
$upload_dir="./upload/";
$userfile = $_FILES["userfile"]['tmp_name'];
$userfile_name = $_FILES["userfile"]['name'];
$userfile_size = $_FILES["userfile"]['size'];

$filename = basename($userfile_name);
if(! copy( $userfile, $upload_dir.$filename )) {
echo "error in upload";
return false;
}
} else {
$userfile_name="close.gif";
}

$describ=$_REQUEST['describ'];

if(empty($data)||empty($name)||empty($mahsol)||emp ty($mobil)||empty($tele)||empty($adres1)||empty($d escrib)||empty($pass1)) {
echo '<p>اطلاعاتتون کامل نیست لطفا به صفحه سفارشات باز گردید و سفارش خود را کامل کنید</p>';
exit;
}

$db = new mysqli('localhost','omom','varedi','marja');
if(!$db) {
exit(mysqli_connect_error());
}

$query = "
update $data
set
name = '$name',
mahsol = '$mahsol',
price ='$pric',
email = '$email',
mobil = '$mobil',
tele = '$tele',
addres1 = '$adres1',
addres2 = '$adres2',
pic = '$userfile_name',
describ = '$describ'
where
email = '$email'
and pass = '$pass1';
";

$result=$db->query($query);
if(!$result) {
exit($db->error);
}

$db->close();
?>
[/php]
شما هم تست کن اگه ارور داشت متن ارور رو بنویس تا ببینیم چی میگه.
البته اینم گفته باشم که این کد از نظر امنیتی خیلی مشکل داره. اگه بره رو اینترنت احتمالاً خیلی زود هک میشه!

نقل قول:

---

البته اینم گفته باشم که این کد از نظر امنیتی خیلی مشکل داره. اگه بره رو اینترنت احتمالاً خیلی زود هک میشه!

---

امکانش هست کمی واضحتر بفرمایید ؟

از روی کدام دستورات می گی که این کد ها مشکل دارند ؟

اگه خواستید یه تایپیک جدید بزنید و بگید واقعا ممنون شما می شم .

من می خوام بدونم اگه امکانش هست بگید ممنون می شم

ممنون از لطفت. درست شد. اما در مورد امنیت گفتی. من تمام کدها رو قرار ندارم. فقط اونهایی که لازم بود و احتمال اشکال داشت رو قرار دادم. ولی یه سوال اگه در حین ویرایش عکسی که اپلود میشه هم نام یه عکس دیگه باشه چی ؟ چه طور می تونم اینو بفهمم؟
:20:

نقل قول:

---

نوشته شده توسط elyass [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

ممنون از لطفت. درست شد. اما در مورد امنیت گفتی. من تمام کدها رو قرار ندارم. فقط اونهایی که لازم بود و احتمال اشکال داشت رو قرار دادم. ولی یه سوال اگه در حین ویرایش عکسی که اپلود میشه هم نام یه عکس دیگه باشه چی ؟ چه طور می تونم اینو بفهمم؟
:20:

---

فایل جدید جای فایل قبلی رو میگیره
راه حلش اینه که اول چک کنی ببینی فایلی با نام مورد نظر وجود داره یا نه اگر وجود داشت نام فایل جدید رو باید عوض کنی

نقل قول:

---

نوشته شده توسط as13851365 [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

امکانش هست کمی واضحتر بفرمایید ؟

از روی کدام دستورات می گی که این کد ها مشکل دارند ؟

اگه خواستید یه تایپیک جدید بزنید و بگید واقعا ممنون شما می شم .

من می خوام بدونم اگه امکانش هست بگید ممنون می شم

---

منظورم SLQ Injection بود. البته احتمالاً حملات دیگه ای هم هست که این کد باهاش مشکل داره که بستگی به بقیه کد داره که اینجا نوشته نشده.
به جز مشکل هک شدن ممکنه حین ارسال اطلاعات سالم هم به مشکل بر بخوره به دلیل تداخل اسلشهای magic quote

منظورتون از magic quateرو نفهمیدم. چی کار کنم که این مشکل پیش نیاد. یه جا یه کد [PHP]magic_quote_gpc[/PHP]دیدم. منظورت همونه؟

بله منظورم همونه. توضیحاتش هم اینجا هست:

کد:

---

http://forum.p30world.com/showpost.php?p=2507270&postcount=91

---