Problem z pobieraniem mp3 z serwera Opcje

[Kiubus]

Witam.
Mam na swoim serwerze pliki w formacie MP3. Chciałbym, aby była możliwość ich pobrania. Próbowałem zrobić po prostu w HTML (a href...) ale to otwiera nowe okienko. Znalazłem na internecie skrypt download.php:

[PHP] pobierz, plaintext 
<?php
if (isset($_GET['file']) && substr($_GET['file'], -3)=='mp3') {
	header('Content-type: application/x-download');
	header('Content-Disposition: filename="'.$_GET['file'].'"');
	readfile($_GET['file'] ); 
}
?>
[PHP] pobierz, plaintext 

Po jego użyciu nakazano stosować adres do hrefa w stylu:

[HTML] pobierz, plaintext 
<a href="download.php?file=plik.mp3">
[HTML] pobierz, plaintext 

Ogólnie to działa, tylko, że na 4 linki pobiera tylko 1 piosenkę do odsłuchu, resztę ściąga rozmiarze np. 23 bajtów gdzie utwór ma rozmiar 2,29 MB. Czy ktoś może mi pomóc?
Pozdrawiam!

[by_ikar]

Jest podatny na atak directory traversal, najprościej można zabezpieczyć taki skrypt poprzez basename który zawsze zwróci tylko nazwę pliku, zamiast pełnej ścieżki, dzięki czemu nikt nie będzie mógł się "wydostać" poza katalog w którym będziesz miał pliki do pobrania. Bo w ten sposób można pobrać cały twój kod ze strony, łącznie z konfiguracją (np dostępem do bazy danych).

[PHP] pobierz, plaintext 
<?php
 
if(isset($_GET['file']) && substr($_GET['file'], -3)=='mp3')
{
    $file = basename($_GET['file']);
 
    header('Content-Type: application/octet-stream');
    header('Content-Transfer-Encoding: Binary'); 
    header('Content-disposition: attachment; filename="'.$file.'"'); 
    header('Content-Length: '.filesize($file));
 
    readfile($file);
}
[PHP] pobierz, plaintext