# V5013. OWASP. Storing credentials inside source code can lead to security issues.

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

Хранение учетных данных в исходном коде может привести к нарушению контроля доступа к данным и возможностям, не предназначенным для публичного использования. Имея доступ к сборке, злоумышленник при помощи дизассемблера сможет увидеть все строковые литералы, которые в ней используются. В случае open-source проектов всё ещё проще, так как злоумышленник может изучать непосредственно исходный код.

Таким образом, все секретные данные могут оказаться публично доступными. Уязвимости, связанные с недостаточной защищённостью конфиденциальных данных, выделены в отдельную категорию рисков в OWASP Top 10 Application Security Risks 2017: A2:2017-Broken Authentication.

Рассмотрим пример:

    bool LoginAsAdmin(const std::string &userName,
                      const std::string &password)
    {
      if (userName == "admin" && password == "sRbHG$a%")
      {
        ....
        return true;
      }

      return false;
    }

В указанном примере пароль, используемый для входа в систему с правами администратора, хранится в коде. И злоумышленник может легко получить данные для авторизации, а, следовательно, все возможности администратора системы.

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

    bool LoginAsAdmin(const DataStorage &secretStorage,
                      const std::string &userName,
                      const std::string &password)
    {
      var adminData = secretStorage.GetAdminData();

      if (   userName == adminData.UserName
          && password == adminData.Password)
      {
        ....
        return true;
      }

      return false;
    }

Данная диагностика классифицируется как:

CWE-259 CWE-798 OWASP-2.10.4