Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Risk - Potentially Weak Cryptography Implementations [potentially-weak-crypto-impl] #2583

Open
4 tasks
cpholguera opened this issue Mar 1, 2024 · 0 comments

Comments

@cpholguera
Copy link
Collaborator

Description

Create a new risk for "Potentially Weak Cryptography Implementations (MASVS-CRYPTO-1)" using the following information:

Don't use outdated or known weak implementations and don't build your own cryptography. Using custom cryptography instead of relying on established, expert-designed APIs or certified modules exposes apps to vulnerabilities due to potential implementation flaws and lack of rigorous security review.

Create "risks/MASVS-CRYPTO/1-***-****/potentially-weak-crypto-impl/risk.md" including the following content:

---
title: Potentially Weak Cryptography Implementations
alias: potentially-weak-crypto-impl
platform: [android, ios]
profiles: [L2]
mappings:
  masvs-v1: [MSTG-CRYPTO-2]
  masvs-v2: [MASVS-CRYPTO-1, MASVS-CODE-3]
  mastg-v1: [MASTG-TEST-0061, MASTG-TEST-0014]

---

## Overview

## Impact

## Modes of Introduction

## Mitigations

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

Use at least the following references:

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

  • platform-provided cryptographic APIs (e.g. conscrypt/CryptoKit)
  • custom-made cryptographic APIs (e.g. via xor, bit flipping, etc. or cryptographic constants or values such as sbox, etc.)
  • custom algorithms, primitives, protocols
  • specify Cipher.getInstance provider (Android)
  • Android Security Provider (Android)
  • Jetpack Security Crypto Library (Android)
  • BoucyCastle algorithms (Android)

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.

Acceptance Criteria

  • The risk has been created in the correct directory (risks/MASVS-CRYPTO/1-***-****/potentially-weak-crypto-impl/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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant