Funkcja mail plus wysylanie danych z bazy danych Opcje

[cameloos]

Witam.

Mam takie oto dwa pliczki:

formularz.php

[PHP] pobierz, plaintext 
<?php
Przypominacz hasla:<br><br>
<form action="mail.php" method="post"><br>
Podaj swoj login<br>
login: <input type="textbox" name="login"><br>
<input type="submit" value="wyslij">
?>
[PHP] pobierz, plaintext 

oraz<br>
mail.php

[PHP] pobierz, plaintext 
<?
require_once ("funkcje.php");
$polaczenie= pol_mysql();
$wynik = mysql_query("select uzyt_login, uzyt_haslo, uzyt_emial from uzytkownicy where uzyt_login
='$login'");
 
$haslo=='uzyt_haslo';
$email=='uzyt_email';
 
mail($email, $haslo, $login);
?>
Wyslano
[PHP] pobierz, plaintext 

Czy takie dziwactwo ma prawo dzialac? Bo poki co, nie dotarl do mnie zaden emial na adres przyporządkowany ktoremukolwiek z loginow, znajdujacych sie w bazie danych Funkcje łączenia z bazą danych są oczywiście zdefiniowane w pliku funkcje.php

[TomASS]

<?php //uzywaj zawsze tych tagów php, nie samego <?

A php scripting block always starts with <?php and ends with ?>. A php scripting block can be placed anywhere in the document.

On servers with shorthand shorthand support enabled you can start a scripting block with <? and end with ?>.

However, for maximum compatibility, we recommend that you use the standard form (<?php) rather than the shorthand form.

Czyli <? też jest niby zgodne ze standardem (tak jak kiedyś <% ) to wszystko zależy od dyrektyw:

short_open_tag = On
asp_tags = Off

w pliku php.ini.

[PHP] pobierz, plaintext 
<?php
$login = $_POST['login'];
$wynik = mysql_query("select uzyt_login, uzyt_haslo, uzyt_emial from uzytkownicy where uzyt_login
='$login'");
 
?>
[PHP] pobierz, plaintext 

A jak login będzie miał wartość jakis_login' OR 1 ?
Lepiej daj:

[PHP] pobierz, plaintext 
<?php
$login = mysql_escape_string($_POST['login']);
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$haslo==$uzytkownik['uzyt_haslo'];
$email==$uzytkownik['uzyt_email'];
?>
[PHP] pobierz, plaintext 

A nie powinno być raczej:

[PHP] pobierz, plaintext 
<?php
$haslo=$uzytkownik['uzyt_haslo'];
$email=$uzytkownik['uzyt_email'];
?>
[PHP] pobierz, plaintext