X   Сообщение сайта
(Сообщение закроется через 3 секунды)



 

Здравствуйте, гость (

| Вход | Регистрация )

Открыть тему
Тема закрыта
> помогите исправить ошибки в скрипте регистрации
kirill24
kirill24
Topic Starter сообщение 3.11.2009, 21:34; Ответить: kirill24
Сообщение #1


Всем привет.
Где то здесь на сайте нашел ссылку на урок по написанию скрипта регистрации.
http://freshcoder.ru/archives/249

там скрипт начинается так
[php]
<?php
session_start();
//Поключаем конфиг
include("config.inc.php");

//Необходимо подключиться к БД
$link = mysql_connect($DBSERVER, $DBUSER, $DBPASS)
or die("Не могу подключиться" );
// сделать $DB текущей базой данных
mysql_select_db($DB, $link) or die ('Не могу выбрать БД');

//Если нет сессий
if(md5(crypt($_SESSION['user'],$_SESSION['password'])) != $_SESSION['SID']) {/*здесь ошибка*/
//Если кнопка не нажата, отображаем форму
if(!$_POST['do']){/*здесь ошибка*/

echo '
<a href="registration.php">регистрация</a> <br/><br/>
<form name="1" action="" method="post">
<input name="login" type="text" value=""> <br/>
<input name="password" type="password" value=""> <br/>
<input name="do" type="submit" value="Войти">
</form>';

}
//Если кнопка нажата
if($_POST['do']) {/*здесь ошибка*/
[/php]при выполнении на локальном сервере браузер пишет:
Notice: Undefined index: user in z:\home\localhost\www\mywallstreet\reg\index.php on line 13

Notice: Undefined index: password in z:\home\localhost\www\mywallstreet\reg\index.php on line 13

Notice: Undefined index: SID in z:\home\localhost\www\mywallstreet\reg\index.php on line 13

Notice: Undefined index: do in z:\home\localhost\www\mywallstreet\reg\index.php on line 15

потом выводит форму логин/пароль

и потом еще две ошибки такого же плана
(строки на которые он ругается я выделил в скрипте)
сам скрипт вообще работает.
залил на хостинг - там эти ошибки не показывает.
я так понимаю там вроде есть где то параметр какие ошибки отображать а какие нет.
но я все равно волнуюсь - регистрация как-никак, если тут че то с ошибкой....
помогите пожалуйста разобраться, серьезные это ошибки или нет, и если да, то как их исправить?
0
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Apay
Apay
сообщение 3.11.2009, 22:34; Ответить: Apay
Сообщение #2


[PHP]if( empty($_SESSION['user']) || empty($_SESSION['user']) ||
md5( crypt( $_SESSION['user'], $_SESSION['password'] ) ) != $_SESSION['SID'] ){
[/PHP]
и
[PHP]if( empty( $_POST['do'] ) ){[/PHP]
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
kirill24
kirill24
Topic Starter сообщение 3.11.2009, 22:54; Ответить: kirill24
Сообщение #3


БЛИН!
да я вижу где эти ошибки. что возникают они потому что там такие индексы стоят, я понимаю.
но автор то этот скрипт в урок выложил!
значит у него работает..

и еще он все время пишет про какую то кнопку - проверяем нажата ли кнопка. я так понимаю что по нажатии кнопки эти значения и появятся в массиве POST.
но у меня целиком его скрипт, и вот такая ошибка.
может он на них внимания не обращает просто? типо пока их нет, то они и не нужны, а как появятся, то и хорошо...? не может такого быть?
или все таки что-то там не так, и в таком варианте скрипт будет неверно работать?
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Apay
Apay
сообщение 3.11.2009, 23:31; Ответить: Apay
Сообщение #4


так скрипт тоже работает, вместо несуществующих переменных подставляет null, но выводит notice. на хостингах обычно вывод notice отключают
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
kirill24
kirill24
Topic Starter сообщение 3.11.2009, 23:53; Ответить: kirill24
Сообщение #5


то есть впринципе ничего страшного в этом нет?
и скрипт работать будет, и несанкционированный доступ никто не получит?
ну то есть уязвимостей и глюков от этого не добавится?
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Apay
Apay
сообщение 4.11.2009, 0:03; Ответить: Apay
Сообщение #6


(kirill24 @ 4.11.2009, 01:53) *
и скрипт работать будет
будет
(kirill24 @ 4.11.2009, 01:53) *
и несанкционированный доступ никто не получит?
а вот это - отдельная тема
(kirill24 @ 4.11.2009, 01:53) *
ну то есть уязвимостей и глюков от этого не добавится?
может да может нет
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
kirill24
kirill24
Topic Starter сообщение 4.11.2009, 0:22; Ответить: kirill24
Сообщение #7


(Apay @ 4.11.2009, 02:03) *
а вот это - отдельная тема


ну, я думаю, вы уже догадались о чем я теперь спрошу? ;)
как сделать, чтобы конкретно из-за этого никто несакционированный доступ не получил?
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Witu
Witu
сообщение 4.11.2009, 1:34; Ответить: Witu
Сообщение #8


если ты делаешь авторизацию через сессии то тогда ты должен session_id() прикрутить к IP и USER AGENT а еще лучше помимо юзер агента ко всем установленным плагинам
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
kirill24
kirill24
Topic Starter сообщение 4.11.2009, 2:52; Ответить: kirill24
Сообщение #9


(HTMLandPHP @ 4.11.2009, 03:34) *
если ты делаешь авторизацию через сессии то тогда ты должен session_id() прикрутить к IP и USER AGENT а еще лучше помимо юзер агента ко всем установленным плагинам

вопросов два:
1) то есть впринципе из-за описанных мной ошибок безопасность не ухудшится?
2) если не трудно, расскажи, как сделать то о чем ты написал. хотя бы в двух словах. или хоть ссылку где почитать поподробнее
спасибо заранее
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Witu
Witu
сообщение 4.11.2009, 10:16; Ответить: Witu
Сообщение #10


в двух словах

когда пользователь авторизовался запоминаешь ип и юзер агент
ну а потом всегда сравниваешь

Замечание модератора:
Эта тема была закрыта автоматически ввиду отсутствия активности в ней на протяжении 100+ дней.
Если Вы считаете ее актуальной и хотите оставить сообщение, то воспользуйтесь кнопкой
или обратитесь к любому из модераторов.
Вернуться в начало страницы
 
Ответить с цитированием данного сообщения
Открыть тему
Тема закрыта
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0


Свернуть

> Похожие темы

  Тема Ответов Автор Просмотров Последний ответ
Открытая тема (нет новых ответов) Помогите вспомнить рекламного бота Телеграм для обмена рекламой
0 Mixatraider 728 21.2.2024, 23:42
автор: Mixatraider
Открытая тема (нет новых ответов) Посоветуйте партнерку, что бы исправить ситуацию
11 Maneja 3713 28.9.2021, 23:36
автор: NikosTM
Открытая тема (нет новых ответов) Куплю аккаунт smsc 2018 года регистрации или раньше
0 Ramires 2378 9.6.2021, 19:32
автор: Ramires
Открытая тема (нет новых ответов) Помогите рещить проблему с монитором.
Почему низкое разрешение при подключении монитора через displayport?
0 Levels 1702 11.12.2020, 0:48
автор: Levels
Открытая тема (нет новых ответов) Исправить проблему в сайте, требуется помощь программиста
1 woolf 1181 14.10.2020, 14:47
автор: WoWeb


 



RSS Текстовая версия Сейчас: 23.4.2024, 14:58
Дизайн