Куки в авторизации

[bloger]

Поставил авторизацию а она не работает жалуется на куки.А точнее print "Хм, что-то не получилось";.Помогите разобратся.

Login.php

PHP код:

<?

# Функция для генерации случайной строки
function generateCode($length=6) {
    $chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHI JKLMNOPRQSTUVWXYZ0123456789";
    $code = "";
    $clen = strlen($chars) - 1;  
    while (strlen($code) < $length) {
            $code .= $chars[mt_rand(0,$clen)];  
    }
    return $code;
}

if(isset($_POST['submit']))
{
    # Вытаскиваем из БД запись, у которой логин равняеться введенному
    $query = mysql_query("SELECT id, password FROM users WHERE login='".mysql_real_escape_string($_POST['login'])."' LIMIT 1");
    $data = mysql_fetch_assoc($query);
    
    # Соавниваем пароли
    if($data['password'] == md5(md5($_POST['password']))) 
    {
        # Генерируем случайное число и шифруем его
        $hash = md5(generateCode(10));
            
        if(!@$_POST['ip'])
        {
            # Если пользователя выбрал привязку к IP
            # Переводим IP в строку
            $insip = ", ip=INET_ATON('".$_SERVER['REMOTE_ADDR']."')";
        }
        
        # Записываем в БД новый хеш авторизации и IP
        mysql_query("UPDATE users SET hash='".$hash."' ".$insip." WHERE id='".$data['id']."'");
        
        # Ставим куки
        setcookie("id", $data['id'], time()+60*60*24*30);
        setcookie("hash", $hash, time()+60*60*24*30);
        
        # Переадресовываем браузер на страницу проверки нашего скрипта
        header("Location: check.php"); exit();
    }
    else
    {
        print "Вы ввели неправильный логин/пароль";
    }
}
?>
<form method="POST">
Логин <input name="login" type="text"><br>
Пароль <input name="password" type="password"><br>
Не прикреплять к IP(не безопасно) <input type="checkbox" name="ip"><br>
<input name="submit" type="submit" value="Войти">
</form>

check.php

PHP код:

<?
// Скрипт проверки

if (isset($_COOKIE['id']) and isset($_COOKIE['hash']))
{   
    $query = mysql_query("SELECT *,INET_NTOA(ip) FROM users WHERE id = '".intval($_COOKIE['id'])."' LIMIT 1");
    $userdata = mysql_fetch_assoc($query);

    if(($userdata['hash'] != $_COOKIE['hash']) or ($userdata['id'] != $_COOKIE['id'])
 or (($userdata['ip'] != $_SERVER['REMOTE_ADDR'])  and ($userdata['ip'] != "1")))
    {
        setcookie("id", "", time() - 3600*24*30*12, "/");
        setcookie("hash", "", time() - 3600*24*30*12, "/");
        print "Хм, что-то не получилось";
    }
    else
    {
        print "Привет, ".$userdata['login'].". Всё работает!";
    }
}
else
{
    print "Включите куки";
}
?>

[QunneD]

print "Хм, что-то не получилось";

Точную ошибку которую PHP выдает дай.

[ADSoft]

похоже в логике что-то

PHP код:

if(($userdata['hash'] != $_COOKIE['hash']) or ($userdata['id'] != $_COOKIE['id']) 
 or (($userdata['ip'] != $_SERVER['REMOTE_ADDR'])  and ($userdata['ip'] != "1"))) 


на экран все переменные что в стравнении выводите - и смотрите ... где не выполняется условие и какие значения кук

[bloger]

Ошибки то нет.Просто когда вводишь правильный логин и пароль всегда появляется вот это Хм, что-то не получилось, а нада чтобы Привет, ".$userdata['login'].". Всё работает!

[ADSoft]

Сообщение от bloger

Ошибки то нет.Просто когда вводишь правильный логин и пароль всегда появляется вот это Хм, что-то не получилось, а нада чтобы Привет, ".$userdata['login'].". Всё работает!

говорю же... значит выполняется общее условие в if .... а какое имеено - надо смотреть - ввыводя эти переменные

[QunneD]

говорю же... значит выполняется общее условие в if .... а какое имеено - надо смотреть - ввыводя эти переменные

Поддерживаю

[bloger]

Проверел все переменные выводятся.Но по прежнему не нашол ошибки.

[Metandrostenalon]

последнее условие наверно лишнее

[ADSoft]

Сообщение от bloger

Проверел все переменные выводятся.Но по прежнему не нашол ошибки.

конечно будут выводится... вопрос в том чтобы выводя переменные используемые в условии - проверить логическую цепочку..... что выолняется в условии что нет - тогда будет полная картина....
по поводу того что последнее сравнение ИП адреса с 1 ненужно... - хз... автора скрипта нада спрашивать - какой там тайный смысл .... но так как (1) или (2) а ип сравнение в (2) ... то в принципе достаточно отследить что в (1) происходит

PHP код:

echo '1='.$userdata['hash'];
echo '2='.$_COOKIE['hash']) 
итд 


сделайте вывод и смотрите - реально ли совпадает

а по ИП - точно вот ошибка есть
в селекте выбираете INET_NTOA(ip) - а сравниваете $userdata['ip'], тогда или алиас вводите
или сравнивайте $userdata['INET_NTOA(ip)']

[Android_ua]

Насколько я понял, то сервер ругается на то что параметры хедера уже выставлены, и по этому setcookie() надо ставить в самое начало кода, а именно перед тегом <html> и какими нибудь другими тегами.
Я это сделал так

PHP код:

<?php
include('blocks/db.php');
if (isset($_POST['login'])) { $login = $_POST['login']; if (empty($login)) { unset($login);} } 
if (isset($_POST['password'])) { $password=$_POST['password']; if (empty($password)) { unset($password);} }
if (!isset($login) or !isset($password))
    {
    $error = ("<h1>Вы ввели не всю информацию, пожалуйста заполните все поля!</h1>");
    }
    $login    = stripslashes($login);
    $login    = htmlspecialchars($login);
    $password = stripslashes($password);
    $password = htmlspecialchars($password);
//удаляем лишние пробелы
    $login    = trim($login);
    $password = trim($password);

    $password = md5($password);//шифруем пароль    
if (!empty($login) and !empty($password))
    {
    $check = mysql_query("SELECT pass FROM users WHERE login='$login'",$db);
    $check_row = mysql_fetch_array($check);
        if(empty($check) or $check_row['pass'] != $password)
        {
         $error = ("<h1>Вы ввели неверный логин и/или пароль, пожалуйста попробуйте снова!</h1>");
        }
        else
        {
         $success = ("<h1>Поздравляем, вы успешно вошли в систему, ваш логин <strong>".$login."</strong>! <br /> Вы будете перенаправлены на предыдущую страницу через 3 секунды");
         $message = $success;//потом просто выводим в текстовом блоке
         setcookie("login", $login, time()+9999999);
         setcookie("password", $password, time()+9999999);
        }
    }
$referer = $_SERVER['HTTP_REFERER'];
?>

P.S. Для того что-бы выводить сообщения разного рода(например "вы авторизированы" или "логин и пароль неверны") - я после проверки эти сообщения ввел в переменную $message, а потом в html коде уже выводил сообщение посредством оператора echo