Endlosschleife und MySQL-Befehl

[icecream]

So, hallo Leute!
Erstmal frohes neues

Aber jetzt zu meinem Problem. Ich schreibe ja derzeit an einem Newsscript. Die Eingabe-Datei und die Login, Sowie Auslesungsdatei hab ich schon fertig .

Aber in der Bearbeitungsdatei habe ich einen Fehler:
PHP/MySQL meldet:

Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in /srv/www/htdocs/web40/html/script-tests/editnews.php on line 15

Der Code ist folgender:

<?php
include("#_mysql.inc");
session_start();
if(session_is_registered("login") && $_SESSION['login']) {
$sql = mysql_query("SELECT id, titel FROM news ORDER BY id DESC LIMIT $start,$eintraege");
$count = mysql_query("SELECT COUNT(*) from news");
$all = mysql_result($count, 0);
$start = $_GET['s'];
$eintraege = 15;
if (($start + $eintraege) > $all) {
    $eintraege = $all - $start;
    $next = 0;
} else $next = $start + $eintraege;
echo "<table width=\"600\"><tr><td width=\"50\"><b>id</b></td><td width=\"480\"><b>Titel</b></td><td width=\"70\"><b>Aktion</b></td></tr>";
while ($ds = mysql_fetch_object($sql)) {
$titel = $ds -> titel;
$id = $ds -> id;
echo "<tr><td>$id</td><td>$titel</td><td><a href=\"editnews2.php?id=$id\">Edit</a> | <a href=\"delnews.php?id=$id\">Delete</a></td></tr>";
}
} else include('login.php');
?>

Ich hoffe mir kann jmd. helfen!

Gruß
icecream

[ghostgambler]

echo mysql_error();
unter mysql_query(...)

[icecream]

OK, danke, hab das Problem gefunden!

Gruß
icecream

[icecream]

Naja... dafür hat sich ein neue Problem ergeben... Irgendwie wurde das zu einer endlosschleife und listet mir jetzt alle zu bearbeitende News auf. Aber immer die Selbe! Also wenn ich als id=1 nehm, listet er mir alle news der id 7 auf. Wenn ich id=3 nehm, listet er alle news der id 5 auf.
Welche es logischerweise nur einmal gibt... aber das alles in einer Endlosschleife. Also habe ich jetzt 1000 mal da stehen:
id Titel Aktion
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete

Der aktuelle Code lautet:

<?php
include("#_mysql.inc");
session_start();
if(session_is_registered("login") && $_SESSION['login']) {
$sql = mysql_query("SELECT id, titel FROM news ORDER BY id DESC LIMIT $start, $eintraege");
$count = mysql_query("SELECT COUNT(*) from news");
$all = mysql_result($count, 0);
$start = $_GET['s'];
$eintraege = 15;
echo "<table width=\"600\"><tr><td width=\"50\"><b>id</b></td><td width=\"480\"><b>Titel</b></td><td width=\"70\"><b>Aktion</b></td></tr>";
while ($ds = mysql_fetch_object(mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT $start, $eintraege"))) {
$titel = $ds -> titel;
$id = $ds -> id;
echo "<tr><td>$id</td><td>$titel</td><td><a href=\"editnews2.php?id=$id\">Edit</a> | <a href=\"delnews.php?id=$id\">Delete</a></td></tr>";
}
if (($start + $eintraege) > $all) {
    $eintraege = $all - $start;
    $next = 0;
} else $next = $start + $eintraege;
} else include('login.php');
?>

Danke schonmal im Vorraus!

Gruß
icecream

[icecream]

Endlosschleife und MySQL-Befehl

Irgendwie meine Abfrage zu einer endlosschleife und listet mir jetzt alle zu bearbeitende News auf. Aber immer die Selbe! Also wenn ich als id=1 nehm, listet er mir alle news der id 7 auf. Wenn ich id=3 nehm, listet er alle news der id 5 auf.
Welche es logischerweise nur einmal gibt... aber das alles in einer Endlosschleife. Also habe ich jetzt 1000 mal da stehen:
id Titel Aktion
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete
5 Nochmal ein Test Edit | Delete

Der aktuelle Code lautet:

<?php
include("#_mysql.inc");
session_start();
if(session_is_registered("login") && $_SESSION['login']) {
$sql = mysql_query("SELECT id, titel FROM news ORDER BY id DESC LIMIT $start, $eintraege");
$count = mysql_query("SELECT COUNT(*) from news");
$all = mysql_result($count, 0);
$start = $_GET['s'];
$eintraege = 15;
echo "<table width=\"600\"><tr><td width=\"50\"><b>id</b></td><td width=\"480\"><b>Titel</b></td><td width=\"70\"><b>Aktion</b></td></tr>";
while ($ds = mysql_fetch_object(mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT $start, $eintraege"))) {
$titel = $ds -> titel;
$id = $ds -> id;
echo "<tr><td>$id</td><td>$titel</td><td><a href=\"editnews2.php?id=$id\">Edit</a> | <a href=\"delnews.php?id=$id\">Delete</a></td></tr>";
}
if (($start + $eintraege) > $all) {
    $eintraege = $all - $start;
    $next = 0;
} else $next = $start + $eintraege;
} else include('login.php');
?>

Danke schonmal im Vorraus!

Gruß
icecream

[Shurakai]

jedes mal wenn die schleife durchgeführt wird holt er ein objekt aus der resource, die die mysql query zurückliefert - und die jedesmal neu ausgeführt wird. ....


Informier dich bitte auch mal über XSS - dein Script hat Sicherheitslücken die so groß sind wie nen Scheunentor... (z.B. $start)

[TobiaZ]

OffTopic:

---

Naja, wenigstens wird wohl kaum jemand versuchen ihm über sql-injections schaden zuzufügen.

---

[TobiaZ]

Bitte keine Doppelpostings!!! Die sind in jedem Forum unerwünscht.

Darüber hinaus halte dich bitte an unsere speziellen Regeln:
http://www.php-resource.de/forum/sho...threadid=50454
Das macht es uns leichter dir zu helfen und hätte dir sicher den ein oder anderen Beitrag erspart!

[icecream]

Gut... ich habe mich mal ein bisschen über "XSS" informiert... könnte mir jmd. helfen evtl. Sicherheitslücken auszubessern und mir erklären, wie ich in zukunft vorgehen sollte?

Gruß
icecream

[jahlives]

Du bindest eine Usereingabe $_GET['s'] ungeprüfe in eine mysql query ein ($start)
Zumindest solltest du das machen:

$start = mysql_real_escape_string(strip_tags($_GET['s'])); 

Gruss

tobi

[icecream]

ok, hab ich gemacht danke

aber kann mir jmd. sagen, warum ich ne endlosschleife erzeugt habe? und warum immer nur die eine id angezeigt wird? das problem ist oben genauer beschrieben...

Gruß
icecream

P.S.: Das ist das erste mal, dass ich mit Datenbänken arbeite, deswegen bitte nicht so harte kritik

[jahlives]

Vermutlich liegts an dieser Zeile

while ($ds = mysql_fetch_object(mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT $start, $eintraege"))) {

//besser
$ds = mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT $start, $eintraege");
while($erg = mysql_fetch_object($ds)){
.... 

Überigens hat Shurakai dir schon gesegt in welcher Richtung das Problem der Enlosschleife liegt

jedes mal wenn die schleife durchgeführt wird holt er ein objekt aus der resource, die die mysql query zurückliefert - und die jedesmal neu ausgeführt wird. ....

d.h. die Query liefert immer wieder eine Resultat Ressource, die wieder bei 0 beginnt (weil du query wieder aufrufst)
Du musst die query ausserhalb der Schleife machen und mit der Schleife nur durch die Ressource des Resultats gehen.

Gruss

tobi

[TobiaZ]

bevor du deine Postings nicht an unsere Regeln angepasst hast, macht hier keiner irgendwas!

[icecream]

Jetzt geht alles
Vielen, vielen Dank!

Gruß
icecream