MySQL & PHP-Problem

[ecHo89]

Ahh, danke Matze.

jfheins:

Delphi-Quellcode:

$val = mysql_fetch_assoc($sql);
echo $val['Key']; oder die($val['Key']);

funktioniert wie bereits erwähnt nicht. Mir fällt auch kein Grund ein wieso es beim anderen nach genau dem gleichen Schema funktioniert...

[Matze]

Guten Morgen

Zitat:

Zitat von ecHo89

SQL-Code:

$sql = mysql_query("SELECT Key FROM Table2 WHERE Key='$Seriennummer'");
$val = mysql_fetch_assoc($sql);

die($val['Key']);

Warning: mysql_fetch_assoc() expects parameter 1 to be resource, boolean given in /users/echo/www/ValidateKey.php on line 11

</span>

Ich weiß echt nicht wo da der Fehler liegen sollte... Table2-> NR: int, Key: varchar(8)

Hier fällt mir sofort auf, dass "Key" durch den Syntax-Highlighter hervorgehoben wird. Du kannst mal versuchen, anstelle von "Key" einen anderen Namen zu verwenden. Nicht, dass MySQL das als "PRIMARY KEY" o.ä. interpretiert.
Das würde auch erklären, wieso es bei deinem anderen Aufruf mit dem gleichen Aufbau (aber anderen Spaltennamen) funktioniert.

Grüße, Matze

[ecHo89]

Ja, das ist mir auch schon aufgefallen. Habs auch vorgestern mal getestet mit anderer Spalte Key2.
Werde jetzt nocheinmal systemmatisch von vorne bis hinten alles durchgehen und schauen, dass nirgends Key benutzt wird.
Mal schaun... -.-

[Phoenix]

Zitat:

Zitat von ecHo89

SQL-Code:

mysql_query("UPDATE " . $tablename . " SET  AnzahlReg= " . $val[AnzahlReg] . " WHERE Seriennummer='" . $Seriennummer . "'");

Du weisst schon das genau das einer der am häufigsten genannten Gründe ist, warum PHP-Anwendungen als unsicher gelten?

Stelle Dir mal vor, der User gibt als Seriennummer in Deiner Delphi-Anwendung folgendes ein:

Code:

123'; DELETE FROM TABLE Table2;

Das Statement was bei raus kommt ist folgendes:

SQL-Code:

UPDATE tblBlaBlubb SET  AnzahlReg= 5 WHERE Seriennummer='123';
DELETE FROM TABLE table2;

Das erste Statement liefert noch nichtmal einen Fehler, es wird einfach kein Datensatz gefunden. Das zweite Statement löscht mal kurzerhand den Inhalt einer Deiner Tabellen.

Wenn Du Dich wunderst wie man an die Tabellen kommt: Man kann den Netzwerkverkehr und damit das abgefragte PHP-Script recht einfach mittels Netzwerk-Sniffern herausfinden. Mit anderen Befehlen die man direkt an Dein PHP-Script schicken kann (also ohne Umweg über Deine Delphi-Anwendung) kann man z.B. anstelle des DELETE FROM Befehls erstmal ein SHOW TABLES; absetzen. In der Fehlerausgabe Deines Scriptes sieht man dann schön alle zur verfügung stehenden Tabellen. Und dann kann man die sich sogar komplett ausgeben lassen und findet sogar alle zur Verfügung stehenden Seriennummern heraus. Und man kann sogar ein UPDATE laufen lassen, dass die Registierung wieder auf 0 setzt.

Mach Dir bitte Gedanken ob das so gewollt ist oder ob Du vielleicht ein bisschen mehr Aufwand in parametrisierte Statements stecken willst, mit denen Du so eine SQL-Injection vermeiden könntest.

[ecHo89]

Dachte man kann nur direkte Eingaben per php manipulieren. Aber wenn das Passwort für die Datenbank (im Delphi-Quelltext zum connecten zur DB) mitgesnifft werden kann, dann is das schlecht. Hab mal ein bisschen gesucht, aber nichts gescheites zu dem Stichwort gefunden. Hat jemand ne gute Quelle?

[Valle]

Zitat:

Zitat von ecHo89

Dachte man kann nur direkte Eingaben per php manipulieren. Hab mal ein bisschen gesucht, aber nichts gescheites zu dem Stichwort gefunden. Hat jemand ne gute Quelle?

Die Sache nennt man SQL Injection. Dazu findet man bei einschlägigen Suchmaschinen oder Online Enzyklopädien einiges. ;-)

Edit:// @Phoenix: Die mysql_query Funktion lässt nur ein Query (auch durch Semikolon getrennt) zu. Dein Beispiel gibt einfach nur einen Syntax-Fehler. ;-) Aber das Prinzip ist natürlch korrekt.

Liebe Grüße,
Valle

[jfheins]

Die einfachste Möglichkeit:

Code:

<?php
$sql = mysql_query('SELECT Key FROM Table2 WHERE Key="' . mysql_real_escape($_POST['Seriennummer']) . '"') or die(mysql_error());
if(mysql_num_rows($sql) == 1) 
{ 
  $val = mysql_fetch_assoc($sql); 
  echo $val['Key']; 
} 
else 
{ 
  die('Fehler: Falsche Ergebnismenge') 
}
?>