Webwereld

Is dit veilig?

Gestart doorMarvinn5 reacties51 weergaven
Log in om mee te praten

6 berichten · pagina 1 van 1

Marvinn

Lid · Topicstarter

Berichten
2.609
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Hoi.

Ik heb gebruik een index?p= systeem.

Alleen is dit veilig:
Spoiler tonen
[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000][/COLOR][COLOR= #007700]])) {

    


}[/COLOR][URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]p[/COLOR][COLOR= #007700]])) {
    
function[/COLOR] Checklogin[COLOR= #007700]([/COLOR][COLOR= #0000BB]$url[/COLOR][COLOR= #007700]) {[/COLOR]
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_SESSION[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]id[/COLOR][COLOR= #007700]])) {[/COLOR]
        [URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$url[/COLOR][COLOR= #007700]);
    }[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"Je moet ingelogd zijn voor deze pagina."[/COLOR][COLOR= #007700];
    }
}[/COLOR]

    [URL="http://www.php.net/switch"]switch[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]p[/COLOR][COLOR= #007700]]) {[/COLOR]
        [COLOR= #FF8000]// admin //[/COLOR]
        [COLOR= #007700]case[/COLOR] [COLOR= #DD0000]"profiel2"[/COLOR][COLOR= #007700]:[/COLOR]
            [URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]pagina/profiel.php[/COLOR][COLOR= #007700]);[/COLOR]
            [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        default:[/COLOR]

[URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
case[/COLOR] [COLOR= #DD0000]"paardenbug"[/COLOR][COLOR= #007700]:[/COLOR]
[URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]pagina/dossier_paard.php[/COLOR][COLOR= #007700]);[/COLOR]
[URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
                                    
        default:[/COLOR]
            [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]p[/COLOR][COLOR= #007700]])) {[/COLOR]
                [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/file_exists"]file_exists[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]pagina/[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]p[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000].php[/COLOR][COLOR= #007700])) {[/COLOR]
                    [URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]pagina/[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]p[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000].php[/COLOR][COLOR= #007700]);
                }[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR]
                    [URL="http://www.php.net/echo"]echo[/URL][COLOR= #DD0000]Sorry, Deze pagina bestaat niet of werkt niet meer![/COLOR][COLOR= #007700]; 
                }
            }[/COLOR]
            [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        }
}[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR]
    [URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]pagina/home.php[/COLOR][COLOR= #007700]);
}[/COLOR]
?>
Mocht er een lek inzitten zou iemand hem kunnen fixen biggrin
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
filepath traveling

?p=../index

oplossen mag je zelf doen/
patch staat erin bij eeros,
0
Marvinn

Lid · Topicstarter

Berichten
2.609
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Van:

if(file_exists(pagina/../.$_GET[p]..php)) {
                    include(pagina/../.$_GET[p]..php);


Naar:

if(file_exists(pagina/.$_GET[p]..php)) {
                    include(pagina/.$_GET[p]..php);


Zo biggrin?
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
if (file_exists(pagina/ . $_GET[p] . .php))
    {
        
        include(pagina/ . str_replace(../, , $_GET[p]) . .php);
0
Marvinn

Lid · Topicstarter

Berichten
2.609
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Dankjewel biggrin
0
Antonn

Lid

Berichten
8.649
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Smiegles schreef:
if (file_exists(pagina/ . $_GET[p] . .php))
    {
        
        include(pagina/ . str_replace(../, , $_GET[p]) . .php);
Met ./ werkt het geloof ik ook. Je kan beter enkel de slashes vervangen door "niks" wink
Het aller veiligste is elke pagina in een switch te zetten, en als die daar niet in voorkomt de home includen. Als je bijvoorbeeld in de map pagina een bestand hebt staan dat niet hoort geïnclude te worden omdat er bijvoorbeeld opnieuw een sessie in gemaakt word kan dit dus wel en kunnen deze dingen rare errors creëren enz...
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren