• X xray-16
  • Информация о проекте
    • Информация о проекте
    • Активность
    • Метки
    • Участники
  • Репозиторий
    • Репозиторий
    • Файлы
    • Коммиты
    • Ветки
    • Теги
    • Участники
    • Диаграмма
    • Сравнение
  • Задачи 216
    • Задачи 216
    • Список
    • Доски
    • Спринты
  • Запросы на слияние 31
    • Запросы на слияние 31
  • CI/CD
    • CI/CD
    • Конвейеры
    • Задания
    • Расписания
  • Развертывания
    • Развертывания
    • Окружения
    • Релизы
  • Пакеты и реестры
    • Пакеты и реестры
    • Реестр пакетов
    • Реестр контейнеров
  • Мониторинг
    • Мониторинг
    • Инциденты
  • Аналитика
    • Аналитика
    • Поток ценности
    • CI/CD
    • Репозиторий
  • Wiki
    • Wiki
  • Сниппеты
    • Сниппеты
  • Активность
  • Диаграмма
  • Создать новую задачу
  • Задания
  • Коммиты
  • Доски с задачами
Свернуть панель
  • OpenXRay
  • xray-16
  • Запросы на слияние
  • !1556

xrGame: Fix buffer overflow in `ParseControlString`

  • Ревью изменений

  • Скачать
  • Почтовые патчи
  • Простое отличие
Слиты xottab-duty запросил слияние github/fork/AMS21/fix_buffer_overflow в dev Дек 06, 2023
  • Обзор 2
  • Коммиты 2
  • Конвейеры 0
  • Изменения 1

Created by: AMS21

The given buffers are actually not big enough for the size specified in the sscanf argument.

To quote cppreference

Always stores a null character in addition to the characters matched (so the argument array must have room for at least width+1 characters)

Full credit this was found by CodeQL

Original diagnostics:

src/xrGame/console_commands_mp.cpp:606: This 'sscanf string argument' operation requires 33 bytes but the destination is only 32 bytes. [cpp/very-likely-overrunning-write] src/xrGame/console_commands_mp.cpp:606: This 'sscanf string argument' operation requires 17 bytes but the destination is only 16 bytes. [cpp/very-likely-overrunning-write]

Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/AMS21/fix_buffer_overflow