Здравствуйте, ув. форумчане. Проблема так сказать детская, но решить её всё равно не могу. Предисловие: Имеем игровой сервер одной из популярных онлайн-игр. Так же имеет сайт, который по базе привязан к игровому серверу. На сайте есть страница со сменой пароля. Суть проблемы: На странице со сменой пароля есть уязвимость, можно изменить свой циферный и буквенный пароль, на пароль, который содержит символы * или _ . Так вот, войдя в игру с этими данными, можно полностью выбить игровой сервер, по причине того, что такие символы он не поддерживает. Самое интересно то, что страница регистрации запрещает такие символы, а вот страница смены пароля позволяет. Задача: Переписывать GameServer было бы долго и нудно. Мне кажется проще намного просто ограничить на странице смены пароля возможность введения символов * и _ . Код самой страницы: Код (Text): <?PHP } ?> <?php require('include/right_block.php'); ?> <?php require('include/slider.php'); ?> <!-- News --> <div class='content_top'> <div class='content_bottom' id='cont_styles'> <h1>Сменить пароль</h1> <div class='bord_center'> <div class='bord_top'> <div class='bord_bottom'> <div class='table_bg'> <div id='reg_bg'> <table id='reg_table'> <?PHP if(!isset($_SESSION['username'])) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Вы уже авторизованы, пожалуйста выйдите для продолжения регистрации.</center></span> </tr> </table> <?PHP } else { ?> <?php if(!isset($_SESSION["username"])) { ?> <?php die; } include 'include/config.php'; include 'include/functions.php'; If (isset($_POST['passold'])) { if (empty($_POST['passold']) OR empty($_POST['passnew'])) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Одно или несколько полей являются пустыми.</center></span> </tr> </table> <?php die; } $conn = new mysqli($ip,$sqluser,$sqlpw,$userdb); if (mysqli_connect_errno() == 0) { $user = $conn->real_escape_string(htmlspecialchars(clean_str($_SESSION["username"]))); $passold = $conn->real_escape_string(htmlspecialchars($_POST['passold'])); $passnew = $conn->real_escape_string(htmlspecialchars($_POST['passnew'])); $ip = ip(); If (strlen($passnew) < 6 OR strlen($passnew) > 12) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Пароль должен быть не мение 6-ти символов и не более 12-ти.</center></span> </tr> </table> <?php die; } If ($passold == $passnew) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Старый пароль и новый пароль одинаковы.</center></span> </tr> </table> <?php die; } $realpass = hash("sha256",$user.$salt.$passnew); $realpassold = hash("sha256",$user.$salt.$passold); if ($plain == "no") { $query = "UPDATE bg_user SET passwd = '$realpass', ip = '$ip' WHERE passwd = '$realpassold' AND user_id = '$user' ;"; $exec = $conn->query($query); If ($conn->affected_rows) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Ваш пароль успешно изменён.</center></span> </tr> </table> <?php die; } Else; { die; ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Пожалуйста, свяжитесь с Администратором!</center></span> </tr> </table> <?php } } ElseIf ($plain = "yes"); { $query = "UPDATE bg_user SET truepasswd = '$passnew', passwd = '$realpass', ip = '$ip' WHERE passwd = '$realpassold' AND user_id = '$user' ;"; $exec = $conn->query($query); If ($conn->affected_rows == 1) { ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>Успех!</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Ваш пароль был успешно изменён.<br> <br> Информация об аккаунте:<br> Старый пароль: <b><Font color=green><?php echo "".$passold.""; ?></font></b><br> Новый пароль: <b><Font color=red><?php echo "".$passnew.""; ?></font></b><br> </center></span> </tr> </table> <?php die; } Else; { echo "Error! Please contact an Administrator!!! You'll be redirected in 5 seconds... Or click "; die; ?> <table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Пожалуйста, свяжитесь с Администратором!</center></span> </tr> </table> <?php } } } } ?> <hr> Пароль состоит из английских букв и цифр. Он должен быть от 6 до 15 символов длиной.<br/> Пароль не должен содержать ЗАГЛАВНЫХ букв. <hr> <br> <form method="post" id="reg" name="post" action="chp.php"> <table id='reg_table'> <tr> <td style="width: 150px;"> Старый пароль: </td> <td> <input type="password" name="passold" title="?? ??" value="" /> </td> </tr> <tr> <td style="width: 150px;"> Новый пароль: </td> <td> <input type="password" name="passnew" title="Юї????" value="" /> </td> </tr> <tr> <td style="width: 100px;"> </td> <td> <br> <a href='#' onclick='document.getElementById("reg").submit(); return false;' class='buy_button'>Сменить</a> <br><br/> </td> </tr> </table> </form> <br> <hr> ПОМНИТЕ! Никому не говорите свои данные от учётной записи!Это может быть опасно! <hr> <?PHP Очень прошу вас помочь с этой проблемой и прислать уже готовый кусок кода, который запрещает смену пароля на пароль с содержанием символов. Заранее благодарен.
легче написать которые разрешены символы Код (PHP): <input type="text" name="name" size="42" onkeyup="this.value = this.value.replace (/[^a-zA-Z0-9]+$/, '')"> Удаляет сразу запрещенные символы при введение в строку не соответствующих a-zA-Z0-9 Код (PHP): if (preg_match("/[^a-zA-Z0-9|-|_|.|\s]/", $_POST['name'].$_POST['login'].$_POST['password']) == 1) { echo 'ААА специя!!'; } Тут ищет нет ли других символов кроме разрешенных a-zA-Z0-9 - _ . и пробел
Простите, а не могли бы вы мне в виде моего когда вставить это всё. Я настолько в этом профан, что ещё чего-то не то заменю там. Был бы очень вам благодарен.
Эхехе... У меня сервак, но я профан. Вот придти так в больницу на операцию, к тебе подходит анестезиолог и говорит - простите, отмеряйте себе дозу препарата сами, а то я профан. Ты такой - WAT? А он тебе - эт еще что, вы еще нашего хирурга не видели, а теперь спим, спим..спим....
Очень смешно. Да вот только установка, настройка сервера - это одно.С html даже дружу. И сервер мой среди себе подобных в тройке лидеров по онлайну. Но в php полный ноль. Синтаксис нарушить я не нарушу, а вот прикрутить что-то куда-то вопрос посерьёзней. Таких как ты, не особо люблю. Помочь, не помогут. Зато гонора хоть отбавляй. И стёба не совсем уместного. Если есть что сказать по теме, скажи. Если нет, зачем ходить по темам и умничать? P.S - Не поленился посмотреть написанные тобой сообщения на данном форуме. 90% - такие, как в нашем случае. Всё с тобой ясно. Помощник збс.
Last Chaos. Ну, ради такой мелочной работы нанимать человека, он сам скорее всего не согласится. Либо запросит сумму, которая не будет соответствовать проделанной работе. Если наберётся пак для исправлений, обязательно обращусь. Спасибо.
да ладно тут и за пять баксов люди могут сделать мелочь. вопрос соответствия вообще очень сложный. однако когда вы попробуете разок пересилите свой страх, то поймете что это очень круто делать всякое через знакомых хороших фрилансеров, и ваш проект будет вечно молодой и уважаемый. =) Добавлено спустя 59 секунд: но ваш вопрос да решается одной регуляркой. может ктонить глянет код. я просто в регулярках оч плох.
Человек , который, чуть что, первым делом расстегивает виртуальную ширинку и начинает размахивать виртуальной пипиркой(скачанной и установленной фришечкой), параллельно мгновенно создавая для себя портрет собеседника, автоматически признавая его достаточно точным, чтобы на его основе делать выводы, говорит другим про гонор?
давайте не бум ругаться. "широко известен в узких кругах" это старая добрая шутка. дайте человеку регулярку. =)
Код (PHP): } else; { 72 строка Добавлено спустя 32 секунды: угум Добавлено спустя 8 минут 26 секунд: вот сейчас взялся за код этот. Теперь не пойму.. он вообще рабочий? без ошибок, все пучком пашет?
Моя виртуальная пипирка приносит мне столько денег, что твоя, пардон, не виртуальная столько не видела за год. И не важно какая там фришечка и прочее. Есть масса посторонних аспектов, которые могут дать движение проекту. Поэтому здесь какие либо споры не уместны. Никаких портретов не создавал, обратился к людям за помощью, ожидая помощи, а не философских рассуждений и высмеивания моих слов. Но после того, как увидел список всех сообщений твоих на форуме сразу понял, что это твоя специализация - "пукать" в темах людей, менее обознанных чем ты. Оно разве стоит того? Добавлено спустя 36 секунд: Да, работает.
=) Ну тогда велкам в удивительный мир наемных работников. и пипирками лучше не махать по углам, а то придут из органов и спросют.
Спасибо. По поводу фрилансеров можно подумать. Вопрос то решается, но как видите, не очень он и решается. А ещё есть те, кому интересен не вопрос, а "поболтать за жизнь". Сплошной оффтоп. Добавлено спустя 25 секунд: Про органы в цвет А про наёмных работников то к чему?
нахаляву переписывать ваш код, ибо он говно от начала до конца, никто не хочет. Надо же думать, писать, себя проверять, тестировать и тп. И это дело надо проверить будет на то, как оно встроилось в вашу систему, как минимум получить структуру БД, задать десяток вопросов, сделать новую бд на рабочей машине, в ней таблицу, юзера с правами к ней, сожрать пару бутеров, ибо уже успеешь проголодаться. В это время не получится отдыхать, а люди на помощь которых вы расчитвыаете, обычно работают на работе. Чего жать-то 30-100 баксов, если вы имеете такие колоссальные доходы?
Насколько я знаю, эта тема, как и подфорум созданы для подобных обращений. И в них никто абсолютно не говорит об оплате и прочем. Если я буду заинтересован в покупке, я обращусь в соответствующую тему, на сайт или к человеку. В моём субъективном понимании - это выглядит, как элементарное исправление строки с разрешенными символами, как пример то, что человек выше представил, но не объяснил как и куда прикрутить его к общему коду. Да и цена такому исправлению я не думаю, что варьирует в тех суммах, которые описали вы.
Код (PHP): <?PHP require('include/right_block.php'); require('include/slider.php'); echo ' <div class=\'content_top\'> <div class=\'content_bottom\' id=\'cont_styles\'> <h1>Сменить пароль</h1> <div class=\'bord_center\'> <div class=\'bord_top\'> <div class=\'bord_bottom\'> <div class=\'table_bg\'> <div id=\'reg_bg\'> <table id=\'reg_table\'>'; if(!isset($_SESSION['username'])) { echo '<table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Вы не авторизованы, пожалуйста войдите для продолжения.</center></span> </tr> </table>'; } else { include ('include/config.php'); include ('include/functions.php'); $conn = new mysqli($ip,$sqluser,$sqlpw,$userdb); if (mysqli_connect_errno() == 0) { $user = $conn->real_escape_string(htmlspecialchars(clean_str($_SESSION["username"]))); $passold = $conn->real_escape_string(htmlspecialchars($_POST['passold'])); $passnew = $conn->real_escape_string(htmlspecialchars($_POST['passnew'])); $ip = ip(); $realpass = hash("sha256",$user.$salt.$passnew); $realpassold = hash("sha256",$user.$salt.$passold); if ($_POST['passold'] && $_POST['passnew'] && strlen($passnew) < 6 && strlen($passnew) > 12 && (strtoupper($passold) != strtoupper($passnew)) && preg_match("/[^a-z0-9]/", $passold.$passnew) == 0) { $query = "UPDATE bg_user SET truepasswd = '$passnew', passwd = '$realpass', ip = '$ip' WHERE passwd = '$realpassold' AND user_id = '$user' ;"; $exec = $conn->query($query); if ($conn->affected_rows == 1) { echo '<table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>Успех!</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Ваш пароль был успешно изменён.<br> <br> Информация об аккаунте:<br> Старый пароль: <b><Font color=green>'.$passold.'</font></b><br> Новый пароль: <b><Font color=red>'.$passnew.'</font></b><br> </center></span> </tr> </table>'; } else { echo '<table class="p_table" cellspacing="2" cellpadding="0" border="0"><tr><td align="left" valign="top" class="rtt" colspan="2"> <div class="tct size16 pbot ptop"><center><font color=red>ОШИБКА</font></center></div> <tr class="bgh"><td align="left" valign="top" class="rtt p5"><span class="tct"><center>Старый пароль неверен!</center></span> </tr> </table>'; } } else { echo '<hr> Пароль состоит из английских букв и цифр. Он должен быть от 6 до 15 символов длиной.<br/> Пароль не должен содержать ЗАГЛАВНЫХ букв. <hr> <br>'. ((preg_match("/[^a-z0-9]/", $passold.$passnew) == 1)?'<span style="color:red;">Спец символы...</span><br>':'') .'<form method="post" id="reg" name="post" action="chp.php"> <table id=\'reg_table\'> <tr> <td style="width: 150px;"> Старый пароль: </td> <td> <input type="password" name="passold" maxlength="15" value="" />'. ((empty($_POST['passold']) && $_POST['submit'])?'<br><span style="color:red;">Введите старый пароль</span>':'').'</td> </tr> <tr> <td style="width: 150px;"> Новый пароль: </td> <td> <input type="password" name="passnew" maxlength="15" value="" />'. ((empty($_POST['passnew']) && $_POST['submit'])?'<br><span style="color:red;">Введите новый пароль</span>':''). ((strtoupper($passold) == strtoupper($passnew))?'<br><span style="color:red;">Новый пароль совпадает со старым</span>':'').'</td> </tr> <tr> <td style="width: 100px;"> </td> <td> <br> <input type="submit" name="submit" onclick=\'document.getElementById("reg").submit(); return false;\' class=\'buy_button\' value="Сменить"> <br><br/> </td> </tr> </table> </form> <br> <hr> ПОМНИТЕ! Никому не говорите свои данные от учётной записи!Это может быть опасно! <hr>'; }} else { echo 'Mysql error';}}
Нет, не правда =) Этот форум не создан для халявного кода. Вы получили ответ на свой вопрос, вы ждете код, сами разбираться не желаете, фактически вы ждете бесплатной работы, которую проделает другой человек. =) А раздел форума как и весь форум посвещен решению вопросов по программированию. Ваше решение - использовать регулярку. Решение получено, топик можно закрывать? =) Добавлено спустя 1 минуту 58 секунд: В моем субъективном понимании вы объективно не понимаете что надо сделать. вы просите мнение эксперта - получено. Вы просите проделать работу за вас - думаю и это получено. первое - это то зачем нужен форум. Второе - это то, за что платят деньги. Добавлено спустя 47 секунд: пока вы заинтересованы в халяве? Добавлено спустя 4 минуты 15 секунд: То есть для такого воротилы, как вы, 30 баксов это прям ацкая сумма? =) или вы думаете, что если вам нужна одна строка, то она должна стоить дешево? а то что код, защищенный от инъекций стоит денег и времени - это вы пропустили мимо ушей, когда я расписывал выше? или вы надеетесь, что если в этом месте добавят эту проверку, то к вам не залезут через другую дыру, коих у вас миллион? =) Вы даже не понимате, почему у вас звездочка порождает проблемы. Фактически, вы хотите избавиться от звездочки, не закрывая саму дыру. Звездочку вы отфильтруете. С запятыми, кавычками, двойными кавычками, процентами, управляющими последовательностями других типов идущими за слешем, со словами-коммандами к бд вы разбираться будете каждый раз только после взлома?
У человека есть 2 типовые формы и 2 скрипта. Одна форма со скриптом работает правильно, в другой баг. То есть у человека есть на руках готовый вариант решения. Это все равно что имея готовый бумажный самолетик, не собрать из нового листа еще один по аналогии. Это уже показатель, ставший причиной, собственно, моих хохм И вот это вот... Не, ну как можно относиться всерьез человеку с таким воспаленным самолюбием, что от любого прикосновения начинается боль пониже спины? Да, я люблю похохмить, потому что, увы, куча постов только на хохму и пробивают. Если же видно адекватного человека, у которого заминка, то тут грех не помочь, что, тоже делаю с удовольствием. Но не тот случай.