Dat heb je goed gezien: MySQL injection is nu mogelijk. Je kunt dit op drie manieren oplossen:
- Plaats slashes in de soort (addslashes). En als de soort niet bestaat fout weergeven.
- Eigenlijk hetzelfde idee als de vorige oplossing, maar dan netter: Kijk of de soort enkel letters bevat (preg_match), zo niet, fout, anders kijken of het spel bestaat.
- Laatste mogelijkheid, zet de bestaande spellen in een array, en kijk of het opgegeven spel in die array staat (in_array). Zo niet: Spel bestaat niet, anders wel.

Als je nog meer vragen hebt over de oplossingen, vraag maar raak!

ben jij maf112 van de v3 ?

Nee, htmlspecialchars is ten eerste niet nodig.

Je moet eerst de string addslashen (Zoals je hierboven doet), en vervolgens in de query gebruiken. Vervolgens controleer je met mysql_num_rows of er resultaten zijn. Zo niet: Fout weergeven.

Dat wil ik wel even doen.. Maar dan moet je eigenlijk even ergens het script posten zonder die achterlijke slashes.. Zo is het een beetje lastig.;)

<html>
<head>
<title><?php echo $page->sitetitle; ?></title>
</head>
<body>
<?php


// basis voor elk script
include("_include-config.php");
if(! check_login()) {
    header("Location: login.php");
    exit;
  }


// Codes voor het ophalen van de game
$sSoort	=	$_GET['soort'];
//Kijken of de soort alleen maar uit letters bestaat, zo niet, error.
if(!preg_match("#^[A-Z]*$#is",$sSoort)) {
	print "Opgegeven spel bestaat niet.";
} else {
	$dbres		=	mysql_query("SELECT * FROM `games` WHERE `sitenaam`='{$_GET['soort']}'");
	if(mysql_num_rows($dbres) != 1) {
		print "Opgegeven spel bestaat niet.";
	} else {
		while($cms = mysql_fetch_object($dbres)){
			?>
			<link rel="stylesheet" type="text/css" href="<? echo $sitelink;?>/layout/layout<?php echo $page->layout; ?>/css/css.css"> 
			<table align="center" width=100%>
			<tr><td class="subTitle"><b><center><?=$cms->sitenaam;?></center></b></td></tr>
			<tr><td class="mainTxt">
			<center><?=$cms->flash;?></center>
			</td></tr></table>
			<?php
		}
	}
}


?>
</body>
</html>

Zie mijn commentaar voor meer informatie..