Metamask Snap revisionsrapport för PolkaGate Snap

Ledningssammanfattning

PolkaGate kontaktade Sayfer Security för att utföra penetrationstestning på PolkaGate Snap i februari 2024.

Innan vi utvärderade ovanstående tjänster höll vi ett kickoffmöte med PolkaGates tekniska team och fick en översikt över systemet och målen för denna forskning.

Under forskningsperioden på två veckor upptäckte vi två sårbarheter i systemet.

Sammanfattningsvis bör flera korrigeringar implementeras efter rapporten, men systemets säkerhetsställning är kompetent.

Efter en granskning av Sayfer-teamet intygar vi att alla säkerhetsproblem som nämns i denna rapport har åtgärdats av PolkaGate-teamet.

Riskmetodik

På Sayfer har vi åtagit oss att leverera penetrationstestning av högsta kvalitet till våra kunder. Det är därför vi har implementerat en omfattande riskbedömningsmodell för att utvärdera hur allvarliga våra fynd är och ge våra kunder bästa möjliga rekommendationer för begränsning.

Vår riskbedömningsmodell bygger på två nyckelfaktorer: IMPACT och SANNOLIKHET. Påverkan hänvisar till den potentiella skada som kan bli följden av ett problem, såsom ekonomisk förlust, skada på rykte eller ett icke-operativt system. Sannolikhet avser sannolikheten att ett problem kommer att inträffa, med hänsyn till faktorer som attackens komplexitet och antalet potentiella angripare.

Genom att kombinera dessa två faktorer kan vi skapa en heltäckande förståelse för risken för en viss fråga och ge våra kunder en tydlig och handlingsbar bedömning av problemets allvar. Detta tillvägagångssätt gör att vi kan prioritera våra rekommendationer och säkerställa att våra kunder får bästa möjliga råd om hur de kan skydda sin verksamhet.

Risk definieras enligt följande:

Säkerhetsutvärderingsmetodik

Sayfer använder OWASP WSTG som vår tekniska standard vid granskning av webbapplikationer. Efter att ha fått en grundlig förståelse av systemet bestämde vi oss för vilka OWASP-tester som krävs för att utvärdera systemet.

Säkerhetsbedömning

Efter att ha förstått och definierat omfattningen, utfört hotmodellering och utvärderat de korrekta testerna som krävs för att fullständigt kontrollera applikationen för säkerhetsbrister, utförde vi vår säkerhetsbedömning.

Problemtabell Beskrivning

Nummertitel

BESKRIVNING

Här ger vi en kort beskrivning av problemet och hur det uppstod, stegen vi gjorde för att hitta eller utnyttja det, tillsammans med proof of concept (om det finns) och hur detta problem kan påverka produkten eller dess användare.

Mitigation

Föreslagna lösningsalternativ för det här problemet och länkar till rekommenderade webbplatser för ytterligare åtgärdande.

Säkerhetsbedömningsresultat

Ingen varning vid signering av godtyckliga data

BESKRIVNING

Under vår bedömning märkte vi att signering av godtyckliga data inte utlöser någon ytterligare varning, enligt skärmdumpen nedan.

Som standard lägger Metamask till en varning när en användare ombeds att signera godtyckliga data för att förhindra signaturstölder. Även om sådana typer av bedrägerier främst observeras på EVM-baserade kedjor, skulle liknande bedrägerier säkert kunna utföras på andra kedjor

Mitigation

Visa en varning som liknar den infödda metamaskens. Kanske något liknande "Varning - fortsätt bara om du förstår meddelandet nedan!".

För mer information, se här..

Ofullständig enhetstesttäckning

BESKRIVNING

Snäppet hanterar flera funktioner i sin huvudsakliga OnRpcRequestHandler. Alla av dem förutom setMetadata omfattas av enhetstesterna i index.test.ts.

Mitigation

Lägg till ett enhetstest som täcker setMetadata.