2

Есть ли уязвимости в данном коде?

if (isset($_COOKIE['lang'])) {

    $language = $_COOKIE['lang'];

    $result = "en";

    if (in_array($language, $LANGS)) {

        $result = $language;
    }

    @setcookie("lang", $result, time() + 14 * 24 * 3600, "/");
    include_once("../sys/lang/".$result.".php");

}  else {

    $language = "en";

    if (isset($_SERVER['HTTP_ACCEPT_LANGUAGE'])) {

        $language = substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2);
    }

    $result = "en";

    if (in_array($language, $LANGS)) {

        $result = $language;
    }

    @setcookie("lang", $result, time() + 14 * 24 * 3600, "/");
    include_once("../sys/lang/".$result.".php");
}
1
  • Вас интересует только безопасность? Вас не смущает, что в коде совершенно зря дублируются одни и те же действия?
    – A K
    2 окт 2018 в 18:57

3 ответа 3

2

Проверяйте куку не на существование, а на пустоту. Плюс размер вашего кода можно уменьшить.

Готовый вариант:

$result = 'en';

if(!empty($_COOKIE['lang']) && in_array($_COOKIE['lang'], $LANGS)) {
    $result = $_COOKIE['lang'];
}else if(!empty($_SERVER['HTTP_ACCEPT_LANGUAGE'])) {
    $language = substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2);
    if (in_array($language, $LANGS)) $result = $language;
}

setcookie('lang', $result, time() + 14 * 24 * 3600, '/');
include_once('../sys/lang/'.$result.'.php');
3
  • undefined index.
    – Naumov
    2 окт 2018 в 15:08
  • @Total Pusher благодарю за пример, правда не понятно, как он относится к обсуждаемому выше коду? В скрипте стоит проверка на существование переменной и обратиться к ней можно только, если она установлена
    – skarb
    2 окт 2018 в 20:03
  • Да, вы правы. Почему-то думал, что empty при обращении к несуществующему хешу выдает ворнинг. Не выдает. 2 окт 2018 в 20:13
-1

Не понятно что вы хотели бы услышать, но конечно же не безопасно, простой кайс взлома.

  1. Пишем скрипт test.php
  2. Аплодим на сайт через форму
  3. Находим coockie lang='../../../uploads/test.php'
  4. Success!

Содержимое test.php

<?php
  echo 'test';
6
  • 1
    щито? 1) Где вы тут форму нашли? 2) Как вы что-то искать собрались? 3) Если вы сможете через форму загрузить пхп скрипт, тут дело не в этом куске а в форме.
    – Manitikyl
    2 окт 2018 в 13:00
  • @Manitikyl Шыре мыслите вы даёте возможность исполнить любой скрипт на сервере через include, а это уже дыра.
    – Naumov
    2 окт 2018 в 14:00
  • @Naumov почему любой? Там ведь стоит проверка in_array. Т.е. могут инклудится только те скрипты, которые разработчиком прописаны в массиве $LANGS
    – skarb
    2 окт 2018 в 14:29
  • @skarb через полгода уберут проверку и все, что бы автоматом includit'ь
    – Naumov
    2 окт 2018 в 15:11
  • @Naumov ха)) Признали бы уже ошибку свою, вместо введения в замешательство. Следуя вашей логике, для надежной безопасности сайт вообще следует перенести на свой домашний компьютер и отключиться от интернета
    – skarb
    2 окт 2018 в 17:28
-1

Для PHP7+ можно использовать оператор объединения с null ??.

Будет так:

$result = $_COOKIE['lang'] ?? (
    isset($_SERVER['HTTP_ACCEPT_LANGUAGE']) ? substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2) : 'en'
);
if (!in_array($result, $LANGS)) $result = 'en';
setcookie('lang', $result, time() + 14 * 24 * 3600, '/');
include_once('../sys/lang/'.$result.'.php');
2
  • 1
    Кодинг стаил нервно курит, и разрабы php7+ колят глаза
    – Naumov
    2 окт 2018 в 15:09
  • @Naumov что конкретно вам не нравится в этом примере, чтобы понять свою ошибку? Сложная конструкция разбита на 2 через вынос на отдельную строку. 2 окт 2018 в 20:03

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.