Uitslag prijsvraag workshop veilig programmeren
Zelfs voor lezers van Security.nl valt het niet mee om veilig te programmeren, zo blijkt uit de uitslag van de prijsvraag. Deelnemers konden een workshop veilig programmeren winnen door de fouten in deze PHP code te ontdekken en op te sturen. De nummer één wist elf problemen te vinden van de meer dan 20 die aanwezig waren. Voor iedereen die benieuwd is naar de oplossingen staan hier de antwoorden. De winnaar wensen we een leerzame workshop toe.
"Er ontbreekt controle op de omvang van het geuploade bestand." - Je moet in PHP de max upload size opgeven, dus het is niet zo dat er geen limiet aan zit. Wederom is het wellicht netter om dat in de code te doen.. ook omdat je voor een avatar geen 10 MB ruimte nodig hebt. Maar ik zie het niet als een security probleem.
"Er moet een controle worden gedaan op het type bestand". Tjah, is netter, maar het is wederom geen beveiligingsprobleem. Zolang de webserver bij het sturen van de avatar file naar de browser het type maar op jpeg ofzo zet.. Dan kan je op webpagina hooguit garbage op je scherm krijgen.
"zodat aanvallers geen geen interne informatie over het systeem krijgen". Zowieso moet je op een production server de errors en warnings niet naar de browser, maar naar een file sturen. Het geniet slechts de voorkeur om goed return values te controleren. Maar wederom niet een nijpend security probleem.
De reden staat er toch bij? "anders dient de gehele code te worden beschermd alsof het een wachtwoord is". Zonder wachtwoorden erin zou je de sourcecode nog eens aan iemand kunnen geven zonder gelijk secrets prijs te geven.
"Er moet een controle worden gedaan op het type bestand". Tjah, is netter, maar het is wederom geen beveiligingsprobleem. Zolang de webserver bij het sturen van de avatar file naar de browser het type maar op jpeg ofzo zet.. Dan kan je op webpagina hooguit garbage op je scherm krijgen.
Assumption is the mother of all fuckups! Niet op de webserver gaan vertrouwen, beveiliging hoort op alle lagen thuis. Programmeurs en beheerders communiceren vaak niet voldoende om problemen te voorkomen, waarom het dus niet in de code oplossen?
"zodat aanvallers geen geen interne informatie over het systeem krijgen". Zowieso moet je op een production server de errors en warnings niet naar de browser, maar naar een file sturen. Het geniet slechts de voorkeur om goed return values te controleren. Maar wederom niet een nijpend security probleem.
Information gathering is 1 van de eerste dingen die een hacker doet, het is dus belangrijk om geen info weg te geven. Bovendien laat je daarmee zien dat je er over hebt nagedacht en gaat ie misschien gelijk de buren hacken.
"Er moet een controle worden gedaan op het type bestand". Tjah, is netter, maar het is wederom geen beveiligingsprobleem. Zolang de webserver bij het sturen van de avatar file naar de browser het type maar op jpeg ofzo zet.. Dan kan je op webpagina hooguit garbage op je scherm krijgen.
En als ik nu eens een .php bestand opstuur met daarin allerlei vervelende code? De webserver gaat dat echt wel netjes uitvoeren, alvorens de uitvoer van het script (met wat voor mooie content-type headers dan ook), naar de browser te sturen.
Deze posting is gelocked. Reageren is niet meer mogelijk.
Junior DevOps Engineer
Certified Secure is op zoek naar een Junior DevOps Engineer. Deze functie is een stuk interessanter dan de term doet vermoeden! Om jou als potentiële nieuwe collega meteen te laten zien wat we doen hebben we speciaal voor jou een selectie gemaakt van een aantal leuke security challenges. Are you ready for a challenge?
Cybersecurity Trainer / Streamer
Toe aan een nieuwe nieuwe job waarmee je het verschil maakt? In de Certified Secure trainingen laat je deelnemers zien hoe actuele kwetsbaarheden structureel verholpen worden. Ook werk je actief mee aan de ontwikkeling van onze gamified security challenges. Bij het maken van challenges kom je telkens nieuwe technieken tegen, van Flutter tot GraphQL, van Elastic Search tot Kubernetes.
