Estaria correcto este codigo?

me gustaria que hicierais alguna critica al codigo: Grin

en si es la verificacion es para que nu pagina use usuarios de foros smf, y nose si podria mejorar algo, ayudadme un poco, gracias!

Código:

<?php
session_start();
include("conexion.php");
$HTTP_SESSION_VARS["sesion"]="false";
function mysql_escape($var) {
    if(get_magic_quotes_gpc() != 0) {
        $var = stripslashes($var);
    }
    return mysql_real_escape_string($var);
}
$name=mysql_escape($HTTP_POST_VARS["name"]);
$pass=mysql_escape($HTTP_POST_VARS["pass"]);
if(trim($name) !="" && trim($pass) !="") {
$result=mysql_query("SELECT passwd FROM smf_members WHERE memberName='$name'");
if($row=mysql_fetch_array($result)) {
if($row["passwd"]==sha1(strtolower($name).$pass)) {
$result2=mysql_query("SELECT ID_GROUP FROM smf_members WHERE memberName='$name'");
$row2=mysql_fetch_array($result2);
if($row2["ID_GROUP"]=="1") {
$HTTP_SESSION_VARS["name"]=$name;
$HTTP_SESSION_VARS["sesion"]="true";
header("location: index.php");
} else {
header("location: error.php?error=login");
}} else {
header("location: error.php?error=login");
}} else {
header("location: error.php?error=login");
}} else {
header("location: error.php?error=login");
}
mysql_close();
?>

si esta bien el code.. de hecho se parece un poco al de smf o eso me parecio... Tongue

Hosting en México, WebHosting, Dominios, Servidores Privados Virtuales, Servidores Dedicados
HostingMexicano.com

HTTP_POST_VARS ??

PD: Pregunto eso porque ahora puedes usar $_POST, $_SESSION, $_COOKIE

Bueno amigo el code esta bien, te felicito...

El límite del éxito lo establece la mente humana, por eso, si quieres ver cosas que nunca haz visto, empieza a hacer cosas que nunca haz hecho.

Crear un captcha con PHP

Un .GIF o .JPG que muestre la IP de una persona (PHP)

jeje gracias lo de HTTP_POST_VARS seque se puede poner $_POST y eso pero antes en mi hosting antiguo no me aceptaba la version "reducida" de las variables y habia que poner todo entero, no se exactamente porque... supongo que quizas por la version de PHP, asique desde entonces por si acaso escripo todo completo jeje

espero que a la gente le ayude el aporte.. jeje

_________________$Saludos!

	Bienvenido(a), Visitante. Favor de ingresar o registrarse. ¿Perdiste tu email de activación? - Octubre 12, 2008, 08:43:27