Problema con ELSE al iniciar sesión en un sistema

[iGussy]

Buen día amigos, tengo un problema con un else de un codigo php, el cual es el siguiente:

<?php

include("/config.php");
if(isset($_POST['logear']))
{
/*tomaremos el valor de las variables user y pass de los textboxs*/
$user = mysql_real_escape_string($_POST["nick"]);
$pass = mysql_real_escape_string($_POST["pass"]);
/* haremos un query para ver si los datos introducidos existen la base de datos*/
$login_query = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."'");
$fetch = mysql_fetch_array($login_query);
if($fetch['usuario'])
{
if($fetch['contra'] == $pass)
{
echo '<div id="boxreg"><font color="green">Sesión iniciada con exito</font></div>';
}else{
echo '<div id="boxreg"><font color="green">Algunos datos est&aacute;n incorrectos</font></div>';
}
}
}else{
?>

El problema es que al iniciar sesión si manda el mensaje bien de Sesión iniciada con exito, pero al poner algún dato incorrecto o falta de espacios en blanco no me manda error de else :S...

Que puedo hacer?

[medcol]

por lo que veo tienes errada la validación.

if($fetch['contra'] == $pass && $fetch['usuario']==$user)
{
echo '<div id="boxreg"><font color="green">Sesión iniciada con exito</font></div>';
}else{
echo '<div id="boxreg"><font color="green">Algunos datos est&aacute;n incorrectos</font></div>';
}

[iGussy]

Iniciado por medcol

por lo que veo tienes errada la validación.

if($fetch['contra'] == $pass && $fetch['usuario']==$user)
{
echo '<div id="boxreg"><font color="green">Sesión iniciada con exito</font></div>';
}else{
echo '<div id="boxreg"><font color="green">Algunos datos est&aacute;n incorrectos</font></div>';
}

Muy bien, amigo, por lo que veo tiene lógica su acomodación, ahora el detalle es que al no dar ningún dato al formulario de registro a los inputs, ejemplo

usuario:
Mi usuario
contraseña:
*****

Al no poner ningún dato y le doy iniciar sesión me sale: Sesión iniciada con éxito, que podría ser?

[medcol]

puedes hacer varios trucos.

1. crear una función en javascript que valide los campos vacios y no te deje pasar hasta que haya algun dato en usuario o contraseña.

2. la session la debes de crear si el usuario o contraseña son validos. de lo contrario te seguirá saliendo la session iniciada porque obviamente al enviar datos siempre va estar el isset

[andresgerard]

Hola, veo tu login, y te puedo dar varias recomendaciones:

1- Usa strips_tags para crear integridad dentro de la variable que obtienes por el $_POST
[URL="http://www.php.net/manual/es/function.strip-tags.php"]http://www.php.net/manual/es/function.strip-tags.php[/URL]

2- No le veo logica a esta linea "if(isset($_POST['logear']))". Debes hacer una reestructura del logeo, por lo que observo.

3- Deberias utilizar algun metodo de encriptacion de la contraseña, particularmente utilizo sha1. Pero en tiempos como los de ahora es dificil conseguir las medidas de seguridad perfecta para contrarestar ataques a tu pagina

Un Ejemplo del login

Código PHP:

  $user = (strip_tags($_POST['user']));
$password = (strip_tags(sha1($_POST['password'])));
$query = @mysql_query('SELECT * FROM usuarios WHERE user="'.mysql_real_escape_string($user).'" AND password="'.mysql_real_escape_string($password).'"');
 
if($existe = @mysql_fetch_object($query))
{
        $_SESSION['logged'] = 'yes';
        $_SESSION['user'] = $user;
        $mensaje='Bienvenido '.$user.', Ya va a su destino';
        print "<script>alert('$mensaje')</script>";
        print("<script>window.location.replace('destino.php');</script>");
}
 else 
{ $mensaje2='El Usuario y/o contraseña son incorrecto/s, por favor verifique los datos de Acceso.';
    print "<script>alert('$mensaje2')</script>";
    print("<script>window.location.replace('index.php');</script>");
}