Ver Mensaje Individual
  #23 (permalink)  
Antiguo 26/12/2006, 14:11
Avatar de nicolaspar
nicolaspar
 
Fecha de Ingreso: noviembre-2004
Ubicación: Villa Ballester Bs-As|Ar
Mensajes: 2.002
Antigüedad: 21 años, 10 meses
Puntos: 34
Re: ¿Es segura esta forma de autentificarse?

foo, no esta mal lo que decís, pero estamos confundiendo, y a mi se me paso por alto.
Yo cuando exprese opinión sobre mysql_real_escape_string me refería explícitamente a sql inyection, si no quedo claro lo aclaro ahora :P. En la función que presenta ferny tenes el htmlentities para los casos de xss (cosa que que no alcanza a mi parecer y quedo mas que obvio con tu ejemplo), pero no había visto el comentario de la función que dice: "Función para evitar ataques SQL injection y XSS".
, de haberlo visto habría hecho un comentario a esto, sorry.

Personalmente tengo ambas cosas separadas, y si sirve de algo, una de las funciones que uso para xss es la siguiente (es la que tengo a mano)

Código PHP:
function sacarXss($val) {
   
$val = preg_replace('/([\x00-\x08][\x0b-\x0c][\x0e-\x20])/', '', $val);
   
$search = 'abcdefghijklmnopqrstuvwxyz';
   
$search .= 'ABCDEFGHIJKLMNOPQRSTUVWXYZ';
   
$search .= '1234567890!@#$%^&*()';
   
$search .= '~`";:?+/={}[]-_|\'\\';
   for (
$i = 0; $i < strlen($search); $i++) {
      
$val = preg_replace('/(&#[x|X]0{0,8}'.dechex(ord($search[$i])).';?)/i', $search[$i], $val); // with a ;
      
$val = preg_replace('/(&#0{0,8}'.ord($search[$i]).';?)/', $search[$i], $val); // with a ;
   
}
   
$ra1 = Array('javascript', 'vbscript', 'expression', 'applet', 'meta', 'xml', 'blink', 'link', 'style', 'script', 'embed', 'object', 'iframe', 'frame', 'frameset', 'ilayer', 'layer', 'bgsound', 'title', 'base');
   
$ra2 = Array('onabort', 'onactivate', 'onafterprint', 'onafterupdate', 'onbeforeactivate', 'onbeforecopy', 'onbeforecut', 'onbeforedeactivate', 'onbeforeeditfocus', 'onbeforepaste', 'onbeforeprint', 'onbeforeunload', 'onbeforeupdate', 'onblur', 'onbounce', 'oncellchange', 'onchange', 'onclick', 'oncontextmenu', 'oncontrolselect', 'oncopy', 'oncut', 'ondataavailable', 'ondatasetchanged', 'ondatasetcomplete', 'ondblclick', 'ondeactivate', 'ondrag', 'ondragend', 'ondragenter', 'ondragleave', 'ondragover', 'ondragstart', 'ondrop', 'onerror', 'onerrorupdate', 'onfilterchange', 'onfinish', 'onfocus', 'onfocusin', 'onfocusout', 'onhelp', 'onkeydown', 'onkeypress', 'onkeyup', 'onlayoutcomplete', 'onload', 'onlosecapture', 'onmousedown', 'onmouseenter', 'onmouseleave', 'onmousemove', 'onmouseout', 'onmouseover', 'onmouseup', 'onmousewheel', 'onmove', 'onmoveend', 'onmovestart', 'onpaste', 'onpropertychange', 'onreadystatechange', 'onreset', 'onresize', 'onresizeend', 'onresizestart', 'onrowenter', 'onrowexit', 'onrowsdelete', 'onrowsinserted', 'onscroll', 'onselect', 'onselectionchange', 'onselectstart', 'onstart', 'onstop', 'onsubmit', 'onunload');
   
$ra = array_merge($ra1, $ra2);
   
$found = true; 
   while (
$found == true) {
      
$val_before = $val;
      for (
$i = 0; $i < sizeof($ra); $i++) {
         
$pattern = '/';
         for (
$j = 0; $j < strlen($ra[$i]); $j++) {
            if (
$j > 0) {
               
$pattern .= '(';
               
$pattern .= '(&#[x|X]0{0,8}([9][a][b]);?)?';
               
$pattern .= '|(&#0{0,8}([9][10][13]);?)?';
               
$pattern .= ')?';
            }
            
$pattern .= $ra[$i][$j];
         }
         
$pattern .= '/i';
         
$replacement = substr($ra[$i], 0, 2).'<x>'.substr($ra[$i], 2);
         
$val = preg_replace($pattern, $replacement, $val);
         if (
$val_before == $val) {
            
$found = false;
         }
      }
   }
} 
__________________
Mi punto de partida es Que Bueno Lo Nuevo