Secure Code Review досліджує, як код реалізує захист і бізнес-правила. Функція може працювати для звичайного користувача та водночас неправильно перевіряти право на чужий документ. Ручний аналіз доповнює автоматичні інструменти: фахівець простежує шлях даних і перевіряє, чи відповідає поведінка задуму системи.
Почніть із критичних функцій і меж довіри
Описуємо архітектуру, ролі, цінні дані й зовнішні інтеграції. Визначаємо код, який приймає рішення про доступ, оплату, зміну прав або завантаження файлу. Для оцінки потрібен контекст: однаковий рядок може бути безпечним в одному місці й небезпечним в іншому.
Погоджуємо версію джерела та склад перевірки. Аналіз окремої зміни й огляд усієї бази мають різну глибину. Якщо залежності або конфігурація недоступні, це обмеження фіксуємо, а не підміняємо припущенням про їхню правильність.
Перевірте рішення на сервері
Прихована кнопка не забороняє прямий запит. Умовний приклад: користувач змінює ідентифікатор документа в адресі. Сервер повинен повторно перевірити право на цей конкретний об’єкт, а не лише факт входу. Аналізуємо всі відповідні шляхи, включно з експортом і API.
Вхідні дані перевіряємо за очікуваним типом і призначенням. Запити до бази, відображення HTML і робота з файлами потребують своїх захисних механізмів. Один загальний фільтр рядків не замінює правильну обробку в конкретному контексті.
Зіставте автоматичні сигнали з поведінкою
Статичний аналіз допомагає знайти підозрілі шаблони, а перевірка залежностей — відомі проблеми компонентів. Результати перевіряємо, щоб відділити реальний шлях до помилки від хибного спрацювання. Бізнес-логіка, наприклад повторне використання операції, часто потребує ручного розбору.
Окремо дивимося на секрети, журнали й повідомлення про помилки. Корисна діагностика не повинна розкривати паролі, ключі або зайві персональні дані. Звіт містить потрібний доказ, але не перетворюється на загальнодоступну копію чутливого середовища.
Передайте виправлення з перевіркою першопричини
Для знахідки вказуємо місце, умови, вплив і рекомендований принцип виправлення. Не обмежуємося косметичним приховуванням симптомів. Якщо перевірка права відсутня в спільному методі, потрібно оцінити всі місця його використання, а не лише продемонстрований маршрут.
Після зміни повторюємо сценарій і перевіряємо потрібну функцію для дозволених ролей. Передаємо залишені обмеження й рекомендації для наступних змін. Review не гарантує відсутності всіх помилок, але дає конкретні перевірені висновки для визначеного джерела.
Джерела та подальше читання
