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 :: Вставить переменные в JS

 PHP.SU

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


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

> Без описания
Поиск в теме | Версия для печати
vadim2
Отправлено: 30 Июля, 2010 - 15:31:44
Post Id


Новичок


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


Помог: 0 раз(а)
Подскажите, что делаю не так :
PHP:
скопировать код в буфер обмена
  1.  
  2. <?PHP
  3. if(!$_get['page']){
  4. echo '<form action="./index.php?page=pay" method="POST">
  5. <input type="text" name="amount">
  6. <input type="submit" value="Pay">
  7. </form>';
  8. }
  9. else
  10. {
  11. if($_get['page']=='pay'){
  12. $id='192';
  13. $time=date('d/n/y H:i');
  14. echo '<script src=\'http://mtgox.com/js/BTCButton.js\' ></script>
  15. <script> var mtgoxOptions = {
  16. merchID: "45",
  17. amount: "'.$_POST['amount'].'",
  18. custom: "Pay from '.$id.', at '.$time.'" };
  19. </script>
  20. <a style="cursor:pointer;" onClick="mtgox_pay(this)" >Pay using Bitcoins!</a> ';
  21. }
  22. else
  23. {
  24. echo 'something wrong!';
  25. }
  26. }
  27. ?>
 
 Top
JustUserR
Отправлено: 30 Июля, 2010 - 15:46:12
Post Id



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


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


Помог: 17 раз(а)
vadim2 пишет:
Подскажите, что делаю не так :
Вероятно интерполируемые вами строки содержат определенные спецсимволы например двойные кавычки или обратный слеш - так как вы не производите дополнительного экранирования то они получаются на одном и том же уровня в JS-кодом - и соответственно могу вызывать синтаксическую ошибку в нем Кроме того такая прямая интерполяция POST-значения может привести не только к синтаксической ошибке - но и достаточно заметному эксплойту поскольку имеется возможность вставить любой HTML-код в страницу


-----
Сделать можно все что угодно - нужно только старание, терпение и хороший поисковик Улыбка
Безлимитный web-хостинг от 15 рублей за 40 МБ дискового пространства - http://ihost[dot]oks71[dot]ru/
 
 Top
Реф
Отправлено: 01 Августа, 2010 - 06:10:41
Post Id



Частый гость


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


Помог: 0 раз(а)
PHP регистрозависимый язык, поэтому массивы $_get, $_gET и $_GET - разные массивы. В Вашем случае нужно использовать $_GET


-----
Думать надо головой, а не головкой!
 
 Top
JustUserR
Отправлено: 01 Августа, 2010 - 13:23:58
Post Id



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


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


Помог: 17 раз(а)
Реф пишет:
PHP регистрозависимый язык, поэтому массивы $_get, $_gET и $_GET - разные массивы. В Вашем случае нужно использовать $_GET
В любом случае первым делом необходимо прокэкранировать интерполируемую в исходный JS-код переменную $_POST['amount'] - без этого PHP-скрипт подвержен не только возможной XSS-уязвимости но и при подстановки неправильных значений может попросту не работать
vadim2 Даже если ваша форма на HTML-странице которая совершает POST-запрос имеет проверку вводимых данных с помощью специального JS-скрипта - то вы должны учитывать что передать данные обрабатывающему PHP-скрипту можно передать из поддельной форму или попросту через telnet


-----
Сделать можно все что угодно - нужно только старание, терпение и хороший поисковик Улыбка
Безлимитный web-хостинг от 15 рублей за 40 МБ дискового пространства - http://ihost[dot]oks71[dot]ru/
 
 Top
Реф
Отправлено: 01 Августа, 2010 - 15:58:30
Post Id



Частый гость


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


Помог: 0 раз(а)
JustUserR
Это тоже верно, но в ту часть, где происходит формирвоание JS не попасть. Условие $_get['page']=='pay' всегда будет возвращать ложь.


-----
Думать надо головой, а не головкой!
 
 Top
DeepVarvar Супермодератор
Отправлено: 01 Августа, 2010 - 16:40:57
Post Id



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


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


Помог: 353 раз(а)
Что-то вроде:
PHP:
скопировать код в буфер обмена
  1. <?PHP
  2. if(!$_POST['page']) {
  3.         echo '
  4.         <form action="./index.php" method="post">
  5.         <input type="hidden" name="page" value="pay">
  6.         <input type="text" name="amount">
  7.         <input type="submit" value="Pay">
  8.         </form>';
  9.         }
  10. else {
  11.         if($_POST['page'] == 'pay') {
  12.                 $id='192';
  13.                 $time=date('d/n/y H:i');
  14.                 $amount = $_POST['amount'];
  15.                 echo '
  16.                 <script src="http://mtgox.com/js/BTCButton.js"></script>
  17.                 <script type="text/javascript">
  18.                         var mtgoxOptions = {
  19.                                 merchID: "45", amount: "'.$amount.'", custom: "Pay from '.$id.', at '.$time.'"
  20.                                 }
  21.                 </script>
  22.                 <a style="cursor:pointer;" onClick="mtgox_pay(this);">Pay using Bitcoins!</a>';
  23.                 }
  24.         else {
  25.                 echo 'something wrong!';
  26.                 }
  27.         }
  28. ?>
 
 Top
JustUserR
Отправлено: 02 Августа, 2010 - 00:56:03
Post Id



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


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


Помог: 17 раз(а)
Реф пишет:
JustUserR Это тоже верно, но в ту часть, где происходит формирвоание JS не попасть. Условие $_get['page']=='pay' всегда будет возвращать ложь
Да при использовании конкретной реализации PHP-скрипта приведеная уязвимость не будет представлять опасности для web-сайта - поскольку дейтсвительно секция PHP-скрипта в которой она располагается не может быть выбрана Тем не менее найденные ошибки и уязвимости лучше исправлять сразу при обнаружении - поскольку если например в данном PHP-скрипте исправить условие отвечающее за выбор выводимых секций - то сразу же появляется опасность взлома web-сайта которую достаточно просто обнаружить Дейтсивтельно при POST-запросе соответствующий секции данного PHP-скрипта и передаче в него параметров по аналогии с оригинальной формой - можно легко обраружить что в JS-скрипт напрямую интерполируется значение параметра $_POST['amount'] даже без обычного обрамления в один уровень кавычек Поэтому в данную переменную достаточно передать значение типа " } \n CODE \n mtgoxOptions= {merchID: mtgoxOptions.merchID, amount: "somevalue - и это дает возможность исполнять на генерируемой HTML-страница абсолютно любой JS-код которые следует расположить в указанной позиции - при этом из-за сохраненя уровня вложенности кавычек и экранирования с ним даже не нужно выполнять никакие дополнительные преобразования
DeepVarvar Приведенные вами вариант PHP-скрипта также содержит указанную уязвимость - которая также при передаче неправильных параметров может приводить к синтаксической ошибка в JS-скрипте Для решения этой проблемы стоит произвести escape- и HTML-экранирование переменной $_POST['amount'] - причем второй вид экранирования также нужен для защиты от ввода соответствующего завершающего HTML-тега наличие которого также приведет к синтаксической ошибке


-----
Сделать можно все что угодно - нужно только старание, терпение и хороший поисковик Улыбка
Безлимитный web-хостинг от 15 рублей за 40 МБ дискового пространства - http://ihost[dot]oks71[dot]ru/
 
 Top
Поиск в теме | Версия для печати
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Программирование на PHP »


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



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

 
Powered by ExBB FM 1.0 RC1. InvisionExBB