Problemer med sikkerhedshuller - Injection af Include funktionen

<?php if(isset($_GET['goto']))
            { include($_GET['goto']); }
            else { include("news/index.php"); }; ?>

Kan du skrive lidt mere sikkert om til:

<?php

    $root = realpath( "." ) . "/";
    $goto = ( !empty( $_GET['goto'] ) ? $_GET['goto'] : "news/index.php" );
    $goto = realpath( $root . $goto );
    if ( !is_file( $goto ) || substr( $goto, 0, strlen( $root ) ) != $root
        || $goto == __FILE__ )
    {
        $goto = 'news/index.php';
    }

    include( $goto );

?>

Du kan øge sikkerheden yderligere ved ikke at angive .php i URL'en, men tilføje den i scriptet sådan:

<?php

    $root = realpath( "." ) . "/";
    $goto = ( !empty( $_GET['goto'] ) ? $_GET['goto'] : "news/index.php" );
    $goto = realpath( $root . $goto . ".php" );
    if ( !is_file( $goto ) || substr( $goto, 0, strlen( $root ) ) != $root
        || $goto == __FILE__ )
    {
        $goto = 'news/index.php';
    }

    include( $goto );

?>

Jeg kan se du bruger $_GET, måske er det helt ok for dig at slå register_globals fra, prøv det og se om det gør en forskel... Langt de fleste PHP4-scripts kører fint i PHP5! Igen, prøv det og se :)

Det du gør svarer til at lade din pung ligge fremme på en bænk i parken hele dagen. Når du henter den om aftenen, så er der nok ikke så meget tilbage. Det er et kendt og udnyttet hul, og noget man bare ikke gør.

Har du fået koden, eller hjælp til koden, et sted, så vil jeg anbefale dig aldrig at få hjælp derfra igen.

Du skal huske at slette alle filer i alle kataloger, da der sagtens kan være lagt en bagdør på serveren, som du ikke lige lægger mærke til.

En løsning, der er knap så avanceret, men ganske sikker, og nem at forstå, er denne:

1) Links ser ud som    dinside.php?id=4

2) Din kode ser i alt væsentligt ud som

    $id='0';
    if (isset($_GET['id'])) $id = $_GET['id'];
    switch ($id) {
      case '1':  include("sidenummer1.php"); break;
      case '2':  include("sidenummer2.php"); break;
      case '3':  include("sidenummer3.php"); break;
      case '4':  include("sidenummer4.php"); break;
      default:    include('forside.php');
  }