Seguridad

[Kristt]

Buenas gente , quisiera que alguien mirara un par de funciones que tengo para seguridad en consutas y para que tampoco me cuelen etiquetas html:

Código PHP:

  function limpiarHTML($string)
{
 $html=htmlspecialchars($string);
 $html=trim($html);
 return $html;
}
 
// y para mysql
 
function sql_safe($string){
  if(get_magic_quotes_gpc()) 
  {
    $string = stripslashes($string);
  }
  $badWords = "(delete)|(update)|(union)|(insert)|(drop)|(http)|(--)";
  $string = eregi_replace($badWords, "", $string);
  $string = mysql_real_escape_string($string);
  return $string;
} 
   

gracias a todos

[Labelin]

También puedes usar

Código PHP:

  htmlentities(); 
   

eficaz para evitar ataques XSS

[okram]

Dependiendo de la finalidad que tenga tu sistema, no creo que sea lo más óptimo borrar palabras tan "triviales" como SELECT, UNION, HTTP, o todas las que allí pones.

Supongo que no es necesario decirte que magic_quotes debe ser preferentemente desactivada en el php.ini, pero si no es posible hacerlo, realizar el escape al inicio del script. Así tienes TODOS los datos limpios:

Código php:

function stripslashes_array(&$datos) {
    $datos = is_array($datos) ? array_map('stripslashes_array', $datos) : stripslashes($datos);
    return $datos;
}
 
if( 
    ( function_exists("get_magic_quotes_gpc") and get_magic_quotes_gpc() ) or 
    ( ini_get('magic_quotes_sybase') and (strtolower(ini_get('magic_quotes_sybase')) != "off" ) )
) {
    stripslashes_array($_GET);
    stripslashes_array($_POST);
    stripslashes_array($_COOKIE);
}
 

A lo que voy es que ese proceso debe realizarse globalmente y no sólo a ciertos datos cuando van a formar parte de la consulta.

Por otro lado, regresando a los de la eliminación de palabras, como te digo no es recomendable borrarlas, aunque esto obviamente depende de la finalidad de tu sistema. Usar simplemente mysql_real_escape_string() sobre los datos, además de una validación con expresiones regulares, es más que suficiente para evitar inyecciones SQL.

[Kristt]

pues ni idea de lo de tener desactivado el get_magic_quotes del php.ini, a ese punto no he llegado, si me puedes orientar un poco mas,
de todas formas esta funcion que me has pasado, es válida tanto como si esta o como si no esta la directiva get_magic_quotes activada?.
muchas gracias por responder

[okram]

Ese bloque de código detecta el estado de magic_quotes (La parte del if), y si está activado realiza el stripslashes correspondiente para restaurar estos datos a sus valores originales. Si está en off, el script no hace nada.

Es mejor que leas directamente en el manual de php sobre magic_quotes y el riesgo que conlleva. Explicarlo por aquí sería repetir lo mismo que allí dice.