Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

critical audit fix (remove hardcoded address and bump solidity version) #5214

Open
noxonsu opened this issue Sep 1, 2023 · 1 comment
Open
Labels
ready The required changes are merged into the master

Comments

@noxonsu
Copy link
Member

noxonsu commented Sep 1, 2023

https://github.com/search?q=repo%3Aswaponline%2FMultiCurrencyWallet%200x180c3B784f3425B40fAE0eD8CeFF6bBc577A3c13&type=code

см остальное https://scrutify.super.site/encore-defi см остальное

@noxonsu
Copy link
Member Author

noxonsu commented Sep 1, 2023

в аудите есть номер строки которые исправить и идентификатор файлов.

Идентификатор "FMFT-1" указывает на конкретный файл, в котором обнаружена ошибка или уязвимость. В данном аудиторском отчете файлы обычно перечислены в разделе, где говорится о проверенных контрактах. В вашем случае, идентификатор "FMFT" скорее всего относится к файлу contracts/MultiCurrencyWallet/FarmFactory.sol.

Почему? Обычно аудиторы используют уникальные идентификаторы для каждого файла или группы ошибок, чтобы было проще понять, к какому файлу относится каждая уязвимость или ошибка. В данном отчете видно, что к каждому номеру строки добавлен префикс, который служит идентификатором файла. В данном случае, "FMFT-1" указывает на первую уязвимость в файле, который в отчете обозначен как "FMFT".

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

@shendel shendel added the ready The required changes are merged into the master label Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready The required changes are merged into the master
Projects
None yet
Development

No branches or pull requests

2 participants