Download Script
Door C. Steenbrink, 21 jaar geleden, 8.854x bekeken
Dit script heb ik in elkaar gezet door middel van deze site en php.net en wat eigen inbreng.
Als jullie tips, opmerkingen, verbeteringen hebben post ze dan ff dan kan ik het verranderen en eventueel er iets van leren.
Het css bestand heb ik niet inbegrepen.
Voorbeeld: http://www.devilsquare.eu/downloadscript/
Gesponsorde koppelingen
PHP script bestanden
Er zijn 12 reacties op 'Download script'
0 
-0 
-0
Ziet er zo netjes uit. Misschien even een voorbeeldje.
Daarnaast zag ik zo even snel dat je op regel 4 controleert of $_GET['download'] bestaat, maar op regel 8 gebruik je ook nog een $_GET['p'], die je niet controleert..
Daarnaast zag ik zo even snel dat je op regel 4 controleert of $_GET['download'] bestaat, maar op regel 8 gebruik je ook nog een $_GET['p'], die je niet controleert..
0 -0
-0
Offtopic: misschien even rond je hele code, dan wordt ie iets overzichtelijker en makkelijker te kopieren ;)
ps. Ook het gebruik van shorttags is niet aan te raden.
Dit staat lang niet op elke server aan namelijk.
ps. Ook het gebruik van shorttags is niet aan te raden.
Code (php)
1
<?= ... ?>
0 -0
-0
miss kan je dit:
naar zo iets veranderen?
(niet getest)
Code (php)
1
2
3
4
5
6
7
8
9
10
2
3
4
5
6
7
8
9
10
<?php
$page = $_GET['p'];
$page = str_replace("../", "", $page);
$page = str_replace("./", "", $page);
$page = str_replace("/..", "", $page);
$page = str_replace("/../", "", $page);
$page = str_replace(".", "", $page);
$page = str_replace(chr(92), "", $page);
$page = str_replace("?", "", $page);
?>
$page = $_GET['p'];
$page = str_replace("../", "", $page);
$page = str_replace("./", "", $page);
$page = str_replace("/..", "", $page);
$page = str_replace("/../", "", $page);
$page = str_replace(".", "", $page);
$page = str_replace(chr(92), "", $page);
$page = str_replace("?", "", $page);
?>
naar zo iets veranderen?
Code (php)
1
2
3
2
3
<?php
$page = str_replace (array ("../", "./", "/..", "/../", ".", chr(92), "?"), "", $_GET['p']);
?>
$page = str_replace (array ("../", "./", "/..", "/../", ".", chr(92), "?"), "", $_GET['p']);
?>
(niet getest)
0 -0
-0
kheb nu het een en ander aangepast misschien nog meer tips suggestisch?
@PHP NEWBIE ( regel 4 = 11 en 8 = 15 geworden na edit ) waarom zou ik regel 15 moeten controleren alles word toch gecontroleert door: file_exists()? Als de download-link niet kloptvanwege foute $_GET['p'] krijg je automatisch:
"File does not exist"
Of klopt dat misschien niet wat ik zeg?
@PHP NEWBIE ( regel 4 = 11 en 8 = 15 geworden na edit ) waarom zou ik regel 15 moeten controleren alles word toch gecontroleert door: file_exists()? Als de download-link niet kloptvanwege foute $_GET['p'] krijg je automatisch:
"File does not exist"
Of klopt dat misschien niet wat ik zeg?
0 -0
-0
Volgens mij krijg je door dit in je .htaccess bestand te zetten van de download map het zelfde effect,
Options +Indexes
Options +Indexes
0 -0
-0
http://www.devilsquare.eu/downloadscript/index.php?p=///../..///.//
lekker dan. zo lek als een mandje
lekker dan. zo lek als een mandje
0 -0
-0
Zoals Sander boven me al zij, zo lek als een mandje.
Het zou slim zijn om het script er nu meteen af te halen. Aangezien iedereen al je pagina's kan downloaden. :]
Het zou slim zijn om het script er nu meteen af te halen. Aangezien iedereen al je pagina's kan downloaden. :]
0 -0
-0
Als je het veiliger wil maken , creer je een dynamische array en maak je de links in de trand van: http://www.devilsquare.eu/downloadscript/index.php?p=&downloadID=een_unieke_waarde_uit_array
0 -0
-0
Als je nu eens dit er nog eens boven zet.
Dan heb je bijna alle problemen verholpen! en dan is die niet zo lek meer.
Gr Haneev
Code (php)
1
2
3
4
5
6
7
8
9
10
11
2
3
4
5
6
7
8
9
10
11
<?PHP
$rootmap = $_GET['p'];
$rootmap = str_replace("../", "", $rootmap);
$rootmap = str_replace("./", "", $rootmap);
$rootmap = str_replace("/..", "", $rootmap);
$rootmap = str_replace("/../", "", $rootmap);
$rootmap = str_replace(".", "", $rootmap);
$rootmap = str_replace(chr(92), "", $rootmap);
$rootmap = str_replace("?", "", $rootmap);
$_GET['p'] = $rootmap;
?>
$rootmap = $_GET['p'];
$rootmap = str_replace("../", "", $rootmap);
$rootmap = str_replace("./", "", $rootmap);
$rootmap = str_replace("/..", "", $rootmap);
$rootmap = str_replace("/../", "", $rootmap);
$rootmap = str_replace(".", "", $rootmap);
$rootmap = str_replace(chr(92), "", $rootmap);
$rootmap = str_replace("?", "", $rootmap);
$_GET['p'] = $rootmap;
?>
Dan heb je bijna alle problemen verholpen! en dan is die niet zo lek meer.
Gr Haneev
0 -0
-0
kan iemand me dingen leren over scripts enz. ?? ik wil graag leren hoe je bjivoorbeel op een spel zo gezegd kan '' cheaten''
mijn msn: [email protected]
bedankt,
mijn msn: [email protected]
bedankt,
0 -0
-0
Ik heb het script gedownload en zeer kleine aanpassingen aangebracht naar mijn wens....
Alles gaat goed, maar zodra het bestand op de computer download is, klopt de bestandsindeling niet meer, is er iemand die dit probleem ook heeft gehad, of weet hoe ik dit herstel.
gr,
Alles gaat goed, maar zodra het bestand op de computer download is, klopt de bestandsindeling niet meer, is er iemand die dit probleem ook heeft gehad, of weet hoe ik dit herstel.
gr,
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu