Отчет MetaMask Snap Audit для Polkadot

Резюме управления

Polkadot связалась с Sayfer Security с целью провести аудит безопасности приложения Polkadot Metamask Snap в августе 2023 года.

Прежде чем оценить вышеуказанные услуги, мы провели стартовую встречу с технической командой и получили обзор системы и целей этого исследования.

За двухнедельный период исследования мы обнаружили 2 уязвимостей в кодовой базе Snap.

В заключение следует отметить, что после отчета необходимо внести несколько исправлений, но уровень безопасности системы является компетентным.

Методология риска

В Sayfer мы стремимся предоставлять нашим клиентам тестирование на проникновение высочайшего качества. Вот почему мы внедрили комплексную модель оценки рисков, чтобы оценить серьезность наших выводов и предоставить нашим клиентам наилучшие рекомендации по их снижению.

Наша модель оценки рисков основана на двух ключевых факторах: ВЛИЯНИЕ и ВЕРОЯТНОСТЬ. Под воздействием понимается потенциальный вред, который может возникнуть в результате проблемы, например финансовые потери, репутационный ущерб или неработоспособность системы. Вероятность означает вероятность возникновения проблемы с учетом таких факторов, как сложность атаки и количество потенциальных злоумышленников.

Объединив эти два фактора, мы можем получить полное представление о риске, связанном с конкретной проблемой, и предоставить нашим клиентам четкую и действенную оценку серьезности проблемы. Такой подход позволяет нам расставлять приоритеты в наших рекомендациях и гарантировать, что наши клиенты получат наилучшие советы о том, как защитить свой бизнес.

Риск определяется следующим образом:

Методология оценки безопасности

Сайфер использует ОВАСП ВСТГ в качестве нашего технического стандарта при рассмотрении веб-приложений. Получив полное представление о системе, мы решили, какие тесты OWASP необходимы для ее оценки.

Оценка безопасности

После понимания и определения области применения, выполнения моделирования угроз и оценки правильных тестов, необходимых для полной проверки приложения на наличие недостатков безопасности, мы провели оценку безопасности.

Описание таблицы проблем

Название проблемы

Описание

Здесь мы даем краткое описание проблемы и того, как она возникла, шаги, которые мы предприняли для ее обнаружения или использования, а также доказательство концепции (если таковое имеется), а также то, как эта проблема может повлиять на продукт или его пользователей.

риска

Предлагаемые варианты решения этой проблемы и ссылки на рекомендованные сайты для дальнейшего устранения.

Результаты оценки безопасности

URL-адреса RPC определяются как всРпкурл при использовании HTTP

Описание

URL-адреса RPC отдельных протоколов были изменены в последней версии снапа. Вместо WebSocket (wss://) используется HTTP (http://).

Это неожиданно, поскольку официальная документация Polkadot указывает только адреса веб-сокетов в своих ресурсах. Кроме того, во многих местах кода имена переменных по-прежнему содержат префикс ws, например wsRpcurl.

предопределенный.ts:

wsRpcUrl: 'https: /

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

риска

Убедитесь, что предполагается использовать протокол HTTP. Если это так, измените официальную документацию, чтобы отразить это, и измените соответствующие имена переменных.

Конформации ключевые функции не обрабатываются при отклонении

Описание

Для функций привязки ключей (таких как отображение закрытого ключа или подписание сообщения) пользователю отображаются диалоговые окна, позволяющие подтвердить, знают ли они, что принимают. Если пользователь подтверждает транзакцию, она выполняется. Обратное явление, когда пользователь отклоняет транзакцию, вообще не обрабатывается.

ExportSeed(SnapsGlobalObject), обратите внимание, что здесь нет блока else

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)
};

К сожалению, поскольку автоматизированные тесты не сработали, мы не смогли окончательно проверить и подтвердить этот сценарий.

риска

Справиться с еще случае, когда пользователь отклоняет транзакцию, и в этом случае должна отображаться соответствующая ошибка.

Нет диалогового окна подтверждения при отправке

Описание

При подписании сообщений или отображении закрытого ключа Snap отображает диалоговое окно с вопросом, уверены ли они, что хотят продолжить. Однако при отправке такой диалог подтверждения не отображается.

риска

Реализуйте диалоговое окно, предлагающее пользователю подтвердить транзакцию.

включитьPolkadotSnap Возвращает ошибку при использовании значения по умолчанию имя сети

Описание

Было замечено, что несмотря на выбор Westend в качестве сети по умолчанию в snap/src/configuration/predefine.ts, если адаптер выполняет свою единственную доступную функцию, EnablePolkadotSnap(), без указания config.networkName, то будет возвращена ошибка.

Это несовместимо с предположениями Snap, описанными, среди прочего, в README.md адаптера, которые указывают, что указание конфигурации переопределяет конфигурацию по умолчанию.

пакеты/привязка/src/конфигурация/predefine.ts; строки 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;

пакеты/привязка/src/конфигурация/predefine.ts; строки 27-29

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

риска

Судя по документации и коду в предопределенный.ts, эта ошибка не должна возвращаться и следует использовать Westend. И наоборот, Westend не следует устанавливать по умолчанию, и в документацию следует вносить соответствующие изменения.

Нерабочая демо-версия

Описание

Согласно документации, команда yarn install & yarn run demo should run a working demo Snap, что позволяет вам установить его и использовать различные функции. К сожалению, во время установки консоль браузера выдала ошибку. Таким образом, проверка Snap как «живого организма» была невозможна}

риска

Мы предлагаем привести Snap в работоспособное состояние, а затем провести углубленные тесты на реальных транзакциях с сетью Polkadot.