readfile() manda immagini danneggiate

Buongiorno, vi posto il codice che ho usato:

Codice PHP:

---

<?php
$directory = "foto/";
$filecount = 0;
$files = glob($directory . "*");
foreach($files as $valore){
$dimensioni = getimagesize($immagine); //cose aggiuntive
$tot = (500 * $dimensioni[1])/$dimensioni[0]; //cose aggiuntive
echo "<input type='image' src='$valore' height='$tot'.'px' width='500px' value='$valore' name='foto'>";
echo "<br/>";
}
if($_POST['foto']){
$filenome=explode('/', $_POST['foto']);
echo $filenome[1];
header("Content-Type: image/png");
header("Content-Disposition:attachment;filename=".basename($filenome[1]));
readfile($filenome[1]);
}
?>

---

In pratica devo contare quante immagini ci sono in una data cartella, e poi stamparle, una sotto l'altra, sotto forma di bottoni immagini.
Quando premo sul bottone mi deve scaricare l'immagine corrispondente.
Fin qui tutto bene, l'unico problema è che quando provo ad aprire un file appena scaricato il visualizzatore foto di windows mi dice:
Impossibile aprire l'immagine. File probabilmente danneggiato o troppo grande.

Cercando su google ho visto che dicevano di togliere tutti gli spazi tra <?php e readfile(), ho provato ma niente...
Oppure dicevano di togliere ?> alla fine, ma essendo che il mio codice è tra altro codice html non lo posso fare.

Ti faccio notare che prima di specificare un header, non può esserci alcun output.
Quindi già qui sorge un problema:

Codice PHP:

---

echo $filenome[1];
header("Content-Type: image/png");


---

Questo è dovuto alla struttura di una richiesta/risposta HTTP (prima c'è l'intestazione, poi il corpo).

Inoltre, le immagini all'interno di foto/ sono in png?
Ti consiglio di stare attento al valore di $_POST['foto'] perché potrebbe consentire ad utenti malevoli di scaricare qualsiasi file del tuo sito. Quindi filtra bene il suo contenuto prima di utilizzarlo, rimuovi .. per salire di una directory, fai in modo che il suo valore sia imprigionato nella cartella in cui si colloca.

Alcune cose:

• Assicurati di NON scrivere nient'altro mentre stampi la foto con la readfile: qualsiasi carattere in più viene incluso nell'immagine, compromettendola. Quindi niente foreach() iniziale, niente echo dell'immagine, niente caratteri prima di <?php, niente form...
• Devi essere sicuro che $filenome[1] sia il percorso all'immagine: secondo me potevi usare direttamente $_POST['foto'] senza explode (anche se comunque poi dovrai controllare se si tratta di un file che vuoi che sia scaricato). Se però vuoi mettere quel codice, devi farlo precedere da $directory. Inoltre, una controllata con la file_exists() per essere sicuro che il file esista, la farei.

Questo dovrebbe andare:

Codice PHP:

---

<?php
$directory = "foto/";
$filecount = 0;
$files = glob($directory . "*");

if($_POST['foto']){
$filenome=explode('/', $_POST['foto']);
//echo $filenome[1];
$filenome[1] = $directory.$filenome[1];
if(!file_exists($filenome[1]))
echo 'Il file '.$filenome[1].' non esiste!';
else
{
header("Content-Type: image/png");
header("Content-Disposition:attachment;filename=".$filenome[1]);
readfile($filenome[1]);
die();
}
}

foreach($files as $valore){
$dimensioni = getimagesize($immagine); //cose aggiuntive
$tot = (500 * $dimensioni[1])/$dimensioni[0]; //cose aggiuntive
echo "<input type='image' src='$valore' height='$tot'.'px' width='500px' value='$valore' name='foto'>";
echo "<br/>";
}
?>

---

EDIT: anticipato.

Ciao!

@alemoppo
Grazie mille, ho provato il tuo codice e funziona!

@javascripter
Hai ragione, l'echo era una cosa che avevo messo per vedere cosa conteneva $filenome, e poi mi sono scordato di cancellarlo...
Con cosa potrei sostituire $_POST['foto'] ? Di solito usavo mysqli_real_escape_string solo per gli input di testo, non pensavo ci fossero problemi anche sui pulsanti :???: