include;(

[Pencil]

Original geschrieben von boom
Ich bin mir 100%ig sicher das nicht PHP den Banner verschiebt, sondern HTML ...

naja das ist ja jetzt auch geklärt, da ich jetzt div align genutzt hab

aso in einem anderen board wurde folgendes zu deinem code geschrieben:

Arrrg! Lösch das! So darfst Du das nie nie niemals machen!
Es gibt im HackIt-Forum ein Beispiel, das Deinem sehr ähnlich ist!

Dein Skript hat eine fiese Sicherheitslücke, weil die Variable 'site' ganz einfach verändert werden kann. Wenn man dort "böses" hineinschreibt, passieren unter Umständen ärgerliche Sachen - auch wenn ich bislang nicht sehe, daß man bei Dir etwas richtig Schlimmes machen könnte - Nähere Details nenne ich mal nicht... - Eine Sicherheitslücke ist das allemal.

Vorschlag:
1. Speichere ein Array, wo zu jeder zu verlinkenden Seite eine ID zugeordnet wird.
2. In der Variable site steht dann nur noch eine Zahl.
3. Diese Zahl ist dann der Index im Array, wo die einzubindende Seite steht.

Bei weiteren Fragen melde Dich am besten per PM

hope you help

 

[THE REAL TOOLKIT]

Naja lol,man könnte dadurch zb. eine Admin Datei oder so includen...
:RTFM:

 

[socKe|bla]

oder tipp mal index als site ein, damit kannst dem webserver gut einheizen, weil der dann in einer endlos-schleife läuft. mach es am besten mit switch. hier mal ein beispiel..

switch($_GET['site']){
  case "news":
    include ("news.php");
    break;
  case "rulez":
    include ("rulez.php");
    break;
  default:
    //Hier schreibst du hin, was der machen soll, wenn kein "case" eintritt
}

http://www.php.net/manual/de/control-structures.switch.php