Security context

What an agent needs to avoid regressing past fixes and find the next vuln in this repo.

mycryptohq/mycrypto

              master @ 457ec2f
            

Fixes

CVEs

HIGH

Peak severity

Highlights

Denial of Service: 2 prior fixes. Scrutinize any change in this area.

common/v2/features/SendAssets/components/SendAssetsForm.tsx: most-fixed (2 issues). Treat as high-risk during review.

5 high-severity fixes in this history; regressions here are high-impact.

Recurring patterns

The bug types that recur here, drawn from past fixes, not open vulnerabilities.

Sandbox Escape: Insecure default Electron configurations, specifically enabling nodeIntegration and disabling contextIsolation, allowed compromised renderer processes to execute arbitrary OS commands. This was resolved by disabling nodeIntegration and enabling contextIsolation in the webPreferences configuration.

Auth Bypass: Incomplete ENS resolution error-handling allowed transaction fields to persist invalid or unvalidated values, risking assets being routed to unintended destinations. Resolving this required forcing field values to reset to empty upon resolution errors.

Navigation Hijacking: Unrestricted URL navigation and lack of validation on openExternal shell operations allowed arbitrary protocol handlers to trigger local system commands. The defense requires strict URI scheme validation during window navigation events.