🚀 go-pugleaf

Op Sun, 07 Nov 2004 16:05:46 +0000, schreef JeroenV:

> Cajmere wrote:
> 
>>>> if (isset($_GET['verplaats'])){
>>>>$filemove�=�$uploadDir.$_GET['verplaats'];
>>>>$movetoDir�=�$uploadDir.$_REQUEST['toDir']."/";
>>> 
>>> Je snapt dat dit stukje code *extreem insecure* is?
>> 
>> ff vraagje. Waarom?
>> is dit omdat er geen controle is op wat er verplaatst wordt?
>> kwestie van iets bij te leren :-).
> 
> Inderdaad. Met een beetje creatieve parameters door te geven aan het
> PHP-script kan je elk bestand op de server naar elke directory verplaatsen.
> 
> Een gouden regel is: vertrouw nooit op wat je binnenkrijgt van de gebruiker.
> Check alles en zorg dat je zeker bent dat je enkel en alleen binnenkrijgt
> wat je verwacht.

dus

in mijn uploadscriptje voor foto's, doe ik controle op de inhoud.
kijk welke extensies ze hebben, ik laat bv elk jpg's toe(dmv
getimagesize() ). dat ook nog bepaalde voorwaarden heeft(grootte bestand &
pic) is aanvaardbaar of kan je dit nog verder drijven? door bv
na uploaden misschien permissies te veranderen? voor zover dit invloed
heeft(dunno of er bij upload een vaste permissie gegeven wordt, of dit de
permissie krijg van het bestand dat op de pc zelf staat).

-- 
Mayday - The Religion,
In music & dance I express myself as a member of a higher community....
I have forgotten how to walk and speak and I am towards flying in the air.
Raving.