Metamask Snap Audit Report für Qtum

Management-Zusammenfassung

Qtum kontaktierte Sayfer Security, um im Mai 05 einen Penetrationstest für Qtums MetaMask Snap durchzuführen.

Bevor wir die oben genannten Dienste bewerteten, hielten wir ein Kickoff-Meeting mit dem technischen Team von Qtum ab und erhielten einen Überblick über das System und die Ziele dieser Untersuchung.

Während der zweiwöchigen Untersuchung entdeckten wir fünf Schwachstellen im System, keine davon kritisch.

Zusammenfassend lässt sich sagen, dass nach dem Bericht mehrere Korrekturen implementiert werden sollten, der Sicherheitsstatus des Systems jedoch angemessen ist.

Nach einer Überprüfung durch das Sayfer-Team bestätigen wir, dass alle in diesem Bericht erwähnten Sicherheitsprobleme vom Qtum-Team behoben wurden.

Risikomethodik

Bei Sayfer sind wir bestrebt, unseren Kunden Penetrationstests höchster Qualität anzubieten. Aus diesem Grund haben wir ein umfassendes Risikobewertungsmodell implementiert, um die Schwere unserer Ergebnisse zu bewerten und unseren Kunden die bestmöglichen Empfehlungen zur Risikominderung zu geben.

Unser Risikobewertungsmodell basiert auf zwei Schlüsselfaktoren: IMPACT , WAHRSCHEINLICHKEIT. Auswirkungen beziehen sich auf den potenziellen Schaden, der aus einem Problem resultieren könnte, beispielsweise finanzieller Verlust, Rufschädigung oder ein nicht betriebsbereites System. Die Wahrscheinlichkeit bezieht sich auf die Wahrscheinlichkeit, dass ein Problem auftritt, unter Berücksichtigung von Faktoren wie der Komplexität des Angriffs und der Anzahl potenzieller Angreifer.

Durch die Kombination dieser beiden Faktoren können wir ein umfassendes Verständnis des von einem bestimmten Problem ausgehenden Risikos schaffen und unseren Kunden eine klare und umsetzbare Einschätzung der Schwere des Problems liefern. Dieser Ansatz ermöglicht es uns, unsere Empfehlungen zu priorisieren und sicherzustellen, dass unsere Kunden die bestmögliche Beratung zum Schutz ihres Unternehmens erhalten.

Risiko wird wie folgt definiert:

Methodik der Sicherheitsbewertung

Sayfer verwendet OWASP WSTG als unser technischer Standard bei der Überprüfung von Webanwendungen. Nachdem wir uns ein gründliches Verständnis des Systems angeeignet hatten, entschieden wir, welche OWASP-Tests erforderlich sind, um das System zu evaluieren.

Sicherheitsbewertung

Nachdem wir den Umfang verstanden und definiert, eine Bedrohungsmodellierung durchgeführt und die richtigen Tests bewertet hatten, die erforderlich waren, um die Anwendung vollständig auf Sicherheitslücken zu überprüfen, führten wir unsere Sicherheitsbewertung durch.

Beschreibung der Problemtabelle

Titel der Ausgabe

Beschreibung

Hier geben wir eine kurze Beschreibung des Problems und seiner Entstehung, die Schritte, die wir unternommen haben, um es zu finden oder auszunutzen, zusammen mit einem Konzeptnachweis (falls vorhanden) und wie sich dieses Problem auf das Produkt oder seine Benutzer auswirken kann.

Mitigation

Vorgeschlagene Lösungsoptionen für dieses Problem und Links zu empfohlenen Websites zur weiteren Behebung.

Ergebnisse der Sicherheitsbewertung

Keine Methode zum Entfernen hinzugefügter Netzwerke

Beschreibung

Der Code ermöglicht das Hinzufügen zusätzlicher Netzwerke und erweitert deren Liste über die Standardliste hinaus. Es wurde jedoch darauf hingewiesen, dass diese nur durch eine Neuinstallation des Snaps entfernt werden können. Diese Funktionalität war wahrscheinlich geplant, da es zwar eine Funktion removeNetwork() gibt, diese aber nicht über öffentlich verfügbare Methoden aufgerufen werden kann.

export const networks = {
    get: getNetworks,
    setCurrent: setCurrentNetwork,
    add: addNetwork,
    remove: removeNetwork,
};

Mitigation

Wir empfehlen, diese Methode zu implementieren, wie es wahrscheinlich beabsichtigt war.

Fehlende Bestätigungsaufforderung zum Exportieren privater Schlüssel

Beschreibung

Bei den meisten kritischen Funktionen im Snap wird ein Dialogfeld angezeigt, das den Benutzer über die ausgeführte Aktion informiert und um seine Zustimmung bittet.

Diese Sicherheitsmaßnahme fehlt jedoch in der Methode WalletExportPrivateKey, die zur Anzeige des privaten Schlüssels auf dem Snap-Bildschirm verwendet wird. Bei einem versehentlichen Aufruf würde der private Schlüssel sofort angezeigt.

Mitigation

Fügen Sie vor der Anzeige des privaten Schlüssels eine Bestätigungsaufforderung hinzu, in der Sie die Zustimmung des Benutzers einholen.

Mehrere nicht implementierte Methoden

Beschreibung

Ein großer Teil der im Snap vorhandenen Methoden scheint ungenutzt zu bleiben. Es existiert lediglich der sie umgebende Boilerplate-Code, jedoch keine konkrete Logik.

Mitigation

Wir empfehlen, die Methoden, die Sie in Ihrem Snap implementieren möchten, zu überprüfen und sicherzustellen, dass sie alle tatsächliche Logik enthalten. Der Benutzer sollte nicht unnötige Methoden aufrufen können, die Sie nicht implementieren möchten.

Redundanter Code

Beschreibung

Während unserer Analyse haben wir festgestellt, dass zwei RPC-Methoden, EthRequestAccounts und EthAccounts, scheinbar dasselbe tun und dieselben Werte zurückgeben.

case RPCMethods.EthRequestAccounts: {
    const wallet = await getWallet();
    return [wallet.address];
}
case RPCMethods.EthAccounts: {
    console.log('RPCMethods.EthAccounts');
    console.log('request.params', JSON.stringify(request.params));
    const wallet = await getWallet();
    return [wallet.address];
}

In beiden Fällen rufen diese Methoden die Daten der aktiven Wallet mithilfe von getWallet() ab und geben anschließend deren Adresse zurück. Der einzige Unterschied besteht derzeit darin, dass EthAccounts protokolliert, EthRequestAccounts hingegen nicht.

Mitigation

Erwägen Sie, entweder eine der Methoden zu entfernen oder eine zu ändern, um sie von den anderen zu unterscheiden

Übrig gebliebene TODO-Kommentare

Beschreibung

Die Codebasis enthält mehrere TODO-Kommentare und auch Abschnitte mit auskommentiertem Code, der nicht mehr verwendet wird.

Dies ist zwar in einer Testumgebung akzeptabel, sollte aber in der Produktion vermieden werden. Solche Kommentare können potenziellen Stakeholdern wertvolle Informationen über fehlende Funktionen, Sicherheitsmaßnahmen und die Gedanken der Entwickler liefern, die zur Profilierung von Angriffen genutzt werden können.

Mitigation

Wir empfehlen, als TODO gekennzeichnete Kommentare und nicht verwendete Codefragmente in Ihrem Produktionszweig zu entfernen.