Legacy Trash (III) (Manejo de archivos subidos)

Buenas! Hoy seré breve.
¡Encontré otra función para quitar acentos! Llamada quitarAcentosb($str). Supongo que la "b" final es de "Bien" ya que esta no hace manipulaciones extrañas a los strings.
Sin embargo, no todo es diversión. La aplicación permite que un usuario suba imágenes. Si queremos ver el vaso medio lleno podemos decir que se verifica que el tipo de archivo sea una imagen. Y dejarlo ahí. Pero si nos ponemos más rigurosos, lo que en realidad se verifica, es que el tipo de archivo haya sido nombrado como imagen 


if (($ext[count($ext)-1]!=='jpg')&&($ext[count($ext)-1]!=='gif')&&($ext[count($ext)-1]!=='png')&&($ext[count($ext)-1]!=='jpeg'))

Imagino que existen innumerables sitios que tengan esta misma lógica detrás y eso da un poco de miedo. Si por esas vueltas de la vida, el que escribió esta aplicación encuentra el blog estaría bueno que sepa que la extensión de un archivo NO indica el tipo de archivo con el que estamos trabajando, la extensión solo es parte del nombre.
PHP trae constantes predefinidas para el manejo de imágenes.
Con algunos ajustes del contexto, este control de tipos podría hacerse mucho más seguro y sencillo con una condición del tipo:


if (($file_type !== IMAGETYPE_GIF) &&
    ($file_type !== IMAGETYPE_JPEG) &&
    ($file_type !== IMAGETYPE_PNG))


Otro tipo de verificación que se hace sobre el archivo subido es que su tamaño no supere el máximo. ¿Bien, no? Más o menos. La aplicación exige que el archivo tenga menos de 50MB. Lo que te permitiría subir una imagen y 3 instaladores de Windows 95 que hayan sido renombrados como gatitos.gif

Nos vemos!

Comentarios

Publicar un comentario

Entradas populares de este blog

Introduccion a x86 ASM - Registros

Imagen
Assembler no es un lenguaje. Es una familia de lenguajes dependientes de la arquitectura de los procesadores. Esta familia de lenguajes es fundamental en ingeniería inversa. Me fascina el concepto de ingeniería inversa aunque aún soy un lego casi total en la materia: esta guía también me sirve a mi. Por otra parte, demasiados desarrolladores no saben lo que es la ingeniería inversa y demasiados desarrolladores piensan en Assembler como si fuera un lenguaje de programación obsoleto. Esta serie de entradas, inspirada en esta otra Guía de la Universidad de Virginia pretende ser una guía introductoria al lenguaje assembler usado en los procesadores Intel x86. Como guía introductoria cubre solamente una pequeña porción (pero útil) de las directivas e instrucciones disponibles. Así como existen un lenguaje ensamblador para cada arquitectura, existen varios lenguajes para generar código máquina para procesadores x86. Esta guía utiliza el assembler de Microsoft Macro Assembler (MASM) , q...
Leer más

JS: Triángulo de Pascal

Hoy, un pequeño problema de código Javascript. El siguiente snippet pretende crear un triángulo de Pascal . Al constructor Triangle se le pasa el número de filas que se quiere generar para la solución. La función fillRows() llama a fillRow(numRow) una vez por cada fila que se quiere generar. La función fillRow(numRow) se encarga de poner los números que van en cada posición de cada fila. Para ello usa un Array llamado row donde pone auxiliarmente los valores antes de agregarlos a this.rows . Estarán de acuerdo conmigo si les digo que el console.log() comentado como log "B" debería devolver lo mismo que el de la línea 5 -log "A"-. Pero no. Los resultados de cada console.log para new Triangle(2).rows son: Log "B": [[1], [1, 1]]. Log "A": [[1], undefined]. Es decir, this.rows tiene los valores correctos dentro de la función fillRow(numRow) pero no fuera de ella. ¿Cuál es el problema?¿Es una cuestión de scopes?¿Debería cambiar la...
Leer más

Legacy Trash (II)

Segunda entrega de este paseo por una aplicación PHP sangrienta. Después de pasar los ojos sobre un montón de líneas de código cuya complejidad solo es justificable en caso de que el programador estuviera realmente preocupado por su estabilidad laboral y después de encontrar: -Una simpática variable llamada $cúmulo. -Un script cuyo título era el nombre y apellido del autor. -Un comentario /***SEGUIMOS***/ , que me tranquilizó porque pensé que ya no estábamos siguiendo... Encontré un par de cosas más interesantes. Dialecto simplificado function quitarAcentos($cad) { //40 lineas de código que no quiero discutir $arr['z']='s'; $arr['ce']='se'; $arr['ci']='si'; $arr['ge']='je'; $arr['gi']='ji'; $arr['v']='b'; $arr['ll']='y'; $arr[' h']=' '; $arr['x']=''; foreach ($arr as $i => $v) { $cad=str_replace($i,$v,$cad); } } ¿Vieron eso? a...
Leer más