Comentarios en: PHP: Uso adecuado de parse_str http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/ Diseño y Programación Wed, 08 Feb 2012 16:34:31 +0000 hourly 1 Por: ¿Estamos seguros con la nueva versión de WordPress? en Buayacorp - Diseño y Programación http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-18553 ¿Estamos seguros con la nueva versión de WordPress? en Buayacorp - Diseño y Programación Tue, 07 Aug 2007 00:51:53 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-18553 [...] está en desarrollo — muchos problemas de seguridad, que en su mayoría se deben al uso de cadenas (tipo querystring) para pasar parámetros a funciones y a la ausencia de consultas parametrizadas. Unos cuantos bugs son graves (Fig. 1) y [...] [...] está en desarrollo — muchos problemas de seguridad, que en su mayoría se deben al uso de cadenas (tipo querystring) para pasar parámetros a funciones y a la ausencia de consultas parametrizadas. Unos cuantos bugs son graves (Fig. 1) y [...]

]]> Por: SQL Injection en el plugin de estadísticas de WordPress.com en Buayacorp - Diseño y Programación http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-17047 SQL Injection en el plugin de estadísticas de WordPress.com en Buayacorp - Diseño y Programación Mon, 30 Jul 2007 13:48:31 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-17047 [...] existe ningún tipo de validación en los parámetros que esa función recibe, y como mencioné en ocasiones anteriores, usar esta forma (query string) para pasar parámetros a otras funciones es bastante [...] [...] existe ningún tipo de validación en los parámetros que esa función recibe, y como mencioné en ocasiones anteriores, usar esta forma (query string) para pasar parámetros a otras funciones es bastante [...]

]]> Por: WordPress: Más vulnerabilidades de inyección de SQL en Buayacorp - Diseño y Programación http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-14465 WordPress: Más vulnerabilidades de inyección de SQL en Buayacorp - Diseño y Programación Fri, 13 Jul 2007 04:43:25 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-14465 [...] variante está en estrecha relación a lo discutido en “PHP: Uso adecuado de parse_str“, en esta entrada, a pesar de que existen más lugares donde se puede hacer lo mismo, sólo [...] [...] variante está en estrecha relación a lo discutido en “PHP: Uso adecuado de parse_str“, en esta entrada, a pesar de que existen más lugares donde se puede hacer lo mismo, sólo [...]

]]> Por: g30rg3_x http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12881 g30rg3_x Sat, 30 Jun 2007 01:17:57 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12881 jejejejeje.. Se me fue me concentre tanto en post_type que no le di importancia, si perdon error mio lapsus brutus de conocimiento de sql... Saludos jejejejeje..

Se me fue me concentre tanto en post_type que no le di importancia, si perdon error mio lapsus brutus de conocimiento de sql...

Saludos

]]> Por: alex http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12876 alex Sat, 30 Jun 2007 01:07:22 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12876 <strong>g30rg3_x</strong>, justamente me refería a ese problema, puesto que con <a href="http://test.buayacorp.com/wp.php?limit=2&type=foo%26limit=1%20UNION%20ALL%20..." rel="nofollow">valores parecidos</a> (<code>?limit=2&type=foo%26limit=1%20UNION%20ALL%20...</code>) al que mencionas, se sobreescribe el valor de la variable <code>limit</code> haciendo posible la inyección de SQL. [sql]SELECT * FROM posts WHERE post_type = 'foo' LIMIT 1 UNION ALL ...[/sql] <blockquote><p>[...], aunque claro no obtendremos nada beneficioso ya que nos encontramos de tras de un LIMIT [...]</p></blockquote> Eso es correcto siempre y cuando se haga uso de <code>ORDER BY</code> en la consulta, pero como en el ejemplo no existe esa parte se puede construir sentencias SQL perfectamente válidas. En el comentario anterior hacía referencia a <em>arrays</em>, porque básicamente al ser éstas tablas hash, la última asignación a una misma <em>llave</em> (<code>limit</code>) es la que cuenta. Saludos g30rg3_x, justamente me refería a ese problema, puesto que con valores parecidos (?limit=2&type=foo%26limit=1%20UNION%20ALL%20...) al que mencionas, se sobreescribe el valor de la variable limit haciendo posible la inyección de SQL.

sql: 
SELECT * FROM posts WHERE post_type = 'foo' LIMIT 1 UNION ALL ...

[...], aunque claro no obtendremos nada beneficioso ya que nos encontramos de tras de un LIMIT [...]

Eso es correcto siempre y cuando se haga uso de ORDER BY en la consulta, pero como en el ejemplo no existe esa parte se puede construir sentencias SQL perfectamente válidas.

En el comentario anterior hacía referencia a arrays, porque básicamente al ser éstas tablas hash, la última asignación a una misma llave (limit) es la que cuenta.

Saludos

]]> Por: g30rg3_x http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12869 g30rg3_x Sat, 30 Jun 2007 00:07:38 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12869 Bueno sigo sin ver lo que creo que estas pidiendo.. Pero mirando con mas detenimiento veo un medio-SQL-injection en $limit, ya que parse_str nos permite sobreescribir variables locales (http://www.acid-root.new.fr/advisories/14070612.txt) podriamos llamar a algo asi... ?limit=2&type=foo%26limit=bar Asi basicamente estamos pasando el filtro de solo numeros (o mejor dicho el casting a integer) que hay un poco mas arriba, aunque claro no obtendremos nada beneficioso ya que nos encontramos de tras de un LIMIT pero talvez nos podria servir para sacar informacion. Aunque como te dije aun no le veo por donde intente meter arrays durante pero ninguna de mis pruebas me dio positivo, asi que yo tambien quedo al pendiente de la solucion de la segunda parte... Saludos alex... Bueno sigo sin ver lo que creo que estas pidiendo..
Pero mirando con mas detenimiento veo un medio-SQL-injection en $limit, ya que parse_str nos permite sobreescribir variables locales (http://www.acid-root.new.fr/advisories/14070612.txt) podriamos llamar a algo asi...

?limit=2&type=foo%26limit=bar

Asi basicamente estamos pasando el filtro de solo numeros (o mejor dicho el casting a integer) que hay un poco mas arriba, aunque claro no obtendremos nada beneficioso ya que nos encontramos de tras de un LIMIT pero talvez nos podria servir para sacar informacion.

Aunque como te dije aun no le veo por donde intente meter arrays durante pero ninguna de mis pruebas me dio positivo, asi que yo tambien quedo al pendiente de la solucion de la segunda parte...

Saludos alex...

]]> Por: jdeveloper http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12767 jdeveloper Fri, 29 Jun 2007 08:08:35 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12767 gracias alex, en cuanto tenga tiempo lo mirare mas detenidamente, pero ya voy pillando la dinamica a seguir. gracias alex, en cuanto tenga tiempo lo mirare mas detenidamente, pero ya voy pillando la dinamica a seguir.

]]> Por: alex http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12760 alex Fri, 29 Jun 2007 04:54:57 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12760 El segundo error tiene que ver en parte con <code>parse_str</code> y el tratamiento de <em>arrays</em>. El segundo error tiene que ver en parte con parse_str y el tratamiento de arrays.

]]> Por: paco http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12733 paco Thu, 28 Jun 2007 20:06:25 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12733 yo no veo nada, podrías echarnos una manita? :S yo no veo nada, podrías echarnos una manita? :S

]]> Por: alex http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12723 alex Thu, 28 Jun 2007 17:47:17 +0000 http://www.buayacorp.com/archivos/php-uso-adecuado-de-parse_str/#comment-12723 @g30rg3_x: Todavía queda otro problema de SQL Injection ;) @jdeveloper: Simplemente está codificando el apóstrofe dos veces. [php]<?php echo urlencode(urlencode('\'')); ?>[/php] Se hace esto para que la función <code>addslashes</code> no tenga ningún efecto sobre el valor de <code>$_GET['type']="algo%27"</code>, este último se convierte en <code>$post_type="algo'"</code> gracias a la acción de parse_str. Puedes ver un ejemplo en la <a href="http://test.buayacorp.com/wp.php?limit=2&type=algo%2527%20AQUI%20INYECTAR%20SQL%20/*" rel="nofollow">página de prueba</a>. Saludos @g30rg3_x: Todavía queda otro problema de SQL Injection ;)

@jdeveloper: Simplemente está codificando el apóstrofe dos veces.

php: 
<?php echo urlencode(urlencode('\'')); ?>

Se hace esto para que la función addslashes no tenga ningún efecto sobre el valor de $_GET['type']="algo%27", este último se convierte en $post_type="algo'" gracias a la acción de parse_str.

Puedes ver un ejemplo en la página de prueba.

Saludos

]]>