Witam wszystkich
Otrzymałem w pracy bojowe zadanie zabezpieczenia formularza php po przez wpisanie kodu z obrazka, jednak jestem totalna lama i coś mi nie wyszło, obrazek się wyświetla i na tym koniec dobrych wieści, obojętnie co nie wpiszę to formularz można wysłać, dodatkowo przychodzi bez treści ! są tylko 3 linijki z tytułami kategorii, co zrobiłem źle ?

sory że tak wkleję z grubej rury wszystko ale muszę bo kompletnie nie wiem gdzie błąd jest
php----------------------------------------
<?


$mail='blur2@o2.pl';
$topic='Zapytanie maszyny';
$params= "From: $email\r\n"
// ."Reply-To: $email\r\n"
."Content-Type: TEXT/HTML; CHARSET=ISO-8859-2";

//wlasciwa czesc wiadomosci
if ($nazwa_firmy<>"") $message="$message <B>Nazwa firmy: </B>$nazwa_firmy<BR>";
if ($ulica_i_numer<>"") $message="$message <B>Ulica i numer: </B>$ulica_i_numer<BR>";
if ($kod_i_miejscowosc<>"") $message="$message <B>Kod i miejscowo?溠</B>$kod_i_miejscowosc<BR>";
if ($telefon<>"") $message="$message <B>Telefon: </B>$telefon<BR>";
if ($email<>"") $message="$message <B>Adres e-mail: </B>$email<BR>";
if ($osoba_kontaktowa<>"") $message="$message <B>Osoba kontaktowa: </B>$osoba_kontaktowa<BR>";
if ($dzial<>"") $message="$message <B>Dzia?: </B>$dzial<BR>";
if ($fax<>"") $message="$message <B>Fax: </B>$fax<BR>";
if ($telefon_komorkowy<>"") $message="$message <B>Telefon kom󲫯wy: </B>$telefon_komorkowy<BR>";

ciach

function passcheck($pss) {
$nr = floor(time() / 3600);
for($i = 0; $i < strlen($pss); $i++)
if($i % 2 == 0) $a .= $pss[$i]; else $b .= $pss[$i];
$a = base_convert(strrev($a), 36, 10);
$b = base_convert($b, 36, 10);
if($a + $b <= $nr && $a + $b >= $nr - 2) return true;
return false;
}

if(!passcheck( $passa )) die("Haslo niepoprawne");

mail($mail, $topic, $message, $params);;
echo("<META HTTP-EQUIV=\"Refresh\" CONTENT=\"0; URL=../index.php\">");
?>

html---------------------------------------------

<! Tu dodalem1>
<img src="image.php">
<p>Aby wysłać formularz wpisz hasło z obrazka</p>
<INPUT TYPE="TEXT" NAME="passa" SIZE="8">


<input class="inputbutton"
onclick="MM_validateForm('nazwa_firmy','','R','Telefon','','R','email','','R','','','R','osoba_kontaktowa','','R','telefon_komorkowy','','R','','','R','','','RisEmail');return document.MM_returnValue"
value="Wyslij" name="Wyslij" type="submit">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<input value="Skasuj" name="skasuj" type="reset"><br>
<br>

</FORM>
</TD></TR></TABLE>

image.php--------------------------------------------

<?php
//ob_start();
//ob_implicit_flush(0);
function passgen() {
$nr = floor(time() / 3600);
srand(array_sum(explode(" ", microtime())) * 100000);
$a = rand(0, min(getrandmax(), $nr));
$b = $nr - $a;
$a = base_convert($a, 10, 36);
$b = base_convert($b, 10, 36);
$a = strrev(str_pad($a, max(strlen($a), strlen($b)), 0, STR_PAD_LEFT));
$b = str_pad($b, max(strlen($a), strlen($b)), 0, STR_PAD_LEFT);
for($i = 0; $i < strlen($a); $i++) $pass .= $a[$i].$b[$i];
return $pass;
}
header("Content-type: image/jpeg");
$img = imagecreate(110, 30);
$background = imagecolorallocate($img, 206, 207, 099);
$textcolor = imagecolorallocate($img, 0, 0, 0);
imagestring($img, 5, 10, 7, passgen(), $textcolor);
imagejpeg($img);
imagedestroy($img);
ob_end_flush();
?>
-------------------------------------------

powinienes użyć tablic $_POST[], w sensie przypisać dane do zmiennych najpierw bo teraz to masz puste te zmienne

Hej, troszkę bałaganu zrobiłeś

Zrob tak

1. POST wyżej
2. w pliki images.php numer którey wyswetli się na obrazku zapisz do sesji
3. Skryt ktory wysyla dodaj tam ifa gdzie np if ($_POST['KOD'] == $_SESSION['KOD']) ...
4. mail($mail, $topic, $message, $params);; za duzo ";"

jak taki sam tzn ze jest ok jak nie to wyswetl komunikat.

PS proponuję jakaś klase np phpmailer czy cos innego...