To read this page in English, please use your best translator.

Analyse de sécurité

SafeBox V 1.5.0 | Analyse du 03 Août 2026

By AI Copilot in VScode

J’ai analysé les fichiers clés : safebox.py, security.py, functions.py, safebox_setup.py, safebox_update.py, logger_config.py et constants.py.

Le projet utilise de bonnes primitives cryptographiques (bcrypt, PBKDF2HMAC, Fernet) mais la conception et l’usage logiciel présentent plusieurs faiblesses importantes.

Vulnérabilités principales

1) Exposition locale des données sensibles

Fichiers concernés : config.local.json, safebox.json, settings.json, lang.json

Risque :

• Tout accès local peut lire le hash, le sel et les données chiffrées.

Exploitation :

• Un malware ou un utilisateur malveillant peut copier les fichiers puis lancer une attaque par force brute hors ligne.

Correction :

• Restreindre les permissions.
• Écrire les fichiers de manière atomique.
• Utiliser un répertoire protégé.

Exemple :

import tempfile
import stat

def secure_write_json(path, data):
    os.makedirs(os.path.dirname(path), exist_ok=True)

    fd, tmp_path = tempfile.mkstemp(
        dir=os.path.dirname(path),
        suffix=".tmp"
    )

    try:
        with os.fdopen(fd, "w", encoding="utf-8") as f:
            json.dump(data, f, indent=4, ensure_ascii=False)
            f.flush()
            os.fsync(f.fileno())

        os.replace(tmp_path, path)

        try:
            os.chmod(path, stat.S_IRUSR | stat.S_IWUSR)
        except OSError:
            pass

    finally:
        if os.path.exists(tmp_path):
            os.remove(tmp_path)

3) Configuration de mise à jour non fiable

Fichiers : safebox_update.py, constants.py

Risque :

• L’URL des mises à jour est modifiable via settings.json.

Correction :

• Utiliser une URL codée en dur. PATCHED
• Vérifier l’intégrité des fichiers téléchargés.

TRUSTED_BASE_URL = "https://raw.githubusercontent.com/Gwigzz/test_version/main/"
URL_VERSIONS = TRUSTED_BASE_URL + "version.json"

4) Validation insuffisante des fichiers de configuration

Correction :

def load_config():
    config = load_json_file(get_config_path())

    if not config or not isinstance(config, dict):
        raise ValueError("config.local.json invalide")

    if "password_hash" not in config or "salt" not in config:
        raise ValueError("config.local.json incomplet")

    return config

5) Politique de mot de passe trop faible

Correction :

def validate_master_password(password):

    if len(password) < 12:
        return False

    if not any(c.islower() for c in password):
        return False

    if not any(c.isupper() for c in password):
        return False

    if not any(c.isdigit() for c in password):
        return False

    if not any(c in "!@#$%^&*()-_=+[]{};:'\",.<>?/\\|" for c in password):
        return False

    return True

6) Écriture non atomique du coffre

def save_vault(data):

    vault_path = get_safebox_file()

    dir_name = os.path.dirname(vault_path)

    os.makedirs(dir_name, exist_ok=True)

    tmp_path = vault_path + ".tmp"

    with open(tmp_path, "w", encoding="utf-8") as f:
        json.dump(data, f, indent=4)
        f.flush()
        os.fsync(f.fileno())

    os.replace(tmp_path, vault_path)

7) Permissions de fichiers sous Windows

import ctypes

def set_hidden_file(path):
    FILE_ATTRIBUTE_HIDDEN = 0x02
    ctypes.windll.kernel32.SetFileAttributesW(
        str(path),
        FILE_ATTRIBUTE_HIDDEN
    )

Sur Windows, il est préférable d'utiliser les API de sécurité (SetNamedSecurityInfoW) afin de restreindre réellement les ACL.

Points OWASP applicables

• A2 Broken Authentication
• A6 Security Misconfiguration
• A9 Using Components with Known Vulnerabilities
• A5 Broken Function Level Authorization / A1 Injection

Recommandations de conception

Séparer la configuration de l'application

• Ne pas stocker l'URL des mises à jour dans un fichier modifiable. PATCHED
• Utiliser une constante interne ou une signature. PATCHED

Éviter l'exposition des secrets

• Limiter leur présence en mémoire.
• Proposer une copie temporaire dans le presse-papiers.

Valider les fichiers JSON

• Intercepter les erreurs JSON.
• Contrôler les types.
• Vérifier les clés attendues.

Protéger les sauvegardes

• Chiffrer les sauvegardes.
• Utiliser une clé dérivée de la clé maître.

Conclusion

Le projet utilise des primitives cryptographiques solides, mais plusieurs défauts d'implémentation diminuent son niveau de sécurité.

• Protection insuffisante des fichiers locaux.
• Mise à jour externe non sécurisée. PATCHED
• Écriture non atomique.
• Validation insuffisante des configurations. PATCHED

La mise en œuvre des recommandations proposées permettra d'améliorer sensiblement la robustesse de l'application.