Mysql Where Probleme

[Acer90]

Hallo irgent wie schein mein Mysql update mit den Where niht zu funktionieren.

hier mal mein code

<?php
require_once 'admin/inc/konfiguration.php';

mysql_connect(MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT);		   	
mysql_select_db(MYSQL_DATENBANK)
 	or die("Auswahl der Datenbank fehlgeschlagen");
	

$disabledg = $_GET['disabled'];
$nummer = $_GET['nummer'];

$resmonitor = mysql_query("SELECT * FROM `monitoring` WHERE nummer = '$nummer'");

 while ($resm = mysql_fetch_assoc($resmonitor)):
 
    $commentg = $resm['comment'];
	$portg = $resm['c_port'];
	$ip = $resm['ip'];
	$game = $resm['type'];

endwhile;


if ( $disabledg == "1")
{
	$dis = "0";
}
else
{
	$dis = "1";
}
	
	
	
$eintragen = mysql_query("UPDATE monitoring SET disabled = '$dis' WHERE ip = '$ip' AND game = '$game' AND c_port = '$portg' AND comment = '$commentg'");

Hoffe jemand hat die lösung, denn ich sitze schon nen paar tage dran, wäre sehr dankbar.

Es geht um die Letzte zeile.

 

[HasenBerg]

Was genau ist denn der Fehler? Also was genau funktioniert nicht?
Außerdem solltest du dich dringend mal über Sicherheit bei PHP & MySQL informieren - ein ungetestetes $_GET in einer MySQL-Abfrage hat schon so manchen Schaden erzeugt!

 

[aichingm]

hi, was mein Vorposter mit Sicherheit meint solltest du mit mysql_real_escape_string() abdecken.

und zu deinem problem: ich verwende wenn ich updates mache nur mysql_query(); und nicht $x = mysql_query(); versuch das doch bitte mal!

MfG Mario

 

[vitus37]

Lasse doch bitte mal einen MySQL-Error ausgeben.

mysql_query("...") or die(mysql_error());

 

[Acer90]

Also hab den fehler gefunden, desweitern ist get leider von nöten da das Skript im nachhienein über mehrer webserver geht und daten austausch.
Es können dabei aber auch keine wichtige daten verloren gehen.

 

[vitus37]

Wenn du dich vor SQL-Injections nicht schützt, ermöglichst du Angreifern undgeahnte Möglichkeiten. vlmedia » SQL-Injection

 

[sysop]

hi, was mein Vorposter mit Sicherheit meint solltest du mit mysql_real_escape_string() abdecken. .....

Mal abgesehen davon, dass man in $nummer ein delete oder insert verstecken kann, dass dann die Datenbank/tabelle loeschen oder mich als Admin eintragen kann.