Very good. Allthough, I think it's much easier to check user and pass at the same time:SELECT * FROM users where usmeanre='$usmeanre' AND password = $pass'Plus it's more secure because you don't let hackers now that the usmeanre exisits you just display usmeanre and/or password doesn't exist I don't understand the while in conjunction with mysql_fetch_assoc, doesn't work the same without the while?