kleines Problem mit Script

renzo · 15.06.2005, 22:30

Hallo,

habe ein kleines Problem mit meinem Script, Flüchtigkeitsfehler inbegriffen :wink: **grins**

also hier der Code:

PHP-Code:

  <?php

<?php

$db_server = "***";

  $db_name = "***";

  $db_user = "***";

  $db_passwort = "***";

 $pass = 45; /* das habe ich nur, dass man nicht mit einer Leereingabe sich Zugriff erstatten kann */

  $db = @MYSQL_CONNECT($db_server,$db_user,$db_passwort); 

$db_select = @MYSQL_SELECT_DB($db_name);

$sql = "SELECT pass, name FROM pass where name='".$_REQUEST['user']."'";

$result = mysql_query($sql); 

   while($zeile = mysql_fetch_array($result))

   {

$pass = $zeile["pass"];

}

 
if($_REQUEST['Send'] && md5($_REQUEST['Passwort'])==$pass)

{

    session_start();

    session_register("SESSION_Passwort");

    $SESSION_Passwort = $pass;

}

 
?>

 
<html>

<head>

<title>Passwortgeschützter Bereich</title>

</head>

<body>

 
<?php

 
if($SESSION_Passwort==$pass)

{

    echo "Du bist eingelogt";

}

else

{

    echo "Du bist nicht berechtigt die Seite zu betreten!";

}

 
?>

 
</body>

</html>

?>

dabei bekomme ich eine Fehlermeldung:
Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /check.php on line 11

wo hab ich der Fehler, sollte mein Script nicht so gut sein, Vorschläge und Ratschläge sind immer gestattet.

danke schon im Voraus

cu renzo

Der_Gerhard · 15.06.2005, 22:39

Hier kommt gleich mal eine ganze Liste von Verbesserungsvorschlägen. Der Fehler ist aber vermutlich nicht dabei, weil das wichtigste fehlt.

1. session_register ist veraltet. Nimm lieber $_SESSION[].
2. Hinter jedes mysql_query gehört im Fehlerfall eine Ausgabe von mysql_error().
3. @ verbergen nur Fehler. Sie verhindern sie nicht.
4. error_reporting(E_ALL) oben im Script wäre auch eine tolle Sache.
5. Kümmer Dich mal drum, ob Passwort und name überhaupt im $_REQUEST gesetzt sind.

Wenn Du 2. schon gemacht hättest, würde Dir MySQL eine Fehlermeldung vorwerfen. Da Du Dich nicht um den Rückgabewert kümmerst, merkst Du erst beim mysql_fetch_array(), dass was nicht stimmt.
Vermutlich findet MySQL die Spaltennamen name und pass sowie den Tabellennamen pass nicht unbedingt gut. Such mal nach den reservierten Wörtern bei MySQL. Das eine oder andere davon könnte darin enthalten sein.

15.06.2005, 23:48

6. $_REQUEST[] kannst du bei register_globals auf Off knicken. gewöhn dir an mit $_POST[],$_GET[],$_COOKIE[],und $_SERVER[] zu arbeiten

axo · 16.06.2005, 00:55

Zitat:

Zitat von Du-weisst-schon-wer

6. $_REQUEST[] kannst du bei register_globals auf Off knicken.

??? wage ich mal anzuzweifeln. http://us2.php.net/reserved.variable...iables.request

16.06.2005, 01:04

Zitat:

Zitat von axo

wage ich mal anzuzweifeln.

Nur Mut und keine falsche Bescheidenheit. <ggg>

renzo · 16.06.2005, 07:43

danke erstmal,

ich muss jetzt zur Schule, werde aber danach die Fehler versuchen zu beheben bzw. die Vorschläge befolgen.

cu renzo

16.06.2005, 07:51

Zitat:

Zitat von axo

Zitat:

Zitat von Du-weisst-schon-wer

6. $_REQUEST[] kannst du bei register_globals auf Off knicken.

??? wage ich mal anzuzweifeln. http://us2.php.net/reserved.variable...iables.request

Waa.. ja natürlich.. war wohl einfach zu spät gestern Abend.
Ich meinte du solltest GET,POST und COOKIE verwenden anstelle von REQUEST[] benutzen und dazu register_globals anpassen..

*gähn* *kaffe trink*

16.06.2005, 09:11

Zitat:

Zitat von Du-weisst-schon-wer

Ich meinte du solltest GET,POST und COOKIE verwenden anstelle von REQUEST[] benutzen und dazu register_globals anpassen..

Der Kontext ist immer noch falsch. Wichtig ist register_globals = Off
Ob man eine Formularvariable in $_POST oder in $_REQUEST sucht, ist seit der Erfindung von fsockopen() bzw. cURL egal.

Zitat:

*gähn* *kaffe trink*

Weniger gähnen, mehr Kaffee.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Erledigt] Statistik Script Referrer Problem	7Style	PHP Tipps 2008	28	01.07.2008 15:14
Problem mit php script	Jelko	PHP Tipps 2008	2	19.08.2007 23:28
mal funktionierts, mal nicht - großes Problem mit PHP Script	nicobischof	PHP Tipps 2005-2	5	01.10.2005 16:13
Problem mit Tracker script	b++	PHP Tipps 2005-2	1	14.09.2005 01:54
[Erledigt] FAQ erstellen, kleines Problem mit Kategorien		PHP Tipps 2005-2	1	09.09.2005 14:09
mod_rewrite kleines problem -umschreiben url		PHP Tipps 2005-2	2	21.07.2005 13:16
n kleines Problem...		PHP Tipps 2005-2	3	28.06.2005 08:54
Problem mit Script zur Dateiauflistung		PHP Tipps 2005-2	3	24.06.2005 16:12
[Erledigt] File download > kleines Problem		PHP Tipps 2005	3	11.05.2005 23:11
Problem mit Script auf verschiedenen Servern		PHP Tipps 2005	40	22.03.2005 17:26
problem fertiges script anzupassen	kid01	HTML, Usability und Barrierefreiheit	8	22.03.2005 07:25
Problem mit Script	Mex	PHP Tipps 2005	5	13.03.2005 21:29
kleines Problem mit Bildergalerie	alisa	PHP Tipps 2005	3	09.02.2005 12:59
n kleines problem		PHP Tipps 2004	18	12.07.2004 21:13
Kleines Prob mit Login Script		PHP Tipps 2004	2	09.07.2004 01:39

15.06.2005, 22:30
renzo Neuer Benutzer Registriert seit: 15.06.2005 Beiträge: 6	kleines Problem mit Script Hallo, habe ein kleines Problem mit meinem Script, Flüchtigkeitsfehler inbegriffen :wink: grins also hier der Code: PHP-Code: <?php <?php $db_server = "*"; $db_name = ""; $db_user = ""; $db_passwort = ""; $pass = 45; / das habe ich nur, dass man nicht mit einer Leereingabe sich Zugriff erstatten kann */ $db = @MYSQL_CONNECT($db_server,$db_user,$db_passwort); $db_select = @MYSQL_SELECT_DB($db_name); $sql = "SELECT pass, name FROM pass where name='".$_REQUEST['user']."'"; $result = mysql_query($sql); while($zeile = mysql_fetch_array($result)) { $pass = $zeile["pass"]; } if($_REQUEST['Send'] && md5($_REQUEST['Passwort'])==$pass) { session_start(); session_register("SESSION_Passwort"); $SESSION_Passwort = $pass; } ?> <html> <head> <title>Passwortgeschützter Bereich</title> </head> <body> <?php if($SESSION_Passwort==$pass) { echo "Du bist eingelogt"; } else { echo "Du bist nicht berechtigt die Seite zu betreten!"; } ?> </body> </html> ?> dabei bekomme ich eine Fehlermeldung: Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /check.php on line 11 wo hab ich der Fehler, sollte mein Script nicht so gut sein, Vorschläge und Ratschläge sind immer gestattet. danke schon im Voraus cu renzo


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

15.06.2005, 22:39
Der_Gerhard Erfahrener Benutzer Registriert seit: 08.11.2004 Beiträge: 2.079	Hier kommt gleich mal eine ganze Liste von Verbesserungsvorschlägen. Der Fehler ist aber vermutlich nicht dabei, weil das wichtigste fehlt. 1. session_register ist veraltet. Nimm lieber $_SESSION[]. 2. Hinter jedes mysql_query gehört im Fehlerfall eine Ausgabe von mysql_error(). 3. @ verbergen nur Fehler. Sie verhindern sie nicht. 4. error_reporting(E_ALL) oben im Script wäre auch eine tolle Sache. 5. Kümmer Dich mal drum, ob Passwort und name überhaupt im $_REQUEST gesetzt sind. Wenn Du 2. schon gemacht hättest, würde Dir MySQL eine Fehlermeldung vorwerfen. Da Du Dich nicht um den Rückgabewert kümmerst, merkst Du erst beim mysql_fetch_array(), dass was nicht stimmt. Vermutlich findet MySQL die Spaltennamen name und pass sowie den Tabellennamen pass nicht unbedingt gut. Such mal nach den reservierten Wörtern bei MySQL. Das eine oder andere davon könnte darin enthalten sein. __________________ ******************************** Nein, ich bin nicht die Signatur. Ich putze hier nur. ********************************

15.06.2005, 23:48
Gast Beiträge: n/a	6. $_REQUEST[] kannst du bei register_globals auf Off knicken. gewöhn dir an mit $_POST[],$_GET[],$_COOKIE[],und $_SERVER[] zu arbeiten

16.06.2005, 07:43
renzo Neuer Benutzer Registriert seit: 15.06.2005 Beiträge: 6	danke erstmal, ich muss jetzt zur Schule, werde aber danach die Fehler versuchen zu beheben bzw. die Vorschläge befolgen. cu renzo