Hallar la inyeccion en el codigo

Auditar una inyeccion es seguir un hilo: donde entra el dato del usuario y donde termina. En catalogo.php de Rividata la entrada es $_GET['cat'], y el hilo termina dentro de una cadena que se arma pegando ese valor entre comillas en un SELECT. Cuando el dato y la instruccion viven en la misma cadena, el motor no puede distinguir uno de otra: ahi esta el fallo. El auditor lo reconoce por la concatenacion, no por probar nada.

Abre el archivo y mira como se construye $sql. La pista es el operador que pega la variable dentro del texto de la consulta.

Encontrar el fallo es media auditoria; la otra media es recomendar el arreglo correcto. Contra la inyeccion SQL el arreglo no es filtrar palabras ni escapar comillas a mano —eso siempre se le escapa un caso—, sino usar una consulta parametrizada: la instruccion va con marcadores fijos y el dato se envia aparte, de modo que el motor nunca lo interpreta como orden. Es la unica correccion que cierra la clase entera del fallo.

Una lista negra de palabras prohibidas parece razonable y no lo es: siempre falta una forma de escribir lo mismo.

La inyeccion SQL es la mas conocida, pero el patron —entrada del usuario que se mezcla con instrucciones sin separar— aparece en otros sitios. En herramientas.php de Rividata el valor $_GET['host'] se concatena dentro de una llamada a shell_exec, el interprete de comandos del sistema. Es la misma clase de fallo con otro destino: inyeccion de comandos. El auditor la clasifica distinto (CWE-78) porque el arreglo tambien cambia: separar los argumentos del comando en vez de pegarlos.

Reconocer que es el mismo patron en otra salida es lo que distingue a quien audita de quien memoriza un caso.

La nota de triaje de Rividata recoge el hallazgo principal: el archivo, el parametro vulnerable, su clasificacion CWE y el arreglo. Abrela en el laboratorio, confirma que describe la inyeccion SQL de catalogo.php y lee el codigo del hallazgo con el que cierra. Ese codigo identifica el fallo en el informe que va a desarrollo.