como evitar los ataques xss atraves de los formularios

[tripoli]

Hola, como seria la forma mas segura de evitar los ataques xss a traves de los formularios, espero sujerencias, gracias de antemano.

[millan2525]

Busqueda de google a "evitar xxs en php" primer resultado
http://blog.unijimpe.net/prevenir-ataques-xss-con-php/


¿nos ves cara de buscador?

[abimaelrc]

Te recomiendo que hagas una búsqueda en el foro para que veas varios temas que te pueden ayudar.

[tripoli]

Gracias a todos por la informacion, saludos.

[tripoli]

Hola, de nuevo estube buscando por el foro y encontre una funcion de gatorV bastante interesante, aunque meda un error, la funcion esta en este post:

http://www.forosdelweb.com/f18/funci...5/#post3497564





yo hago algo asin:

Código PHP:

<?php
if(isset($_POST['dato'])){
    $dato=$_POST['dato'];
    $dato_limpio=filterXSS($dato);
    
   echo $dato_limpio;
}
?>
 

Me da este error:

Parse error: syntax error, unexpected T_PRIVATE in C:\wamp\www\xss.php on line 16

Alguna idea de porque me da ese error?saludos

[webankenovi]

en la funcion private function filterXSS($val) CAMBIA $DATO POR $VAL

<?php
if(isset($_POST['dato'])){
$val=$_POST['dato'];
$dato_limpio=filterXSS($val);

echo $dato_limpio;
}
?>

prueba haber

[tripoli]

El problema lo tengo en la funcion de gatorV, yo la cargo en la cabecera con un include pero me da este error:

Parse error: syntax error, unexpected T_PRIVATE

es esta linea:

Código PHP:

private function filterXSS($val) {
 

Alguna idea?saludos

[webankenovi]

te lo acabo de decir

la funciont actua con el valor de $val por eso cambia $dato a $val PRUEBALO Y NOS DICES

[tripoli]

Iniciado por webankenovi

te lo acabo de decir

la funciont actua con el valor de $val por eso cambia $dato a $val PRUEBALO Y NOS DICES

Eso hice pero el error me da el mismo, yo probe solo de incluir la funcion sin llamarla y me da el error que postee arriba, saludos.

[webankenovi]

function

Código PHP:

/**
         * Removes all XSS attacks that came in the input.
         *
         * Function taken from:
         *
         * http://quickwired.com/smallprojects/php_xss_filter_function.php
         *
         * @param mixed $val The Value to filter
         * @return mixed
         */
        private function filterXSS($val) {
            // remove all non-printable characters. CR(0a) and LF(0b) and TAB(9) are allowed
            // this prevents some character re-spacing such as <java\0script>
            // note that you have to handle splits with \n, \r, and \t later since they *are* allowed in some inputs
            $val = preg_replace('/([\x00-\x08][\x0b-\x0c][\x0e-\x20])/', '', $val);
     
            // straight replacements, the user should never need these since they're normal characters
            // this prevents like <IMG SRC=&#X40&#X61&#X76&#X61&#X73&#X63&#X72&#X69&#X70&#X74&#X3A&#X61&#X6C&#X65&#X72&#X74&#X28&#X27&#X58&#X53&#X53&#X27&#X29>
            $search = 'abcdefghijklmnopqrstuvwxyz';
            $search .= 'ABCDEFGHIJKLMNOPQRSTUVWXYZ';
            $search .= '1234567890!@#$%^&*()';
            $search .= '~`";:?+/={}[]-_|\'\\';
            for ($i = 0; $i < strlen($search); $i++) {
                // ;? matches the ;, which is optional
                // 0{0,7} matches any padded zeros, which are optional and go up to 8 chars
     
                // &#x0040 @ search for the hex values
                $val = preg_replace('/(&#[x|X]0{0,8}'.dechex(ord($search[$i])).';?)/i', $search[$i], $val); // with a ;
                // &#00064 @ 0{0,7} matches '0' zero to seven times
                $val = preg_replace('/(&#0{0,8}'.ord($search[$i]).';?)/', $search[$i], $val); // with a ;
            }
     
            // now the only remaining whitespace attacks are \t, \n, and \r
            $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; // keep replacing as long as the previous round replaced something
            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); // add in <> to nerf the tag
                 $val = preg_replace($pattern, $replacement, $val); // filter out the hex tags
                 if ($val_before == $val) {
                    // no replacements were made, so exit the loop
                    $found = false;
                 }
              }
            }
     
            return $val;
        }
 

tu script

Código PHP:

<?php
    if(isset($_POST['dato'])){
        $val=$_POST['dato'];
        $dato_limpio=filterXSS($val);
       
       echo $dato_limpio;
    }
    ?>
 

asi dices que te da error?

[tripoli]

webankenovi si asin me da error, la linea que comente arriba, alguna idea?


el problema lo tengo en el inicio de la funcion en el private, alguna idea?

[spider_boy]

La confusión de webankenovi es muy común. El nombre que le das a una variable local en una función, es solo para esa función, no para código externo. Lo que tiene tripoli está correcto, el problema es que copió y pegó, ni se fijó que el ejemplo de GatorV es un método, no una función. Pero es tan simple como quitar 'private' antes de 'function'.

[tripoli]

Hola spider_boy, en principio gracias por tu colaboracion, no habia caido en eso, saludos.