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.

Sign up for GitHub

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

Jump to bottom

CEI pattern analysis doesn't consider mint and burn #6153

Closed
IGI-111 opened this issue Jun 20, 2024 · 0 comments · Fixed by #6162
Closed

CEI pattern analysis doesn't consider mint and burn #6153

IGI-111 opened this issue Jun 20, 2024 · 0 comments · Fixed by #6162
Assignees
@jjcnn
Labels
audit-report Related to the audit report bug Something isn't working compiler: frontend Everything to do with type checking, control flow analysis, and everything between parsing and IRgen compiler: static-analysis P: critical Should be looked at before anything else

Comments

@IGI-111
Copy link
Contributor

IGI-111 commented Jun 20, 2024

The cei-pattern-analysis doesn't consider mint / burn as "effects" even though they are part of the contract state.
@IGI-111 IGI-111 added bug Something isn't working compiler: frontend Everything to do with type checking, control flow analysis, and everything between parsing and IRgen compiler: static-analysis P: critical Should be looked at before anything else labels Jun 20, 2024
@IGI-111 IGI-111 changed the title CEI patter nanalysis doesn't consider mint and burn CEI pattern analysis doesn't consider mint and burn Jun 20, 2024
@jjcnn jjcnn mentioned this issue Jun 20, 2024
Merged
8 tasks
@IGI-111 IGI-111 added the audit-report Related to the audit report label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jjcnn jjcnn

Labels
audit-report Related to the audit report bug Something isn't working compiler: frontend Everything to do with type checking, control flow analysis, and everything between parsing and IRgen compiler: static-analysis P: critical Should be looked at before anything else
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make CEI analysis take burn and mint into account FuelLabs/sway
2 participants
@IGI-111 @jjcnn