Login con md5 , BD funciona pero no funciona

[securedigital]

Hola

Bua, sigue aún que no entra bien.
Consegui que entrara, preo si se ponia el passwd bien o mal entraba de todas formas...pero al final lo dejo de otra forma. He cambiado algunas cositas.


Aver si alguien puede ver que demonios le passa.


index.php

Código [Seleccionar] Expandir


<?php


//define('CONFIG','config-main.php');
//include(CONFIG);
include("config-main.php");
if (!$_POST['submit']){
echo "<form name='login' method='post' action='index.php'>
<input type='text' name='user' /><br />
<input type='password' name='pass' /><br>
<input type='submit' value='Entrar' name='sumbit' /><input type='reset' value='Esborrar' />
</form>";
} else { 
$user = $_POST['user'];
$pass = md5($_POST['pass']);

$query = mysql_query("Select * from login where User = '$user'");
$query = mysql_num_rows($query);

if ($query == 0) {
echo "No s'ha trobat l'usuari.<br />
<a href='index.php'>Tornar enrere</a>";
} else {
$query2 = mysql_query("Select * from login where User = '$user'");
$query2 = mysql_fetch_array($query2);
if ($pass != $query2['pass']) {
echo "Les contrasenyes no coincideixen.<br />
<a href='index.php'>Tornar enrere</a>";
} else {

session_start();
$_SESSION['user'] = $user;

header("Location: Principal.php");
}
}
}
?>


Principal.php

Código [Seleccionar] Expandir

<?php
session_start();
include ("config-main.php");

//$query = mysql_query("Select * from login where User = $_SESSION['user']");
// linea que daba ERROR y ahora no
$query = mysql_query("select * from login where User = '".$_SESSION['user']."'");
$query = mysql_num_rows($query);

if ($query != 1){
header("Location: index.php");

} else {
echo "Aqui deberia mostrar el contenido";
}
?>


config-main.php

Código [Seleccionar] Expandir

<?php


$host="localhost";
$user="*********"; //usuari de la base de dades
$pass="********"; //password de la base de dades
$db="******"; //nom de la base de dades 

//$conexion = mysql_connect($host,$user,$pass);
//mysql_select_db($db,$conexion);


//mysql_connect($host,$user,$pass);
//mysql_select_db(*******);

mysql_connect($host,$user,$pass) or die (mysql_error());
mysql_select_db($db) or die(mysql_error());

?>




Saludos

[Rentero]

Holas.


¿Ya funciona?

No estoy siguiendo el post pero un par de cosillas.
En la página de login tienes que filtar la variable $user($_POST['user']) con htmlspecialchars(), addslashes(), etc por el tema de la seguridad. No puedes utilizar una variable para una consulta sin antes filtrar su cotenido, sino tendrás problemas con SQL inyection y demás.

Otra cosilla por decir algo, en las páginas que quieras proteger no es necesarrio que hagas ninguna consulta. Puedes hacerlo simplemente comprobando si existe $_SESSION['user']

Saludos

[securedigital]

Principal.php Arreglado, el tema de sql injection ya me lo he estado mirando, pero antes que nada que el codigo funcione correctamente....antes de lialr mas codigo allí.

Sige sin funcionar.


Principal.php

Código [Seleccionar] Expandir

<?php
session_start();
include ("config-main.php");

//$query = mysql_query("Select * from login where User = $_SESSION['user']");
// linea que daba ERROR y ahora no

//$query = mysql_query("select * from login where User = '".$_SESSION['user']."'");
//$query = mysql_num_rows($query);

//if ($query != 1){
if ($_SESSION[user] == NULL){ 
header("Location: index.php");

} else {
echo "Aqui deberia mostrar el contenido";
}
?>

[Rentero]

Hola.

¿No te funciona? ¿Te da algún error? ¿Cual es el problema?

Prueba esto:

Código [Seleccionar] Expandir

<?php
error_reporting(E_ALL); //para que muestre todos los errores
session_start();
include("config-main.php");

if (isset($_POST['entrar']))
{
$user = $_POST['user']; //filtrar esto
$pass = md5($_POST['pass']);

$query = mysql_query("Select * from login where User='$user'");
$resultados = mysql_num_rows($query);

if ($resultados > 0)
{
$query2 = mysql_query("Select pass from login where User='$user'");
$contra = mysql_fetch_array($query2);

if ($pass == $contra['pass'])
{
$_SESSION['user'] = $user;
header("Location: Principal.php");
}else{
echo "Les contrasenyes no coincideixen.<br /><a href='index.php'>Tornar enrere</a>";
}

}else{
echo "No s'ha trobat l'usuari.<br /><a href='index.php'>Tornar enrere</a>";
}
}
?>
<form name='login' method='post' action='index.php'>
<input type='text' name='user' /><br />
<input type='password' name='pass' /><br>
<input type='submit' value='Entrar' name='entrar' /><input type='reset' value='Esborrar' />
</form>

Teóricamente debería funcionar. error_reporting() lo usamos para que muestre todos los errores o avisos durante la ejecución del archivo. No s sirve para ver si hay algún problema por parte de PHP.

Sinceramente solo le he echado un vistazo por encima, pero prueba ese index.php y nos cuentas que pasa con más detalle

Otra cosa: ¿User va con mayúscula en la base de datos?
Otra más: En las páginas protegidas. Debería bastar con:

Código [Seleccionar] Expandir

<?php
session_start();

if (isset($_SESSION['user']))
{ 
echo "Aqui deberia mostrar el contenido";
} else {
header("Location: index.php");
}
?>

[SeniorX]

Otra más: En las páginas protegidas. Debería bastar con:

Código [Seleccionar] Expandir

<?php
session_start();

if (isset($_SESSION['user']))
{ 
echo "Aqui deberia mostrar el contenido";
} else {
header("Location: index.php");
}
?>

Cuidado con eso, que es muy vulnerable. Debes chequear si la sesion es correcta, es decir si el nombre de usuario y la password coinciden. Lo que haces con el isset($_SESSION['user']) es ver si la sesion tiene un nombre de usuario nada mas.
Ojo con eso.

[Rentero]

Se supone que lo compruebas en la página de login, como dije antes.

De todas formas, el tema de la seguridad corre por cuenta propia.

Saludos

[securedigital]

Hola, tambien me olvidé

Código [Seleccionar] Expandir

session_register(sd); en la pagina a proteger para que coga el session   

Buff, ya estoy hasta los....no tengo tanto nivel para programarlo yo solo y acabar de terminarlo, me da demasiados problemas y me esta comiendo muuuuucho tiempo. He cogido un script de login ya hecho de por ai y lo he modificado a mi gusto. Ya esta totalmente funcional.

Solo tengo una duda.
Tamien le he aplicado un filtro para evitar SQL injection, he "prohibido" algunos caracteres

Código [Seleccionar] Expandir

("([*])|([|])|([;]|([`])","",$string)

Debo de poner otros o alguna cosa mas para aumentar la seguridad al maximo en el tema SQL injection e ataques XSS?


Saludos y gracias