שלום,
רציתי לדעת באיזה מקרים יש סיכונים בשימוש בפונקציה הזאת?
השתמשתי בה רק לבחור עמודות מסויימות ולפלוט. הגדרתי את ה-get כמספר (int).
האם זה מאובטח מספיק או לא?


תודה מראש.

הסכנה היא בעיקר בsql injection, שיוכלו להחליף ערכים בקלות.
זה תלוי, אם בדקת שאותו משתמש יכול לצפות בעמוד כשפרמטר שהעברת הוא בערך שהעברת אז זה בסדר כי אז למשל ביטלת את האפשרות שמשתמש יוכל לקבל גישה לכל הפרופילים או כד דבר אחר בסגנון.

תודה על התגובה.
מדובר ב-RSS לפורומים. ה-get מקבל את מספר הפורום, ולפיו מציג את העדכונים.
הבדיקה היחידה שעשיתי זה אם המספר קטן מ-1 או שווה ל-3 (פורום מנהלים), אזי הכנס את הערך ה-1.
וזה הקוד:

קוד PHP:

 $_GET['id']=(int) $_GET['id'];

if ($_GET['id'] < 1 or $_GET['id'] = 3)
{
$_GET['id'] = 1;
}
mysql_connect('localhost','user','pass');
mysql_select_db('dbname') or die("ERROR: ".mysql_error()); 


זה מאובטח מספיק, או שצריך לנקוט בעוד צעדים?

אם אין לך פורום שמספרו 0, אז זה מצוין מה שכתבת.

תבדוק אם מדובר במספר, עם הפונקציה is_numeric

זה חייב להיות מספר...הוא הגדיר את זה כ- int

תודה רבה עזרתם לי מאד.