Отмена регистрации глобалов?

Да, этот код выглядит странно:

foreach ($GLOBALS[$value] as $key => $var) {
        if ($var === $GLOBALS[$key]) {//<- ?
            unset($GLOBALS[$key]);
         }
     }

Вы можете имитировать происходящее, используя простой массив, и убедиться, что этот if абсолютно бесполезен:

<?php
$data = array(1,2,3);

foreach ($data as $key => $var) {
    var_dump($var);
    var_dump($data[$key]);
    if ($var === $data[$key]) {
        unset($data[$key]);
    }
}

вывод:

int(1)
int(1)
int(2)
int(2)
int(3)
int(3)

Как видите, значения каждый раз равны, и в итоге $ data будет пустым.

Обновлять

Вот сценарий, который вы можете воспроизвести на своем компьютере:

<?php
extract($_REQUEST);

var_dump($auth);
$array = array('_REQUEST', '_SESSION', '_SERVER', '_ENV', '_FILES');
foreach ($array as $value) {
    if(isset($GLOBALS[$value]))
    {
        foreach ($GLOBALS[$value] as $key => $var) {
            unset($GLOBALS[$key]);
        }
    }
}
var_dump($auth);

Называя это так: http://site/script.php?auth=1

Это дает мне следующий результат:

string(1) "1"
<br />
<b>Notice</b>:  Undefined variable: auth on line 14
NULL

Итак, похоже, что эта переменная auth была уничтожена. Но массив $ GLOBALS по-прежнему содержит много данных.

Обновление 2

Думаю, здесь была наша ошибка:

unset($GLOBALS[$key]);

Почему мы сбрасываем ключ внешнего массива. Если вы сделаете это так:

$array = array('_REQUEST', '_SESSION', '_SERVER', '_ENV', '_FILES');
foreach ($array as $value) {
    if(isset($GLOBALS[$value]))
    {
        foreach ($GLOBALS[$value] as $key => $var) {
            unset($GLOBALS[$value][$key]);
        }
    }
}

Он очистит $ _REQUEST, $ _SESSION и все остальные необходимые массивы.

Глобальные таблицы (особенно $ _GET, $ _POST, $ _COOKIE, $ _ENV, ... и $ GLOBALS-Entries этого суперглобала) изначально заполняются PHP и никогда не должны перезаписываться или удаляться каким-либо приложением.

Любая манипуляция суперглобальными значениями снижает подлинность текущего запроса и работает против преднамеренного поведения PHP.

Вложенный foreach означает, что данный массив будет повторяться (вне foreach), и каждая запись также будет повторяться. Ключ-массив используется как $ key, а значение массива как $ var.

Теперь, когда я смог увидеть, как это работает на самом деле, я смог осознать это. По сути, если пользователи используют PHP 5.3.0 или ниже и включен 'register_globals', хакеры могут запрашивать общие имена переменных, такие как 'authorized', 'username' и т. Д., Чтобы ввести собственные значения, если переменная уже была определена разработчиком. .

Пример:

if( is_authorized() ) {
    $auth = 1;
}

if( $auth ) {
    // do authorization code here!!!
} else {
    show_login();
}

Вы можете видеть, что $ auth определен условно. Если хакер приходит и пробует что-то вроде index.php? Auth = 1 или index.php? Auth = true, то PHP зарегистрирует это как глобальное значение, оно будет проверено в условном выражении, и наш хакер получит доступ. к приложению.

Я нашел это...

foreach ($array as $value) {
    foreach ($GLOBALS[$value] as $key => $var) {
        if ($var === $GLOBALS[$key]) {
            unset($GLOBALS[$key]);
        }
    }
}

... на самом деле сбрасывает эти значения, и у хакера не будет доступа, однако, как я ранее думал, есть также ошибка. У меня есть два глобальных объекта в моей структуре, называемые $ app_dir и $ sys_dir, указывающие на два важных каталога. Если хакеры придут и скажут что-то вроде index.php? App_dir = application & sys_dir = system, тогда он фактически сбросит эти глобальные переменные из $ GLOBALS. Это создает потенциальную опасность внутри моего фреймворка для всех, кто использует его на PHP 5.3.0 или ниже.

В основном это работает с помощью условных проверок, чтобы увидеть, совпадает ли $ var (наше значение, 'system') с тем же значением, что и внутри $ GLOBAL ['sys_dir']. Кто-нибудь знает, как это исправить?