Archief - PHP: Include weigert te werken

a

azkme

Legacy Member

• 11 mrt 2006

• #1

Hey,

kheb mezelf een site gemaakt en ik wil graag werken met een include zodat ik 1 pagina heb met alle menutjes en de inhoud van de verschillende pagina's er dan zo in verwerk.. ik ben zo te werk gegaan:

ik heb op de plaats waar de include moet komen dit gezet

Code:

<?
if ($page) {
 include("$page.php");
} else {
 include("news.php");
}
?>

en men links in het menutje heb ik dan zo gelinkt

Code:

<a href="?page=rules">rules</a>

blijkbaar doet hij dit niet, terwijl ik me toch voorstel dat het vroeger WEL werkte .. hij gaat steeds naar news.php, zelfs al wil ik naar rules.php gaan .. heb een tijdje niet veel met zo'n zaken bezig geweest, dus heb ik misschien één of andere update gemist? of wie ziet het foutje?

de website kan gevonden worden op http://ukb.electro-vets.be, alvast bedankt!

B

Bunz

Legacy Member

• 11 mrt 2006

• #2

lol ge doet nergens ne get ?

$page = $_GET['page'];

a

azkme

Legacy Member

• 11 mrt 2006

• #3

oiii vergeten hartelijk dank

d

dJeez

Legacy Member

• 11 mrt 2006

• #4

Nu heb je een veiligheidsgat gecreëerd dat zo groot is dat er een vrachtwagen doorkan. En zo te zien heeft iemand er ook al misbruik van gemaakt.

J

J-Style

Legacy Member

• 11 mrt 2006

• #5

Inderdaad.

Iedereen kan zomaar pagina's includen..
http://ukb.electro-vets.be/?page=http://pornosites.bijvoorbeeld.com/index

a

azkme

Legacy Member

• 12 mrt 2006

• #6

J-Style zei:

Inderdaad.

Iedereen kan zomaar pagina's includen..
http://ukb.electro-vets.be/?page=http://pornosites.bijvoorbeeld.com/index

Klik om te vergroten...

ehm, waarom is dit een veiligheidslek? ik ben geen php specialist, dus wat doet het en wat kan ik eraan doen?

S

Squall-sX-

Legacy Member

• 12 mrt 2006

• #7

Het is zo mogelijk om externe scripts te includen.
https://www.beyondgaming.be
Dees laadt bijvoorbeeld de homepage van telenet in. Zo kun je ook een script schrijven dat alle bestanden op je server ergens anders zet (om mogelijke hardcoded passwords te verkrijgen), of om deze allemaal gewoon te verwijderen en er andere te plaatsen.

F

Fuji1337

Legacy Member

• 12 mrt 2006

• #8

Kzou et zo doe:

PHP:

<?
switch ($_GET['page']){
case "news": include("inc/news.inc.php"); break;
case "page": include("inc/page.inc.php"); break;
enz
enz
enz
default: include("inc/news.inc.php"); break; 
}
?>

Die inc hoeft niet, maakt het enkel veel overzichtelijker

S

Squall-sX-

Legacy Member

• 12 mrt 2006

• #9

Ik zou de pagina's gewoon definiëren in een tabel.

id | pagina
1 | news.php
2 | page.php
...

en dan doe je index.php?page=1 enz.
Dan kun je ook nog indien je wilt een kolom login_required toevoegen, en controle toevoegen om te zien of iemand ingelogd is of niet en deze pagina mag zien, of niet.

a

azkme

Legacy Member

• 12 mrt 2006

• #10

ik ga het snel eens proberen Fuji1337 op jou manier, alvast bedankt

edit: ok, met die code is het me gelukt, nu kan je geen page's meer includen thanks

F

Fuji1337

Legacy Member

• 13 mrt 2006

• #11

Your welcome Zodra de site klaar is laat je hem maar eens zien