¿Cómo es vulnerable mi sitio web para las inyecciones de SQL?

Siempre use consultas parametrizadas, la concatenación de cadenas deja espacio para el error, deje de salir de esa habitación (también se ven mucho mejor).

Además, como id es un int, no necesito comillas para inyectar:

$id = '0; Drop Table myTable;--';

$sql = "SELECT * , COUNT( * ) AS amount
        FROM articles
        WHERE id = 0; Drop Table myTable;--
        GROUP BY id";

Me gusta usar is_numeric para verificar si es un número y eliminarlos no es.

Editar:

PDO proporciona preparación de consultas SQL: enlace

Entonces, por ejemplo:

$sql = "SELECT * , COUNT( * ) AS amount
    FROM articles
    WHERE id = :id
    GROUP BY id";
$sth = $dbh->prepare($sql);
$sth->execute(array(':id' => $id));

En primer lugar, está tomando los pasos correctos, no solo asumiendo que su código es seguro, sino en realidad probándolo.

En segundo lugar, Sqlmap le dará algunos niveles de detalles sobre cómo llegó exactamente a un punto SQLi. Pruebe alguno de los siguientes cuando lo ejecute:

• sqlmap.py -v 3 --dbs -u enlace

El interruptor -v 3 hará que se imprima la carga útil exacta inyectada. También debería poder ver esto en los registros de acceso y error de su servidor web.

En tercer lugar, mirando el fragmento de código: es posible que desee probar:

• (int) $ _ GET ['id']

Pero si puede darnos más detalles sobre lo que sqlmap dice que es el punto de inyección es, como cronometrado, ciego, de unión, booleano, etc., podríamos entrar en más detalles.

Todas las respuestas están bien, pero en su caso particular, escapó del número como una cadena ( mysql_real_escape_string() ), pero no la incluyó entre comillas simples como una cadena.

Si, en lugar de:

$sql = "SELECT * , COUNT( * ) AS amount
        FROM articles
        WHERE id = $id
        GROUP BY id";

usaste:

$sql = "SELECT * , COUNT( * ) AS amount
        FROM articles
        WHERE id = '$id'
        GROUP BY id";

habría ninguna inyección de SQL un menor riesgo de inyección de SQL. Pero siempre es mejor:

• usar declaraciones preparadas
• use la validación de entrada estricta: en su caso, debería estar lanzando id a integer .

La respuesta corta en esta situación en particular, y porque el $id siempre será un número, es como dijo Neil, para filtrar la entrada para asegurarse de que siempre va a ser un número entero, sin importar qué. / p>

Concepto de ejemplo:

$id = ( false !== ( int )$_GET[ 'id' ] >= 0 ) ?
        ( int )$_GET[ 'id' ] :
        die( header( "Location: ./index.php" ) );

Algo así aseguraría que $id solo pueda ser un número entero y no menos de 0 (es decir, no -1). Lo que haga con una identificación fallida depende de usted, en este ejemplo, la página se redirige al index.php .

En este ejemplo anterior, la siguiente solicitud de

id=55+union+select+1,2,3,4,5,6,7,8,9,10,11,12,13

Resultaría en que $sql es:

SELECT * , COUNT( * ) AS amount FROM articles
WHERE id = 55 GROUP BY id

en lugar de lo que devolvería el código actual, que es:

SELECT * , COUNT( * ) AS amount FROM articles
WHERE id = 55 union select 1,2,3,4,5,6,7,8,9,10,11,12,13 GROUP BY id