Imprimir Página - Login con md5 , BD funciona pero no funciona

Título: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 12 Mayo 2007, 20:24 pm

Hola a todos.

Tengo mi web site, con un login para entrar, este login consulta en la BD el usuario y el password en md5. Obviamente la bd contiene por ejemplo

User pepito
pass 21545615645456456465 (md5 inventado)

El hacer clic para entrar vuleve a cargar el login, de forma que no accede a la pagina siguiente, se queda atascado y si pongo un user i pasword inventados hace lo mismo. Segun como esta programado deberia aparecer mensajes de error si el usuario no esta en la BD , pero no aparece NADA, simplemente vuleve a cargar el login.

Teoricamente deberia passar a Principal.php

Código:

<?php


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

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

?>

Código:


<?php 


define('CONFIG','config-main.php');
include(CONFIG);
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_fetch_array($query);
		if ($query2['Pass'] != $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 tiene este script, de forma que si no has superado el login , pues no entras.

Código:

<?php
session_start();
include ("config-main.php");
$query = mysql_query("Select * from login where User = $_SESSION['user']");
$query = mysql_num_rows($query);

if ($query != 1){
header("Location index.php");
} else {
}
?>

Saludos

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Mance en 13 Mayo 2007, 17:05 pm

config-main.php

en la línea.

Código:

mysql_select_db($db) or die(mysq_error());

Código:

mysql_select_db($db) or die(mysql_error());

mysql_error()

la "L"... :P.

Código:

 if ($query2['Pass'] != $pass) {

¿No sería $query2['pass'] != $pass...

lo digo por la mayúscula.

revisa el código entero ...

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 13 Mayo 2007, 23:33 pm

ostras, esto se me havia colado...

He arreglado esos problemas y sigue igual.

Es muy raro ya que si pongo qualquier user y passwd no salta el mensaje de error, por lo que halgo me falla en index.php pero no veo que demonios es.

De todas formas intento ir a Principal.php manualmente que es teoricamente donde tiene que redireccionar el index.php y ahora me sale un error.

Código:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /usr/export/www/hosting/Principal.php on line 4

esta linea que da error esta arriba, esta todo el codigo de Principal.php.

Código:

$query = mysql_query(\"Select * from login where User = $_SESSION[\'user\']\");

No esta passando correctamente el Session, por lo que al index algo le pasa.

Saludos

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Rentero en 14 Mayo 2007, 01:49 am

Parse error: Son errores de sintaxis.

Código:

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

No tienes porqué escapar la comillas.

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 14 Mayo 2007, 16:37 pm

Hola

He dejado la linea esta asi:

Código:

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

Sigue dando el mismo error.

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Hans el Topo en 14 Mayo 2007, 19:41 pm

Cita de: securedigital en 14 Mayo 2007, 16:37 pm

Hola

He dejado la linea esta asi:

Código:

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

Sigue dando el mismo error.

Código:

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

Las cadenas se comparan con comillas simples en sql xD

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 14 Mayo 2007, 20:59 pm

Bien, ahora ya no da el error, pero al introducir el usuario y password correcto entra a Principal.php, pero es que si le meto cualquier otro que no este en la BD tambien entra y no sale ni mensage de advertencia ni te redirecciona a index.php para poner el user adecuado.

Por cierto,

Citar

$puntero_al_conector

?¿? puntero al conector ? no te entiendo

Saludos

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Hans el Topo en 15 Mayo 2007, 00:12 am

http://es2.php.net/manual/es/function.mysql-query.php

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 16 Mayo 2007, 18:42 pm

Arreglado header("Location index.php");

a

Código:

header("Location: index.php");

He retocado un poco la estrucutra del codigo, siquieren la publico, funciona mejor pero aun no del todo bien.

Saludos

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Luisango en 16 Mayo 2007, 20:26 pm

Publica, publica, jejjee, pon el codigo, posiblemente asi sea mas facil de sacar fallos, si esque hay, supongo..

Saludos!

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 19 Mayo 2007, 20:09 pm

Hola

Bua, sigue aún que no entra bien. :xD
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:

<?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:

<?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:

<?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

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Rentero en 20 Mayo 2007, 03:00 am

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 ;)

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 20 Mayo 2007, 19:15 pm

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:

<?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";
}
?>

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Rentero en 21 Mayo 2007, 00:22 am

Hola.

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

Prueba esto:

Código:

<?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:

<?php
session_start();

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

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: SeniorX en 21 Mayo 2007, 04:49 am

Cita de: Rentero en 21 Mayo 2007, 00:22 am

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

Código:

<?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.

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: Rentero en 21 Mayo 2007, 22:49 pm

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 ;)

Título: Re: Login con md5 , BD funciona pero no funciona
Publicado por: securedigital en 25 Mayo 2007, 17:46 pm

Hola, tambien me olvidé

Código:

session_register(sd);

en la pagina a proteger para que coga el session :o :xD

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:

("([*])|([|])|([;]|([`])","",$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

Foro de elhacker.net

Programación => PHP => Mensaje iniciado por: securedigital en 12 Mayo 2007, 20:24 pm