Onemillion

Dieses Script ist eine Art Gästebuch; sollte es zumindest sein!
Das script zeigt zwar keine Fehlermeldung aber etwas senden tut es auch nicht
glaube das irgendetwas an meine action fehlerhaft it





<?php
$url = $_POST['url'];
$name = $_POST['name'];
$banner = $_POST['banner'];
$nachricht = $_POST['nachricht'];
$ip = $_POST['ip'];

$verbindung = mysql_connect("mysql.lima-city.de", "USER", "PW")
or die("Keine Verbindung moeglich");
mysql_select_db("db_110743_3")
or die ("Die Datenbank existiert nicht");

$eintrag = "
INSERT INTO `onemillionnews` (`url`, `name`, `banner`, `nachricht`, `ip`)
VALUES ('$url', '$name', '$banner','$nachricht','$ip')";


$eintragen = mysql_query($eintrag);


?>
<form style="margin:0px auto;width:480px;border:1px solid #000000;text-align:left;"
action= "<?php print $PHP_SELF ?>" method="post" target="_self">

<span id="sprytextfield2">
<label>Adresse der unerwünschten Website:
<input name="url" type="text" id="url" size="45" value="url" />
</label>
<span class="textfieldRequiredMsg">
</span></span>

<hr size="4" />
<span id="sprytextfield1">
<label>Ihre Website:

<input name="name" type="text" id="name" size="45" value="name"/>
</label>
</span>

<label>

Beschreibung ihrer Website:
<textarea name="nachricht" id="nachricht" cols="45" rows="5" value="nachricht"></textarea>
</label>

<label>Url. zu ihrem Banner bitte hier eintragen:
<input name="banner" type="text" id="banner" size="45" value="banner"/>
</label>


<span id="sprycheckbox1">
<input name="Kontrollkästchen" type="checkbox" value="" />

</span>
<p>Ich bin mit den Regeln dieser Website vertraut und werde mich an diese halten! </p>

<label>Wollen sie diese Daten senden?
<input type="submit" name="senden" id="senden" value="Senden" />
</label>

<input type="hidden" name="ip" id="ip" value="ip"/>
<u>Ihre IP Lautet:</u> <span class="Stil1"><? echo $REMOTE_ADDR ?></span>
</form>

Mister Ad

Koala

- PHP_SELF ist nicht gut wegen XSS.

besser: action="<?php echo $_SERVER['SCRIPT_NAME'];?>"

- benutze mysql_error() und schreib in die erste Zeile Deines Scripts error_reporting(E_ALL);

- Prüfe mit if(isset($_POST['senden'])) ob Dein Script abgeschickt wurde und trage die Daten nur nach erfolgreicher Prüfung der eingegebenen Daten in die DB ein (mysql_real_escape_string benutzen !)

Onemillion

wi mache ich das(mein erstes PHP script...)
Könntest du das kurz zeigen...

Koala

PTC

Warum hast du zwei gleiche Posts?
entfernt

litterauspirna

Noch besser so.

Das hat den Vorteil (finde ich) wenn du einen Fehler in mysql_bekommst das dir der sql string in $eintrag gleich mit ausgegeben wird und du da manchmal leichter den Fehler entdeckst.

__________________
Aus dem Dynamo Lande kommen wir. Trinken immer reichlich kühles Bier. Und dann sind wir alle voll, die Stimmung ist so toll. Aus dem Dynamo Lande kommen wir.
http://www.lit-web.de

Wolla

Finde ich nicht gut. So lieferst du einem potentiellen Angreifer gleich Tabellennamen und Feldliste mit, wenn der zufällig auf einen Fehler läuft.

Onemillion

wie gebe ich einem PHP scrit einen Name wenn dies in der quellcode integriert ist?

Koala

gar nicht.

Schau Dir das $_SERVER-Array an:

Onemillion

Habe das jetzt eingetragen. Was sagt mir das?