Nombres temporales de archivo PHP para cargas colisionando

Question

Cuando un usuario carga un archivo, al azar lo reemplaza la carga de otro usuario, finalmente he rastreado el problema hasta PHP y el nombre del archivo tmp se reutiliza. ¿Hay alguna manera de arreglar esto? ¿Hay alguna manera de hacer mejores nombres al azar? Parece degradarse con el tiempo, ya que en el nombre de archivo aleatorio, la semilla se debilita. Esto es en PHP 5.2.8 y FreeBSD 7.0

Aquí es un registro que muestra cómo el mismo nombre de archivo tmp se acostumbra y se sobrescribe con otra carga: http://pastebin.com/m65790440

Cualquier ayuda es muy apreciada. He estado tratando de arreglar esto por más de 4 meses y ha empeorado con el tiempo. Gracias.

EDITAR: Tenga en cuenta que esto no es un problema de código PHP, esto sucede antes de llegar a ningún código PHP, el archivo recibido a través de $ _FILES ['nombre'] ['tmp_name'] es incorrecto cuando se recibe y se remonta a que se sobrescribe con la carga de otra persona antes de que llegue al script de procesamiento de carga

Answer 1

Recomendaría usar un generador de GUID para el nombre de archivo, ya que está obteniendo tantos.

Answer 2

¿Funciona PHP bajo apache, como mod_php?

Usted puede tratar de create a per-process temporary upload directory cuyo nombre contiene el php getmypid(), a continuación, ini_set su proceso de PHP upload_tmp_dir a ese directorio. Esto no funcionará si se genera un nuevo proceso php para cada solicitud.

Answer 3

Mueva sus archivos a un directorio de usuario después de que se hayan cargado. Esos archivos temporales deben eliminarse.

Answer 4

Parece que algo está muy mal con la instalación de PHP o con cualquier llamada al sistema que PHP utiliza internamente para generar los nombres de archivo aleatorios (lo más probable es tempnam).

Para todos los demás: PHP maneja los archivos cargados internamente antes de que se procese el código de usuario. Estos nombres se almacenan en $_FILES['file']['tmp_name'] (donde 'archivo' es el nombre (entre comillas) del elemento de entrada de archivo en el formulario).

Answer 5

Después de perseguir el código correspondiente a _gettemp en la implementación de libc de FreeBSD 7, estoy claro con respecto a cómo el contenido del archivo tmp_name podría ser válido. (Para rastrearlo, puede descargar una copia de PHP 5.2.8 y leer en main/rfc1867.c - llamadas a la línea 1018 en main/php_open_temporary_file.c, la función comienza en la línea 227, que hace su trabajo principal en la función que comienza en la línea 97, que, sin embargo, es esencialmente solo un contenedor para mkstemp en su sistema, que se encuentra en el FreeBSD libc implementation en la línea 66 (vinculado), que usa _gettemp (igual que el anterior) para generar el nombre de archivo aleatorio. Sin embargo, the manpage for mkstemp menciona en la sección BUGS que arc4random() function no reentrante. Es podría ser una posibilidad de que 2 solicitudes simultáneas estén ingresando a la sección del código crítico y devolver el mismo tmp_name - Sé muy poco acerca de cómo funciona Apache con mod_php o php-cgi para comentar allí (aunque usando FastCGI/php -cgi podría funcionar - No puedo comentar con éxito sobre esto en este momento).

Sin embargo, apuntando a la solución más simple, si no está experimentando el archivo tmp_name que no es válido, pero colisionando con otros archivos cargados (por ejemplo, si usa la parte de nombre de archivo de tmp_name como su única fuente de exclusividad en el nombre de archivo almacenado), podría enfrentar colisiones debido al birthday paradox. En another question mencionas tener unos 5,000,000 de archivos para mover, y en still another question mencionas recibir 30-40k cargas por día. Esto me parece una situación primordial para una colisión de paradojas de cumpleaños. El mktemp man page menciona que (si usa seis 'X' como lo hace PHP) hay 56,800,235,584 nombres de archivo posibles (62 ** 6, o 62 ** n donde n = número de 'X', etc.). Sin embargo, dado que tiene más de 5 millones de archivos, la probabilidad de una colisión es approximately 100% (otra heurística sugiere que ya habrá experimentado un pedido de 220 colisiones, si ((archivos * (archivos-1))/2)/(62 ** 6) significa cualquier cosa, donde archivos = 5,000,000). Si este es el problema al que se enfrenta (probable, si no agrega más entropía al nombre de archivo cargado generado), puede intentar algo como move_uploaded_file($file['tmp_name'], UPLOADS.sha1(mt_rand().$file['tmp_name']).strrchr($file['name'], '.')), con la idea de agregar más aleatoriedad al nombre de archivo aleatorio, evitando colisiones. Una alternativa podría ser agregar dos más 'X' a la línea 134 de main/php_open_temporary_file.c y volver a compilar.