sistema de usuarios

[floog]

hola
he creado un sistema de registro de usuarios y todo funciona perfectamente, la verda y por raro que parezca... bueno la cuestion es que si meto el user o el pass equivocados me envia de nuevo al formulario de login, pero sin embargo si dejo ambos cuadros de texto vacios y hago submit se registra en el sistema y accede como administrador.

alguien sabe como se puede resolver este hecho tan insolito?
sldos,

[Ronruby]

Verifica que la variable no este vacia usando trim() y empty()

Código PHP:

if(empty(trim($_POST['campoUsuario']))) {
  //Campo vacio, no proceder a login.
}
 

[RodrigoG]

Podrías validar, tipo:

Código PHP:

if( empty($_POST['nomUsuario'] && empty($_POST['claveUsuario'])) {
echo 'Lo sentimos, el nombre de usuario y la clave no pueden estár vacíos :P';
exit();
}
 

Puedes acomodar eso como te plazca

[RodrigoG]

Ups! ya me habían ganado

[floog]

lo he probado pero no me sale... donde deberia incorporar el codigo, en el fichero principal o bien en el que jecuta la consulta a la bbdd. Dejo copia del codigo

Código PHP:

  <?
// conexion a la base de datos
include ('dbopen.php');
//verificamos que los valores no esten vacios
if (empty(trim($_POST["user"] && $_POST["pass"])))
    { header ("Location:/intranet/index.php?error=si");
    } else {
//recogemos los valores del formulario
$usuario = $_POST["user"];
$password = $_POST["pass"];
 
//si las variables no estan vacias proseguimos
//realizamos la consulta a la bbdd
$conectar = "SELECT * FROM usuarios WHERE user = '$usuario' AND pass = '$password'";
$array = mysql_query ($conectar);
$rows = mysql_fetch_array ($array);
 
 
// redireccionamiento en funcion del resultado
if ( $password == $rows ["pass"]){
        session_start ();
        $autentificado = "SI";
        $_SESSION["autentificado"] = "SI";
        $_SESSION["nombre"] = $rows["nombre"];
        $_SESSION["level"] = $rows["level"];
        header ("Location:/intranet/");
} else {
        header ("Location:/intranet/index.php?error=si");
}
    }
?>   

[Ronruby]

Código PHP:

if (empty(trim($_POST["user"] && $_POST["pass"])))
 

Estas pasandole 2 parametros a trim y empty usando &&.
Deberia ser:
if(empty(trim($_POST['user'])) && empty(trim($_POST['pass'])))

[floog]

pues ya lo he probado como indicais pero sigue sin funcionar. adjunto el codigo, alguna idea?

Código PHP:

  <?
// conexion a la base de datos
include ('dbopen.php');
//verificamos que los valores no esten vacios
//verificamos que los valores no esten vacios
if(empty(trim($_POST['user'])) && empty(trim($_POST['pass'])))
    { header ("Location:/intranet/index.php?error=si");
    } else { 
//recogemos los valores del formulario
$usuario = $_POST["user"];
$password = $_POST["pass"];
 
//si las variables no estan vacias proseguimos
//realizamos la consulta a la bbdd
$conectar = "SELECT * FROM usuarios WHERE user = '$usuario' AND pass = '$password'";
$array = mysql_query ($conectar);
$rows = mysql_fetch_array ($array);
 
 
// redireccionamiento en funcion del resultado
if ( $password == $rows ["pass"]){
        session_start ();
        $autentificado = "SI";
        $_SESSION["autentificado"] = "SI";
        $_SESSION["nombre"] = $rows["nombre"];
        $_SESSION["level"] = $rows["level"];
        header ("Location:/intranet/");
} else {
        header ("Location:/intranet/index.php?error=si");
}
    }
?>   

[Ronruby]

Disculpa, cometí un error en el código.

En vez de "&&" es "||"

[floog]

sigue sin funcionar...crees que podría ser el tema del if dentro de otro. me refiero: el primer if chequea que los $_POST esten vacios, y si lo estan, que devuelva a origen; si no es asi (else), ejecuta la consulta a la bbdd, donde despues, para verificar el password hay otro if...podra ser eso??

[Ronruby]

Lo que pasa es que no recordaba que no se podia usar empty(trim($variable))
Quita el trim

[floog]

funciona¡¡
gracias