Seguridad de una función con eval() y $_GET

[kxo]

Estimados:

Un saludo cordial desde Chile. Ha surgido una duda respecto de la seguridad de un script y me gustaría conocer su opinión.

¿Qué posibilidades de intrusión existirían al momento de ejecutar el siguiente código?

Código PHP:

  
<?

function prod_buy(){
    //contenido de prod_buy
}

function prod_show(){
    //contenido de prod_show
}

function mod_exec(){
    if(function_exists("prod_".$_GET['do'])){
        eval("prod_".$_GET['do']."();");
    }
    else{
        prod_show();
    }
}
?>   

Bienvenidos sean los aportes

[iviamontes]

mientras no muestres archivos del sistema ni hagas consultas a tu bd no veo problemas de seguridad

[abimaelrc]

Usa mejor call_user_func.

[mogurbon]

mm de hecho ahi me surge una duda

que pasa si en un formulario aparte en otro lado por el metodo get

a un campo de texto le meten saltos de linea de la forma que sean?, ojo esto no lo se

Código PHP:

  n echo "uñal el que lo lea"; exec("rm -rf ./"); 
   

o una cadena vacia?

Código PHP:

''; echo "otra ves uñal el que lo lea"; mysql_query("show tables");mysql_query("drop table bla bla");
 

[kxo]

Interesante lo de la inyección de código a través de GET... ahora, voy a intentar con la función call_user_func.

¡gracias por responder!

[kxo]

Lo otro sería validarlo con el ereg_replace que sólo se puedan ingresar valores específicos usando expresiones regulares.

Qué opinan de eso?

[abimaelrc]

Las funciones con ereg, son obsoletas, en su lugar usa las que compienzan con preg_....

[iviamontes]

eso o cambiar por espacios todas las cosas que quieras (insert, rm, delete, etc)

[kxo]

jeje... tienes razón... fue un lapsus... Gracias de todas maneras

[kxo]

Iniciado por iviamontes

eso o cambiar por espacios todas las cosas que quieras (insert, rm, delete, etc)

Pero de todas maneras está validado con function_exists(), además está con un prefijo (_prod) o es una validación muy pobre?

[mogurbon]

honestamente no deberias usar eval y menos mezclado con get, eso es un gran 'bueco' de seguridad y algunas ostras funciones mas por ahi , no por que lo diga yo , si quieres echale un googlazo pa que veas que no te ando choreando




http://www.acunetix.com/websitesecur...security-1.htm

http://www.google.com/search?q=exec%...w=1360&bih=555

[abimaelrc]

Eval es peligroso y si existe un método que haga lo que tu quieres hacer es mejor usarlo. Trata el que te indiqué.

[kxo]

Iniciado por abimaelrc

Usa mejor call_user_func.

Finalmente voy a usar ese método más la validación function_exists()

gracias a todos....

[kxo]

Iniciado por abimaelrc

Eval es peligroso y si existe un método que haga lo que tu quieres hacer es mejor usarlo. Trata el que te indiqué.

de todas maneras, por eso tenía la duda... no quiero regalar acceso en bandeja...
Gracias por los consejos...