PHP Session_start

Oke dit is mijn post hij is niet goed maar dat is waarschijnlijk het probleem.
Deze moet het bericht posten, het ip van degene die post en de gebruikersnaam van de poster en de id van het bericht.

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/require"]require[/URL] [COLOR= #DD0000]connect.php[/COLOR][COLOR= #007700];[/COLOR]
[COLOR= #0000BB]$sql[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT *
FROM  `berichtenbalk` ORDER BY id DESC"[/COLOR][COLOR= #007700]);[/COLOR]
[COLOR= #0000BB]$row[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_fetch_array"]mysql_fetch_array[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$sql[/COLOR][COLOR= #007700]);[/COLOR]
[COLOR= #0000BB]$id[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]$row[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]id[/COLOR][COLOR= #007700]];[/COLOR]
[COLOR= #0000BB]$idp[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]$id[/COLOR] [COLOR= #007700]+[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700];[/COLOR]
[URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"INSERT INTO berichtenbalk (berichten, poster, ip, id) VALUES("[/COLOR] [COLOR= #007700].[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]bericht[/COLOR][COLOR= #007700]] .[/COLOR][COLOR= #DD0000]", "[/COLOR] [COLOR= #007700].[/COLOR] [COLOR= #0000BB]$_SESSION[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]gebruikersnaam[/COLOR][COLOR= #007700]] .[/COLOR][COLOR= #DD0000]","[/COLOR] [COLOR= #007700].[/COLOR] [COLOR= #0000BB]$_SERVER[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]"REMOTE_ADDR"[/COLOR][COLOR= #007700]] .[/COLOR][COLOR= #DD0000]", $idp )"[/COLOR][COLOR= #007700]);[/COLOR]
[URL="http://www.php.net/header"]header[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]Location: ./berichtenbalk.php[/COLOR][COLOR= #007700]);[/COLOR]
?>

Dit is de berichtenbalk + waar het bericht vanaf word gestuurt:
Deze moet alle berichten echoën, Hij echoëd alleen het eerste bericht help me

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/require"]require[/URL] [COLOR= #DD0000]connect.php[/COLOR][COLOR= #007700];[/COLOR]
[COLOR= #0000BB]$sql[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT *
FROM  `berichtenbalk` ORDER BY id DESC"[/COLOR][COLOR= #007700]);[/COLOR]
[COLOR= #0000BB]$row[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_fetch_array"]mysql_fetch_array[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$sql[/COLOR][COLOR= #007700]);
}[/COLOR]
?>
<table border="1" width=500>
<tr>
  <td><marquee direction="left"  hspace=40><b>Berichten: </b>[COLOR= #0000BB]<?php[/COLOR] [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #0000BB]$row[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]berichten[/COLOR][COLOR= #007700]][/COLOR] ?></marquee></td>
</tr>
</table>

<form name="input" action="berichtenbalk_post.php">
<input type=text name="bericht" />
<input type="submit" value="Plaats een bericht" />
</form>
</html>

Er klopt inderdaad een aantal dingen niet:
- Ik mis foutafhandeling
- Waarom gebruik je ==, terwijl je iets wilt toekennen, dan gebruik je=.
- je moet while gebruiken. Kijk eens naar de eerste beste PHP-tutorial
- SQL injection is mogelijk

Sql injectie is niet mogelijk Wat bedoel je met die while ik begrijp hem even niet. Die == klopt Ik wil het liever eerst helemaal hebben werken dan pas de fouten doen.

Goededag schreef:

Sql injectie is niet mogelijk Wat bedoel je met die while ik begrijp hem even niet. Die == klopt Ik wil het liever eerst helemaal hebben werken dan pas de fouten doen.

Ik zie anders wel een onbeveiligde $_GET in je query.

De $_get en $_post worden allemaal gefilterd via de connect.php Toch bedankt voor het waarschuwen.

Dan hoop ik wel dat het $_GET, $_POST en $_COOKIE is.

Jap ,
Zou je nu die while willen uitleggen?