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