How to become an author
Search
Write a publication
Pull to refresh

Comments 44

Баг проявляется при включенном magic_quotes_gpc

Вобщем-то переведу подробней: Даже если в конфиге указано, что директива включена — она не включается.
Total votes 7: ↑7 and ↓0+7
Спасибо, поправил.
This comment wasn’t rated yet0
Тогда это не баг, это фича :)
Total votes 32: ↑32 and ↓0+32
плюс один! я тоже это подумал первым делом
Total votes 7: ↑3 and ↓4-1
Да уж, да уж :))) Точно кто-то из девелоперов почесал репу и поставил if (ini_get(«magic_quotes_gpc») == «on») ini_set(«magic_quotes_gpc», «off»); чтобы неповадно было этот костыль юзать.
Total votes 1: ↑0 and ↓1-1
Ничего собственно и страшного =)
Даже хорошо, что эта опция не включается!
Total votes 17: ↑16 and ↓1+15
Давайте сначала суперсекретный чит-код, накроющий медным тазом все хостинги с версией PHP ниже 5.3, а уж потом… :)
Total votes 3: ↑3 and ↓0+3
бензопилу вам в руки и iddqd в помощь )))
Total votes 2: ↑2 and ↓0+2
+1, давно пора бы уже, юникода хочется нормального
This comment wasn’t rated yet0
Было бы неплохо, если нельзя было сделать register_globals = On
Total votes 7: ↑7 and ↓0+7
Ждите 6й… там и это есть
This comment wasn’t rated yet0
помоему какраз и хорошо получилось.
Total votes 1: ↑1 and ↓0+1
однозначно, это была фича!!! хватит бороться с этими кавычками On/Off — задолбало!
Total votes 5: ↑5 and ↓0+5
ну и за что его заминусовали? человек правильно говорит — из-за этих директив и рождается куча будло-кодеров…
Total votes 1: ↑1 and ↓0+1
Да нет, тут имхо обратная взаимосвязь, не из-за директив рождаются быдлокодеры, а из-за быдлокодеров вводят директивы :)
Total votes 6: ↑6 and ↓0+6
не знаю как было в истории, но я вижу что быдлокодеры пишут код дома — где все с этими директивами итд :) и тут бац — на хостинге уже не то… короче начинается ужас :) скрипты не работают, или работают, но не так как надо )
This comment wasn’t rated yet0
Ну тех кто вообще не знаком с такой вещью как sql-injection, а таких людей на самом деле немало :) — магические кавычки худо-бедно спасают, так что в какой-то степени она оправдывает свое существование. Насчет register_globals конечно ничего такого сказать нельзя, ну да она теперь и отключена по умолчанию.
This comment wasn’t rated yet0
Всегдя использую функцию httpRequest и не думаю что там на сервере стоит
function httpRequest($var, $def = null)
{
$res = null;
if (isset($_POST[$var])) $res = $_POST[$var];
elseif (isset($_GET[$var])) $res = $_GET[$var];
if (isset($res) && get_magic_quotes_gpc())
if (is_array($res))
foreach ($res as $key => $value)
$res[$key] = stripslashes($value);
else $res = stripslashes($res);
return isset($res)? $res: $def;
}
Total votes 1: ↑1 and ↓0+1
Что-то как-то очень замудрено :) По-моему проще тогда уж обертку сделать, а-ля:
class MagicWrapper extends ArrayObject {

	protected $data = array();

	public function __construct(&$data) {
		if(get_magic_quotes_gpc()) {
			$this->data = $data;
			$data = $this;
		}
	}

	public function offsetGet($name) {
		return $this->strip($this->data[$name]);
	}

	public function strip($data) {
		if(is_array($data)) {
			return array_map(array($this, 'strip'), $data);
		} elseif(is_string($data)) {
			return stripslashes($data);
		} else {
			return $data;
		}
	}

}
new MagicWrapper($_GET);
new MagicWrapper($_POST);

или при старте скрипта фильтровать все кавычки, а я лично остановился на том что отключаю оную директиву в .htaccess
Total votes 1: ↑1 and ↓0+1
Тьфу, кривая обертка :) В конструкторе вместо
$this->data = $data;

нужно вызывать родительский метод
parent::__construct($data);

а в offsetGet вместо
return $this->data[$name];

использовать
$this->strip(parent::offsetGet($name));

а то что-то я так и не определился, отнаследовался от готового объекта, а вместо логику стал писать как для реализации интерфейсов :)
Total votes 1: ↑1 and ↓0+1
function noMagic()
{
	if (get_magic_quotes_gpc())
	{
		if (!empty($_GET)) $_GET = stripMagicQuotes($_GET);
		if (!empty($_POST)) $_POST = stripMagicQuotes($_POST);
		if (!empty($_REQUEST)) $_REQUEST = stripMagicQuotes($_REQUEST);
		if (!empty($_COOKIE)) $_COOKIE = stripMagicQuotes($_COOKIE);
    }
}

function stripMagicQuotes($arr)
{
    foreach ($arr as $k => $v) {
        $arr[$k] = is_array($v) ? stripMagicQuotes($v) : stripslashes($v);
    }
    return $arr;
}

noMagic();


Или такой вариант.
Total votes 1: ↑1 and ↓0+1
Ну да, это я и имел в виду под «при старте скрипта фильтровать все кавычки», тут впринципе есть простор для обсуждений, с одной стороны обертка в виде функции или-же объекта будет фильтровать данные снова и снова, с другой обработка при старте чревата тем что понадобиться может только часть данных, а обработаны все равно будут все.
This comment wasn’t rated yet0
>>Решение — откатиться на PHP 5.2.6 или ожидать PHP 5.2.8.
правильно: Решение — откатиться на PHP 5.2.6 и ожидать PHP 5.2.8.
Total votes 1: ↑0 and ↓1-1
а еще правильнее: переписать код, чтоб работал с magic_quotes_gpc=off :)
Total votes 4: ↑4 and ↓0+4
Имелась в виду ошибка перевода.
Но вообще — предложенное вами решение более подходящее :)
This comment wasn’t rated yet0
Спасибо, поправил
This comment wasn’t rated yet0
Вот только в данном случае все решения основанные на if(get_magic_quotes_gpc()) будут не корректными, т.к. при включённом magic_quotes функция вернёт true, а stripslashes затем уберёт нужные слеши.
Total votes 2: ↑2 and ↓0+2
magic_quotes_gpc == ЗЛО
Все данные надо обрабатывать ручками =)
Total votes 1: ↑1 and ↓0+1
Правда ваша, бороться с SQL Injection нужно при формировании запроса к БД, а не получении POST/GET параметров
Total votes 1: ↑1 and ↓0+1
Мне кажется, вопрос о вреде magic_quotes_gpc уже давно решён на всех уровнях. Однако каждому надо высказать своё мнение об этом. Все комментарии в плюсах, но в самом деле что-нибудь полезное говорит редкий комментатор.
Большую часть комментариев можно заменить фразой «magic_quotes — зло».
Это даже не холивар — здесь только одно мнение. Иначе, как словесным поносом, это назвать сложно.
Total votes 1: ↑0 and ↓1-1
«Кто понял жизнь, тот не спешит» ©
отличный урок для того, кто гониться за самыми последними и свежими версиями продуктов.

Хотя, канешн очень не приятно, что такой баг пропустили…
This comment wasn’t rated yet0
Человек умирает. Обидно что такую оплошность при создании допустили. ;)
This comment wasn’t rated yet0
Холивара никто не хочет. Просто Руби всяко лучше. :)
Total votes 3: ↑0 and ↓3-3
1) Баги есть везде.
2) Рубе тормоз т.к. это не VM. Со второй версией будет лучше.
3) Пишите на питоне.
This comment wasn’t rated yet0
Согласен. В рельсах есть!
Total votes 1: ↑0 and ↓1-1
Sign up to leave a comment.

Articles

Show the best of all time
Unlock dark mode

Your account

Sections

Information

Services

Support
© 2006–2024, Habr