Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Скрипт не правильно проверяет логин/пароль из базы

 PHP.SU

 Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Без описания
Поиск в теме | Версия для печати
foozzi
Отправлено: 02 Декабря, 2011 - 00:02:57
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
Вот скрипт
PHP:
скопировать код в буфер обмена
  1. <?
  2. if(!isset($_POST['submit']))
  3. {
  4.         echo "<center><form action='./login.php' method=post>
  5.         <table width=200 border=1>
  6.         <tr height=30><td width=50 valign=center align=center><b>Login:</b></td><td width=50 valign=center align=center><input type=text name=login size=10></td></tr>
  7.         <tr height=30><td width=50 valign=center align=center><b>Password:</b></td><td width=50 valign=center align=center><input type=password name=pass size=10></td></tr>
  8.         </table><br>
  9.         <input type=submit name=submit value=Войти> <input type=reset value=Очистить></form>
  10.         </center>";
  11. }
  12. else
  13. {
  14.         $_POST['pass'] = md5($_POST['pass']);
  15.         $_POST['login'] = mysql_escape_string($_POST['login']);
  16.         require_once "../core/config.php";
  17.         $query = mysql_query("SELECT '.PREFIX.'admins WHERE `login` = '$_POST[login]' AND `password` = '$_POST[pass]'");
  18.         if(mysql_num_rows($query)>0)
  19.         {
  20.                 $r = mysql_fetch_array($query);
  21.                 session_start();
  22.                 session_register("admincp");
  23.                 $_SESSION['login'] = $_POST['login'];
  24.                 $_SESSION['status'] = $r['status'];
  25.                 echo "Вход выполнен успешно, сейчас вы будете автоматически перенаправлены на главную страницу.";
  26.                 echo "<meta http-equiv='refresh' content='3; URL=index.php'>"; 
  27.         } else echo "Неверный логин\пароль<br><a href='./login.php'>Назад</a>";
  28. }
  29. ?>


вот что находится в таблице admins


Ввожу все правильно, пишет что не правильно...
в чем я ошибся?


-----
90% ошибок находятся в полуметре от монитора...
 
 Top
CenapaTop
Отправлено: 02 Декабря, 2011 - 00:14:51
Post Id



Частый гость


Покинул форум
Сообщений всего: 166
Дата рег-ции: Авг. 2009  


Помог: 1 раз(а)
CODE (SQL):
скопировать код в буфер обмена
  1. SELECT '.PREFIX.'admins WHERE `login` = '$_POST[login]' AND `password` = '$_POST[pass]

select что?
CODE (SQL):
скопировать код в буфер обмена
  1. SELECT * FROM `table` WHERE ...

(Отредактировано автором: 02 Декабря, 2011 - 00:17:58)

 
 Top
foozzi
Отправлено: 02 Декабря, 2011 - 00:37:45
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
все равно не пускает


-----
90% ошибок находятся в полуметре от монитора...
 
 Top
etoYA
Отправлено: 02 Декабря, 2011 - 00:49:03
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)
Вариант 1 - при авторизации не шифруете данные из поле pass

(Отредактировано автором: 02 Декабря, 2011 - 00:50:44)

 
 Top
Panoptik
Отправлено: 02 Декабря, 2011 - 00:54:16
Post Id



Постоянный участник


Покинул форум
Сообщений всего: 2493
Дата рег-ции: Нояб. 2011  
Откуда: Одесса, Украина


Помог: 131 раз(а)
в запросе как уже вам подсказали выше напишите
CODE (PHP):
скопировать код в буфер обмена
  1.  
  2.     mysql_query('SELECT 1 `'.PREFIX.'admins` WHERE `login` = \''.$_POST[login].'\' AND `password` = \''.$_POST[pass].'\'');
  3.  

(Добавление)
кстати хеширование правильно производится? может там еще соль добавлена к мд5?


-----
Just do it
 
 Top
foozzi
Отправлено: 02 Декабря, 2011 - 01:13:58
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
соли там нет, запрос попробовал выше, тож самое.

вот как созадвал я таблицу
PHP:
скопировать код в буфер обмена
  1. $name = @$_REQUEST['name'];
  2.                 if ($name == '') $name = $login;
  3.                 mysql_query('INSERT INTO '.PREFIX.'admins SET
  4.                    login = "'.$login.'",
  5.                         status = "admin",
  6.                         password = "'.md5($pass).'"')  or die(mysql_error());


собственно вот весь установочник

PHP:
скопировать код в буфер обмена
  1.  
  2. <? require_once("../core/locale.php"); ?>
  3. <?PHP
  4. #### Если файл lock есть, выходим из установки ####
  5. if(is_file('../core/lock'))
  6.         exit($locale['i11']);
  7. ?>
  8. <?PHP
  9. #### Вывод ошибок ####
  10. ini_set('display_errors',1);
  12. #### Функция-инсталлятор ####
  13. function install_gnub() {
  14.         include_once('../core/config.php');
  15.         $shr = mysql_query('SHOW TABLES');
  16.         for ($tables=array(); $row3=mysql_fetch_array($shr); $tables[$row3[0]]=true); // Сотрим таблицы
  17.         if (!isset($tables[PREFIX.'categories']))  //Если таблиц еще не существует
  18.         {
  19. #### Создаем табличку admins ####
  20.   mysql_query ('CREATE TABLE '.PREFIX.'admins (
  21.   id tinyint(4) NOT NULL auto_increment,
  22.   login tinytext NOT NULL,
  23.   password tinytext NOT NULL,
  24.   status enum("admin") NOT NULL,
  25.   PRIMARY KEY  (`id`)
  26. ) ENGINE=MyISAM AUTO_INCREMENT=8 DEFAULT CHARSET=utf8 AUTO_INCREMENT=8;') or die(mysql_error());
  27.  
  28. #### Создаем табличку config ####
  29.   mysql_query('CREATE TABLE '.PREFIX.'config (
  30.   site_name text NOT NULL,
  31.   quotes_num tinyint(4) default NULL
  32. ) ENGINE=MyISAM DEFAULT CHARSET=utf8;') or die(mysql_error());
  33.  
  34.  
  35. #### Создаем табличку logs ####
  36.   mysql_query('CREATE TABLE '.PREFIX.'logs (
  37.   id tinyint(4) NOT NULL auto_increment,
  38.   qid tinyint(4) NOT NULL,
  39.   ip tinytext NOT NULL,
  40.   rating enum("plus","minus") default NULL,
  41.  PRIMARY KEY  (`id`)
  42. ) ENGINE=MyISAM AUTO_INCREMENT=39 DEFAULT CHARSET=utf8 AUTO_INCREMENT=39;') or die(mysql_error());
  43.  
  44.  
  45. #### Создаем табличку quotes ####
  46.   mysql_query('CREATE TABLE '.PREFIX.'quotes (
  47.   id tinyint(4) NOT NULL auto_increment,
  48.   moderator tinytext NOT NULL,
  49.   date tinytext NOT NULL,
  50.   rating tinyint(4) NOT NULL,
  51.   content text,
  52.   status enum("checked","unchecked") NOT NULL,
  53.  PRIMARY KEY  (`id`)
  54. ) ENGINE=MyISAM AUTO_INCREMENT=19 DEFAULT CHARSET=utf8 AUTO_INCREMENT=19 ;') or die(mysql_error());
  55.  
  56. #### Заносим данные админа в таблицу admins ####
  57. $name = @$_REQUEST['name'];
  58.                 if ($name == '') $name = $login;
  59.                 mysql_query('INSERT INTO '.PREFIX.'admins SET
  60.                    login = "'.$login.'",
  61.                         status = "admin",
  62.                         password = "'.md5($pass).'"')  or die(mysql_error());
  63. #### Ставим блокировку на повторную установку ####
  64. $lockt = 'lock';
  65. $lock = fopen ('../core/lock', "w");
  66. fwrite ($lock, $lockt);
  67. fclose ($lock);                        
  68. require_once("../core/locale.php");
  69. echo $locale['i9']; // Вывод сообщения об удачной установке :)
  70. }
  71.  
  72. else
  73. {
  74.         echo $locale['i10']; // Вывод сообщение об не удачной установке :(
  75.         }
  76.  
  77. }  
  78. ?>
  79.  
  80. <?PHP
  81. #### Если кнопка "Поехали" в форме нажата, заполняем массив ####
  82. if (isset($_REQUEST['install']))
  83.         {
  84.                 $config = array();
  85. #### Заполняем массив настроек тем, что пришло из формы ####      
  86.                 $config['login'] = $_REQUEST['login'];
  87.                 $config['pass'] = $_REQUEST['pass'];
  88.                 $config['servdb'] = $_REQUEST['servdb'];
  89.                 $config['userdb'] = $_REQUEST['userdb'];
  90.                 $config['passwd'] = $_REQUEST['passwd'];
  91.                 $config['namedb'] = $_REQUEST['namedb'];
  92.                 $config['PREFIX'] = $_REQUEST['prefix'];
  93. #### Берем шаблон config.php ####
  94.                 $config_tmpl = file_get_contents('config_tmpl.txt');
  95. #### обходим в цикле, заменяя части шаблона на элементы конф. массива ####
  96.                 foreach ($config as $key=>$value) $config_tmpl = str_replace("<%$key%>", $value, $config_tmpl);
  97.                 $cfg = fopen ('../core/config.php', "w");
  98.                 fwrite ($cfg, $config_tmpl);
  99.                 fclose ($cfg);
  100. #### Запуск функции установки ####
  101.                 install_gnub();
  102.         }
  103. ?>
  104. <center>
  105. <form action="./install.php" method="post" >
  106. <? print_r ($locale['i1']); ?><br><input type="text" name="userdb" value="root" /><br>
  107. <? print_r ($locale['i2']); ?><br><input type="text" name="namedb" value="gnub" /><br>
  108. <? print_r ($locale['i3']); ?><br><input type="text" name="passwd" value="passwd" /><br>
  109. <? print_r ($locale['i4']); ?><br><input type="text" name="servdb" value="localhost" /><br>
  110. <? print_r ($locale['i5']); ?><br><input type="text" name="prefix" value="gnub_" /><br>
  111. <? print_r ($locale['i6']); ?><br>
  112. <? print_r ($locale['i7']); ?><br><input type="text" name="login" value="admin" /><br>
  113. <? print_r ($locale['i8']); ?><br><input type="text" name="pass" value="pass" /><br>
  114. <input type="submit" name="install" value="Поехали" /><br>
  115. </form>
  116. </center>
  117.  

(Отредактировано автором: 02 Декабря, 2011 - 01:40:12)



-----
90% ошибок находятся в полуметре от монитора...
 
 Top
foozzi
Отправлено: 02 Декабря, 2011 - 03:26:32
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
Вычислил траблу, запрос был не виноват
CODE (SQL):
скопировать код в буфер обмена
  1.         $query = mysql_query("SELECT * FROM  '.PREFIX.'`admins` WHERE `login` = '$_POST[login]' AND `password` = '$_POST[passwd]'");

все дело было в константе PREFIX , когда я прописал полное название базы с префиксом, все заработало.

а теперь вопрос, как мне константу префикса вставить в запрос?


-----
90% ошибок находятся в полуметре от монитора...
 
 Top
filkeith
Отправлено: 02 Декабря, 2011 - 08:14:14
Post Id



Гость


Покинул форум
Сообщений всего: 112
Дата рег-ции: Июль 2011  


Помог: 2 раз(а)
foozzi пишет:
как мне константу префикса вставить в запрос?

мб я не все посмотрел но разве у переменных не должно быть значка $ доллара перед именем ?
т.е.
PHP:
скопировать код в буфер обмена
  1. $query = mysql_query("SELECT * FROM  '.$PREFIX.'`admins` WHERE `login` = '$_POST[login]' AND `password` = '$_POST[passwd]'");
 
 Top
Мелкий Супермодератор
Отправлено: 02 Декабря, 2011 - 08:15:56
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)
foozzi, подсветка форума намекает.
Спойлер (Отобразить)

(Добавление)
filkeith пишет:
мб я не все посмотрел но разве у переменных не должно быть значка $ доллара перед именем ?

Переменная != константа.


-----
PostgreSQL DBA
 
 Top
filkeith
Отправлено: 02 Декабря, 2011 - 08:20:39
Post Id



Гость


Покинул форум
Сообщений всего: 112
Дата рег-ции: Июль 2011  


Помог: 2 раз(а)
Мелкий пишет:
Переменная != константа.

Ясно, не всё просмотрел
тогда мб не хватает точек раз строка
PHP:
скопировать код в буфер обмена
  1. $query = mysql_query("SELECT * FROM  '.PREFIX.'`admins` WHERE `login` = '.$_POST[login].' AND `password` = '.$_POST[passwd].';");
 
 Top
foozzi
Отправлено: 02 Декабря, 2011 - 08:26:38
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
не выходит так, скуля не туда лезит, пробовал даже так:
PHP:
скопировать код в буфер обмена
  1.     $query = mysql_query("SELECT * FROM ` .PREFIX.admins` WHERE `login` = '.$_POST[login].' AND `password` = '.$_POST[passwd].';");


-----
90% ошибок находятся в полуметре от монитора...
 
 Top
Panoptik
Отправлено: 02 Декабря, 2011 - 08:44:08
Post Id



Постоянный участник


Покинул форум
Сообщений всего: 2493
Дата рег-ции: Нояб. 2011  
Откуда: Одесса, Украина


Помог: 131 раз(а)
так точно не получится, у вас константа в строке интерпретируется как обычное слово. вообще пробуйте вывести значение константы и строку квери перед запросом и посмотрите что туда попадает
PHP:
скопировать код в буфер обмена
  1.  
  2. define(PREFIX,'myprefix');
  3. //..... php_code ....
  4.  
  5. echo PREFIX;
  6. echo 'SELECT * FROM `' .PREFIX.'admins` WHERE `login` = \''.$_POST[login].'\' AND `password` = \''.$_POST[passwd].'\';'
  7.  


и вы опять поубирали кавычки вокруг строковых значений, а это к добру не приведет.
попробуйте вывести результат. если визуально запрос вроде правильный, выполните его в phpmyadmin или непосредственно в консоли mysql
и вообще выводите echo mysql_error() , мускул сам подскажет что не так, если допущена ошибка


-----
Just do it
 
 Top
DeepVarvar Супермодератор
Отправлено: 02 Декабря, 2011 - 08:53:06
Post Id



Активный участник


Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008  
Откуда: Альфа Центавра


Помог: 353 раз(а)
PHP:
скопировать код в буфер обмена
  1. $query = mysql_query("SELECT * FROM `".PREFIX."admins` WHERE `login` = '".$_POST['login']."' AND `password` = '".$_POST['passwd']."';");

Мужики запутались в кавычках...

(Отредактировано автором: 02 Декабря, 2011 - 08:53:55)

 
 Top
foozzi
Отправлено: 02 Декабря, 2011 - 08:59:51
Post Id



Посетитель


Покинул форум
Сообщений всего: 283
Дата рег-ции: Май 2011  
Откуда: rm -rf /


Помог: 0 раз(а)
DeepVarvar помогло спасиб) в репу +


-----
90% ошибок находятся в полуметре от монитора...
 
 Top
Поиск в теме | Версия для печати
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
  



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB