Rapporto di controllo Snap MetaMask per Polkadot

Sommario di gestione

Polkadot ha contattato Sayfer Security per eseguire un controllo di sicurezza sull'applicazione Metamask Snap di Polkadot nell'agosto 2023.

Prima di valutare i servizi di cui sopra, abbiamo tenuto un incontro iniziale con il team tecnico e abbiamo ricevuto una panoramica del sistema e degli obiettivi di questa ricerca.

Nel corso di un periodo di ricerca di 2 settimane, abbiamo scoperto 5 vulnerabilità nel codice base dello snap.

In conclusione, a seguito del rapporto dovrebbero essere implementate diverse soluzioni, ma il livello di sicurezza del sistema è adeguato.

Metodologia del rischio

Noi di Sayfer ci impegniamo a fornire ai nostri clienti test di penetrazione della massima qualità. Ecco perché abbiamo implementato un modello completo di valutazione del rischio per valutare la gravità dei nostri risultati e fornire ai nostri clienti le migliori raccomandazioni possibili per la mitigazione.

Il nostro modello di valutazione del rischio si basa su due fattori chiave: IMPATTO ed PROBABILITÀ. L'impatto si riferisce al danno potenziale che potrebbe derivare da un problema, come una perdita finanziaria, un danno alla reputazione o un sistema non operativo. La probabilità si riferisce alla probabilità che si verifichi un problema, tenendo conto di fattori quali la complessità dell'attacco e il numero di potenziali aggressori.

Combinando questi due fattori, possiamo creare una comprensione completa del rischio posto da un particolare problema e fornire ai nostri clienti una valutazione chiara e attuabile della gravità del problema. Questo approccio ci consente di dare priorità alle nostre raccomandazioni e garantire che i nostri clienti ricevano la migliore consulenza possibile su come proteggere la loro attività.

Il rischio è definito come segue:

Metodologia di valutazione della sicurezza

Sayfer utilizza OWASP WSTG come nostro standard tecnico durante la revisione delle applicazioni web. Dopo aver acquisito una conoscenza approfondita del sistema, abbiamo deciso quali test OWASP sono necessari per valutare il sistema.

Valutazione della sicurezza

Dopo aver compreso e definito l'ambito, eseguito la modellazione delle minacce e valutato i test corretti richiesti per verificare completamente l'eventuale presenza di difetti di sicurezza nell'applicazione, abbiamo eseguito la nostra valutazione della sicurezza.

Descrizione della tabella dei problemi

Titolo del problema

Descrizione

Qui forniamo una breve descrizione del problema e di come si è formato, i passaggi che abbiamo effettuato per individuarlo o sfruttarlo, insieme alla prova di concetto (se presente) e in che modo questo problema può influire sul prodotto o sui suoi utenti.

Mitigazione

Opzioni di risoluzione suggerite per questo problema e collegamenti a siti consigliati per ulteriori soluzioni.

Risultati della valutazione della sicurezza

Gli URL RPC sono definiti come wsRpcUrl durante l'utilizzo di HTTP

Descrizione

Gli URL RPC dei singoli protocolli sono stati modificati nell'ultima versione dello snap. Viene utilizzato HTTP (http://) invece di WebSocket (wss://).

Questo è inaspettato, perché il documentazione ufficiale di Polkadot indica solo gli indirizzi websocket nelle loro risorse. Inoltre, in molti punti del codice, i nomi delle variabili contengono ancora il prefisso ws, come wsRpcurl.

predefinito.ts:

wsRpcUrl: 'https: /

wsRpcUrl: 'https: /kusama-rpc.polkadot.io/'
{...}
wsRpcUrl: 'https: /westend-rpc.polkadot.io/'
{...}
wsRpcUrl: 'https: /rpc.polkadot.io/'

Mitigazione

Assicurarsi che venga utilizzato il protocollo HTTP. In tal caso, modificare la documentazione ufficiale per riflettere ciò e modificare i nomi delle variabili pertinenti.

Conformazioni per le funzionalità principali non vengono gestite se rifiutate

Descrizione

Per le funzionalità chiave di Snap (come la visualizzazione della chiave privata o la firma di un messaggio), vengono visualizzate finestre di dialogo in cui l'utente può confermare se è a conoscenza di ciò che sta accettando. Se l'utente conferma la transazione, questa viene eseguita. Il contrario, quando l'utente rifiuta la transazione, non viene gestito affatto.

exportSeed(SnapsGlobalObject), nota come non esiste nessun altro blocco

if (confirmation) {
    const bip44Node = (await snap.request({
    method: "snap_getBip44Entropy",
    params: { coinType: kusamaCoinType },
})) as JsonBIP44CoinTypeNode;

signPayloadJSON(SnapsGlobalObject, ApiPromise, SignerPayloadJSON)

if (confirmation) {
    const extrinsic = api.registry.createType('ExtrinsicPayload', payload, { version: payload.version });
    return extrinsic.sign(keyPair);
}

signPayloadRaw(SnapsGlobalObject, ApiPromise, SignerPayloadRaw)

if (confirmation) {
    const signedBytes = keyPair.sign(hexToU8a(payload.data));
    return {
    signature: u8aToHex(signedBytes)
};

Purtroppo, poiché i test automatizzati non hanno funzionato, non abbiamo potuto verificare e confermare definitivamente questo scenario.

Mitigazione

Gestisci il file altro caso in cui l'utente rifiuta la transazione, nel qual caso dovrebbe essere visualizzato un errore appropriato.

Nessuna finestra di dialogo di conferma durante l'invio

Descrizione

Quando si firmano messaggi o si visualizza la chiave privata, lo snap visualizza una finestra di dialogo che chiede all'utente se è sicuro di voler procedere. Tuttavia, durante l'invio non viene visualizzata alcuna finestra di dialogo di conferma.

Mitigazione

Implementare una finestra di dialogo che chiede all'utente di confermare la transazione.

abilitaPolkadotSnap Restituisce un errore quando si utilizza l'impostazione predefinita nome della rete

Descrizione

È stato notato che, nonostante abbia selezionato Westend come rete predefinita in snap/src/configuration/prefined.ts, se l'adattatore esegue la sua unica funzione esposta, EnablePolkadotSnap(), senza specificare config.networkName, verrà restituito un errore.

Ciò non è coerente con i presupposti di Snap, descritti tra gli altri nel README.md dell'adattatore, che indica che la specifica di una configurazione sovrascriverà quella predefinita.

pacchetti/snap/src/configuration/prefined.ts; righe 13-22, 35

export const westendConfiguration: SnapConfig = {
    addressPrefix: 42,
    networkName: "westend",
    unit: {
        decimals: 12,
        image: "https: /svgshare.com/i/L2d.svg",
        symbol: "WND",
    },
    wsRpcUrl: "wss: /westend-rpc.polkadot.io/",
};
[ .]
export const defaultConfiguration: SnapConfig = westendConfiguration;

pacchetti/snap/src/configuration/prefined.ts; righe 27-29

if (!config.networkName) {
throw new Error("Configuration must at least define network type");
}

Mitigazione

A giudicare dalla documentazione e dal codice in predefinito.ts, questo errore non deve essere restituito e deve essere utilizzato Westend. Al contrario, Westend non dovrebbe essere impostato come predefinito e la documentazione dovrebbe essere modificata per rifletterlo.

Demo non funzionante

Descrizione

Secondo la documentazione, il comando yarn install & yarn run demo should run a working demo di Snap, permettendoti di installarlo e di utilizzare le varie funzionalità. Sfortunatamente, durante l'installazione, la console del browser ha restituito un errore. Pertanto, la verifica di Snap come “organismo vivente” era impossibile}

Mitigazione

Suggeriamo di portare Snap in uno stato utilizzabile e quindi di condurre test approfonditi su transazioni reali con la rete Polkadot.