Clase MYSQL e Inyecciones

[McBlink]

Buenas Tardes,
tengo una clase para realizar la conexión y consultas a la base de datos. Por ejemplo:

Código PHP:

<?php
session_start();
class MySQL{
 
    function MySQL(){
            if(!isset($this->conexion)){
                $this->conexion = (mysql_connect($this->host,$this->user,$this->pass)) or die(mysql_error());
                mysql_select_db($this->data,$this->conexion) or die(mysql_error());
            }
        }
        
    function query($consulta) {
        $resultado = mysql_query($consulta,$this->conexion);
        return $resultado; 
    }
 

de forma tal que llamo a :

Código PHP:

$row = $db->query("SELECT * FROM .....");
 

Ahora quiero implementar un poco de seguridad a estas consultas, y estuve leyendo bastante pero no me quedo muy claro.

1- validaría los datos por GET y POST antes de realizar la consulta, con funciones como is_numeric() y demás.

2- Aplicaría la función mysql_real_escape().

tengo la duda si esta función la aplico antes de realizar la consulta o en método de la clase que lo ejecuta, en el ejemplo seria en query().
Aplicando estas dos consideraciones, estarian seguras mis consultas?

Muchas Gracias
Saludos

[abimaelrc]

Usa PDO y con prepend, bindParam o bindValue es suficiente para prevenir inyecciones de primer grado.

[McBlink]

Gracias, pero por el momento no me interesa usar un framework para el manejo de la base de datos y además quiero hacer mi propia clase para aprender.

Saludos

[abimaelrc]

Ehhh PDO es una libreria de PHP asi como la libreria de mysql y PDO es la recomendada al momento hasta que PHP se invente una que lea los pensamientos

[McBlink]

Ah, OK, tenia entendido que era un Framework, le voy a dar una leída pero de te agradecería si me das tu opinión para poder implementar mi propia clase.
Gracias

[abimaelrc]

Bueno yo tengo una clase llamada Filter

Código PHP:

<?php
class Filter
{
    /**
     * Remove XSS attacks that came in the input
     *
     * Function taken from:
     * http://quickwired.com/smallprojects/php_xss_filter_function.php
     * and alter to use in application
     *
     * @param string $value The value to filter
     * @return string 
     */
    public function filterXss($params, $returnStr = false)
    {
        $params = is_array($params)
            ? $params
            : array($params);
 
        foreach($params as $key => $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){
                $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;
                    }
                }
            }
            $params[$key] = is_array($val)
                ? $this->filterXss($val)
                : $val;
        }
 
        return $returnStr
            ? $params[0]
            : $params;
    }
 
    /**
     * Remove XSS attacks and remove tags and extra white spaces
     * that came in the input before and after
     * 
     * @param string|array $params The value to filter
     * @param bool Set this if you want to return string value
     * @return string|array
     */
    public function realEscape($params, $returnStr = false)
    {
        /**
         * Check for XSS atacks
         */
        $params = $this->filterXss($params, $returnStr);
 
        $params = is_array($params)
            ? $params
            : array($params);
 
        foreach($params as $k => $v){
            /**
             * Recursive, re-send all values that are arrays
             */
            if(is_array($v)){
                $params[$k] = $this->realEscape($v);
                continue;
            }
            /**
             * Decode all hexadecimal values (urldecode and html_entity_decode)
             * Clean up all values (strip_tags)
             * Erase all white space before and after string (trim)
             */
            $params[$k] = urldecode($v);
            $params[$k] = html_entity_decode($params[$k]);
            $params[$k] = strip_tags($params[$k]);
            $params[$k] = trim($params[$k]);
        }
 
        return $returnStr
            ? $params[0]
            : $params;
    }
}
 

Uso

Código PHP:

$filter = new Filter;
$values = $filter->filterXss($_POST);
print_r($values);
 
//o puedes limpiarlo así
$filter = new Filter;
$values = $filter->realEscape($_POST);
print_r($values);
 
//Si quieres que te devuelva en string, indica true en el segundo parametro.
$filter = new Filter;
$value = $filter->filterXss($_POST['foo'], true);
 
//o puedes limpiarlo así
$filter =  new Filter;
$value = $filter->realEscape($_POST['foo'], true);
 

[RodrigoG]

Hace tiempo yo tuve el mismo problema, intentaba implementar la clase con mysql_conect, eso ya está anticuado :)

Como dice abimaelrc, PDO es hoy por hoy la mejor librería nativa de PHP para conectarse a cualquier base de datos popular (MySQL, Oracle, etc).

Ahora, al igual como se puede crear una clase con mysql_conect y familia, se puede hacer con PDO.

Aquí deje hace un tiempo una clase con PDO, por si te interesa, esta todo explicado y en español.

Saludos.

[historiasdemaria]

Para evitar inyecciones SQL:
Si no usas PDO en tu clase puedes tener un metodo asi:

Código PHP:

function static quote_smart($value)
{
// Stripslashes
if (get_magic_quotes_gpc()) {
$value = stripslashes($value);
}
// Ver si es numerico
 
if (!is_numeric($value)) {
$value = "'" . mysql_real_escape_string($value) . "'";
}
return $value;
}
 

este metodo debes llamarlo en las SQL queries

"SELECT * FROM lo_que_sea WHERE id=".SQL::quote_smart($value);