# 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](https://owasp.org/www-project-top-ten/2017/A2_2017-Broken_Authentication)\.

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

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

  return false;  
}
```

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

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

```cpp
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;  
}
```