Wątek: [SQL Injection] class/player_class.php

1. Opis błędu

Brak filtracji zmiennej $_GET['pid'].

2. Zagrożenie

Błąd umożliwiający atak SQL Injection.

3. Rozwiązanie

[OTWÓRZ]
class/player_class.php
[ZNAJDŹ]

Kod php:

$stats = $db -> Execute("SELECT * FROM `players` WHERE `id`=".$pid); 


[ZAMIEŃ NA]

Kod php:

        $pid = intval($pid);
        $stats = $db -> Execute("SELECT * FROM `players` WHERE `id`=".$pid);
        if ($stats -> fields['id'] != $pid) return FALSE; 


nie wiem jak działa de facto ta klasa bazy stąd moje pytanie:

$pid = intval($pid);
załóżmy że $pid ma jakąś błędną (np tekstową) wartość to po tej operacji będzie wynosiło 0

$stats = $db -> Execute("SELECT * FROM `players` WHERE `id`=".$pid);
więc to pobierze nam dane usera 0 (zakładam, że takiego w bazie nie ma), więc rozumiem, że $stats będzie puste

czy zatem
if ($stats -> fields['id'] != $pid) return FALSE;
nie wywali błędu, że nie istnieje zmienna fields['id'] ??

nie wiem nie znam się na tej klasie bazy więc pytam ;]

Sypnie błędem "Nie ma takiego gracza!"

Mi nie wywaliło, tylko zwróciło FALSE.

spoko loko tylko pytam
standardowo kod

$stats = mysql_fetch_object(mysql_query("SELECT * FROM `players` WHERE `id`=".$pid));
if ($stats->id != $pid) return FALSE;

by sypnął błędem bo $stats jest puste i nie istnieje $stats->id

dlatego tak skojarzyłem i chciałem zapytać, żeby nie było, że ciągną kod z błędem