scusami ma non ho capito...

a parte che nel form ti conviene mettere un if iniziale per verificare se il form è stato inviato, altrimenti parte la pagina con un errore....

cmq a parte questo,

tu qui

Codice PHP:
$img $valore[0];         
$attach $valore[0]; 
passi due volte la stessa variabile, è normale che img e attach siano la stessa cosa ad ogni giro..

se fai un ciclo foreach, lui la prima volta analizza il primo file, e ti dice due volte la stessa cosa (perchè tu gli hai detto che img e attach sono uguali), poi fa il secondo giro e ti resituisce gli stessi valori (però per il secondo file)