[MASWE-0006] Sensitive Data Stored Unencrypted in Private Storage Locations

[cpholguera]

Description

Create a new risk for "Sensitive Data Stored Unencrypted in Private Storage Locations (MASVS-STORAGE-1)" using the following information:

Sensitive data may be stored in internal locations without encryption and may be accessible to other apps under certain conditions.

Create "risks/MASVS-STORAGE/1-***-****/data-unencrypted-private-storage/risk.md" including the following content:

---
title: Sensitive Data Stored Unencrypted in Private Storage Locations
alias: data-unencrypted-private-storage
platform: [android, ios]
profiles: [L2]
mappings:
  masvs-v1: [MSTG-STORAGE-2]
  masvs-v2: [MASVS-STORAGE-1, MASVS-CRYPTO-2]
  mastg-v1: [MASTG-TEST-0052, MASTG-TEST-0001]

---

## Overview

## Impact

## Modes of Introduction

## Mitigations

To complete the sections follow the guidelines from Writing MASTG Risks & Tests

When creating the corresponding tests, use the following areas to guide you:

• envelope encryption (DEK+KEK) or equivalent (Android)
• Android Security Lib usage (EncryptedFile/EncryptedSharedPreferences) (Android)
• Don't roll your own storage encryption, use platform provided APIs EncryptedFile/EncryptedSharedPreferences. (Android)
• iOS KeyChain DataProtection classes (iOS)
• envelope encryption (DEK+KEK) or equivalent (iOS)
• sensitive data must not encoded (e.g. base64, simple bit operations such as XOR or bit flipping) instead of encrypted

MASTG v1 Refactoring:

If the risk has a MASVS v1 ID, you can use it to search for related tests in the MASTG and use them as input to define your risks and associated tests.

• MASTG-TEST-0052 - Testing Local Data Storage (ios)
• MASTG-TEST-0001 - Testing Local Storage for Sensitive Data (android)

Acceptance Criteria

• [ ] The risk has been created in the correct directory (risks/MASVS-STORAGE/1-***-****/data-unencrypted-private-storage/risk.md)
• [ ] The risk content follows the guidelines
• [ ] At least one GitHub Issue has been created for the corresponding tests (derived from "Modes of Introduction")
• [ ] The risk indicates the related MASTG v1 tests in its metadata.

[cpholguera]

Assign to @thomascannon

[cpholguera]

Hi @thomascannon, I think this PR was automatically closed because the target branch doesn't exist anymore. Sorry about that. Could you please reopen it targeting master? Thank you!