[NEW] Доступ к файловой системе

[artbear]

Описание проблемы, ошибки, которую надо диагностировать

При код-ревью или аудите кода необходимо проверять обращения к файлам, каталогам и набор действий, выполняемых с ними, для исключения передачи конфиденциальной или защищенной информации, а также для исключения деструктивных действий с файловой системой.
Важно проверять код от сторонних разработчиков, подрядчиков, из различных интернет-сервисов, каталогов и т.п.

По найденным замечаниям рекомендуется выполнить ручной аудит кода на предмет его правильности и безопасности.

Примеры

    Текст = Новый ЧтениеТекста("d:\win.txt", КодировкаТекста.ANSI); // есть замечание
    Текст = Новый ЗаписьТекста("d:\win.txt", КодировкаТекста.ANSI); // есть замечание

    ЗначениеВФайл("C:\Temp\PersonalData.txt", ЛичныеДанные); // есть замечание
    КопироватьФайл("C:\Temp\Order.htm", "C:\My Documents\Order.htm"); // есть замечание

    МассивИмен = Новый Массив();
    МассивИмен.Добавить("C:\Windows\Temp\Presentation.ppt.1");
    ОбъединитьФайлы(МассивИмен, "C:\Windows\Temp\Presentation.ppt"); // есть замечание

    ПереместитьФайл("C:\Temp\Order.htm", "C:\My Documents\Order.htm"); // есть замечание
    РазделитьФайл("C:\Windows\Temp\Presentation.ppt", 1024 * 1024 ); // есть замечание
    СоздатьКаталог("C:\Temp"); // есть замечание
    УдалитьФайлы("C:\temp\Works"); // есть замечание

Ссылка на источник, подтверждающее нарушение либо обоснование наличия проблемы

• Стандарт Доступ к файловой системе из кода конфигурации
• Стандарт Безопасность запуска приложений

Параметры диагностики

Тип Статья на русском

• 🐜 Ошибка
• 👮 Уязвимость
• 💂‍♂️ Потенциальная уязвимость
• 💩 Качество кода
• Другое

Важность Статья на русском

• 💔 Блокирующая / Blocker
• ❤️ Критическая / Critical
• 💛 Важная / Major
• 💙 Незначительная / Minor
• 💚 Информационная / Info
• 💞 Другое

Тэги Статья на русском

• STANDARD - "Нарушение стандартов 1С"
• LOCKINOS - "Не будет работать в другой ОС"
• SQL - "Проблема с запросом"
• PERFORMANCE - "Проблема производительности"
• BRAINOVERLOAD - "Непонятный код"
• BADPRACTICE - "Плохая практика программирования"
• CLUMSY - "Излишние действия"
• DESIGN - "Ошибка в проектировании"
• SUSPICIOUS - "Подозрительный код"
• UNPREDICTABLE - "Непредсказуемо работающий код"
• DEPRECATED - "Устаревшая функциональность"
• ERROR - "Ошибочная конструкция"
• LOCALIZE - "Проблемы локализации"

Время на исправление (минут)

3

Дополнительная информация