[php] Foutmelding maar toch ingelogd?

woensdag 16 januari 2002 11:53

zal wel met post/get te maken hebben denk ik
mis ik niet session_start();
??

Doet iets met Cloud (MS/IBM)

Acties:

woensdag 16 januari 2002 11:54

Topicstarter

Op woensdag 16 januari 2002 11:51 schreef D2k het volgende:
zal wel met post/get te maken hebben denk ik
mis ik niet session_start();
??

Stimmt jah!

Maar dit is niet de hele code van die pagina

Als je dat nodig hebt, zeg je het maar.

Acties:

Janoz

Moderator Devschuur®

!litemod

En mischien een $ voor de vars in je query...

Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'

woensdag 16 januari 2002 11:54

Acties:

woensdag 16 januari 2002 11:57

Op woensdag 16 januari 2002 11:54 schreef Janoz het volgende:
En mischien een $ voor de vars in je query...

en een addslashes

Doet iets met Cloud (MS/IBM)

Acties:

woensdag 16 januari 2002 11:58

Topicstarter

Op woensdag 16 januari 2002 11:54 schreef D2k het volgende:

[..]

en een addslashes

Ja, sorry, die staat er wel in. De hele source:

PHP:

<?
 
    session_start(); 
?> 

<html> 
<head> 
<title>Login pagina</title> 
</head> 
<body bgcolor=ffffcc> 
    <CENTER>
        <H1>
        
            <?php 
                $text = "Welkom bij Taxicom"; 
                echo $text
            ?>
        </H1>
    </CENTER>

<form method="post" action="login2.php"> 
<input type="hidden" name="Submit" value="1"> 
<input type="text" name="LoginNaam"> 
<input type="password" name="LoginPassword"> 
<input type="submit" value="Login"> 
</form> 

<p> 

<?php 

If ($Submit) { 

$server = mysql_connect("localhost", "*******", "*******"); 

        @mysql_select_db("*******", $server); 

    $LoginNaam = addslashes($LoginNaam); 
        $LoginPassword = addslashes($LoginPassword); 

$Query = mysql_query("SELECT * FROM leden WHERE Loginnaam = 'Loginnaam' AND LoginPassword = 'Wachtwoord'"); 
    $Results = mysql_num_rows($Query); 

        if ($Results == '1') { 

            $Login = "1"; 

        session_register("Login"); 

            echo "Je bent ingelogd. Klik hier om verder te gaan.\n"; 

        } 
        else { 

        echo " Je gebruikersnaam of password klopt niet.\n"; 

        } 

} 
?> 

</body> 
</html> 
?>

Acties:

Moet je niet de geposte variabelen in de query verwerken?
En $Results bevat een integer dus 1 niet tussen quotjes?

PHP:

<?
$Query = mysql_query("SELECT * FROM leden WHERE Loginnaam='$Loginnaam' AND LoginPassword='$Wachtwoord'"); 
    $Results = mysql_num_rows($Query); 
    if ($Results == 1) { 
?>

woensdag 16 januari 2002 12:01

Acties:

woensdag 16 januari 2002 12:01

Op woensdag 16 januari 2002 11:57 schreef BreadFan het volgende:

[..]

Ja, sorry, die staat er wel in. De hele source:
[php]

mwoah het kan ook zo

code:

1	$Query = mysql_query("SELECT * FROM leden WHERE Loginnaam = "'.addslashes($Loginnaam).'" AND LoginPassword = "'.addslashes($Wachtwoord).'"");

typo's voorbehouwen
zoiets was het geloof ik

Doet iets met Cloud (MS/IBM)

Acties:

woensdag 16 januari 2002 12:04

Topicstarter

Op woensdag 16 januari 2002 11:58 schreef Arnim het volgende:
Moet je niet de geposte variabelen in de query verwerken?

True, had ik eerst wel, heb er tijdens het klooien een keer uitgehaald.

En $Results bevat een integer dus 1 niet tussen quotjes?

PHP:

<?
$Query = mysql_query("SELECT * FROM leden WHERE Loginnaam='$Loginnaam' AND LoginPassword='$Wachtwoord'"); 
    $Results = mysql_num_rows($Query); 
    if ($Results == 1) { 
?>

Ja dat zou goed kunnen. En de regel daarop moeten ze ook weg? En dan ff tussen haakjes zetten?

Acties:

Janoz

Moderator Devschuur®

!litemod

Pas wel op... Vaak staat magic_quotes_gpc aan. Daardoor zijn de get post en cookie vars al automatisch ge'addslashed'. Als je het dan nog een keer doet, krijg je er dubbele slashes in staan...

Mischien kun je

PHP:

<?
 $LoginNaam = addslashes($LoginNaam); 
        $LoginPassword = addslashes($LoginPassword); 
?>

beter vervangen door:

PHP:

<?
if (!magic_quotes_gpc()){
  $LoginNaam = addslashes($LoginNaam); 
  $LoginPassword = addslashes($LoginPassword);
}
?>

Daarnaast heb je nog steeds geen $ (dus vars) in je query staan.....

Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'

woensdag 16 januari 2002 12:05

Acties:

Op woensdag 16 januari 2002 12:01 schreef BreadFan het volgende:
En de regel daarop moeten ze ook weg? En dan ff tussen haakjes zetten?

Zou ik ze inderdaad ook weghalen. En haakjes

Volgens mij niet.

woensdag 16 januari 2002 12:08

Acties:

woensdag 16 januari 2002 12:41

Topicstarter

Op woensdag 16 januari 2002 12:04 schreef Janoz het volgende:
...

Nu krijg ik deze error:

Fatal error: Call to undefined function: magic_quotes_gpc() in /home/sites/site12/users/taxicom/web/login2.php on line 33

Dat is hem dus niet.

die $ heb ik aan de vars toegevoegd.

Acties:

Janoz

Moderator Devschuur®

!litemod

Op woensdag 16 januari 2002 12:08 schreef BreadFan het volgende:

[..]

Nu krijg ik deze error:
[..]

Ik doe het ook maar ff uit mijn hoofd.. De manier zoals ik het boven aangeef zorgt ervoor dat je script onafhankelijk wordt van je server instellingen (Eigenlijk moet je er dan ook rekening mee houden dat globals uit kunnen staan, maar dat is mischien beter iets voor de volgende keer)

Ik zal wel ff voor je in de manual kijken hoe die functie precies heet (Zou je natuurlijk zelf ook kunnen doen)

UIt de manual (die is best handig!!!) bleek dat het de functie get_magic_quotes_gpc() was. (Er moet dus nog get_ voor)

Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'

woensdag 16 januari 2002 12:58

Acties:

woensdag 16 januari 2002 13:00

Topicstarter

Op woensdag 16 januari 2002 12:41 schreef Janoz het volgende:
UIt de manual (die is best handig!!!) bleek dat het de functie get_magic_quotes_gpc() was. (Er moet dus nog get_ voor)

Klopt, ik zat me net al af te vragen waarom hij niet meekleurde, maar bruin bleef.

Nu werkt het wel, zoals je zei.

Maar die foutmelding blijf ik krijgen, op die regel met $Results = mysql_num_rows($Query);

Acties:

woensdag 16 januari 2002 13:12

or die(mysql_error());

Doet iets met Cloud (MS/IBM)

Acties:

Codeer eens alsvolgt. Dan weet je precies wat er allemaal gebeurt...

PHP:

<?
$debug = 1;
..
$sql = "SELECT * FROM leden WHERE Loginnaam='$LoginNaam' AND LoginPassword='$LoginPassword' ";
$result = mysql_query( $sql ); 
if ( $debug )
{
    echo "SQL: $sql <BR>\n";
    echo "Result: $result <BR>\n":
    echo "Error: " . mysql_error() . "<BR>\n";
    echo "Num rows: " . mysql_num_rows( $result ) . "<BR>\n";
}
..
?>

woensdag 16 januari 2002 13:37

Acties:

woensdag 16 januari 2002 13:48

Topicstarter

Op woensdag 16 januari 2002 13:12 schreef Arnim het volgende:

Kewl!

Dat heeft idd geholpen! De namen van de kolommen in de db bleken niet exact overeen te komen met wat ik in mijn code had staan.

Als uitvoer van het bovenstaande krijg ik nu oa. dit:

SQL: Resource id #2
Result:
Error:

Warning: Supplied argument is not a valid MySQL result resource in /home/sites/site12/users/taxicom/web/login2.php on line 51
Num rows:

Wat wil dat achter "SQL:" zeggen? En d'r zit kennelijk nog een kleinfoutje in, maar da's niet erg.

Acties:

Hij zegt dat de query gelukt is en geeft een verwijzing naar een mysql resultset.

Wat eigenlijk de bedoeling is, dat je achtereenvolgens exact weergeeft welke query uitgevoerd wordt, of er een resultset uitkomt, of er een error is en hoeveel records dat hij teruggeeft. Dan moet je wel de variabelen goed gebruiken.