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

add scout.yml #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add scout.yml #1

wants to merge 2 commits into from

Conversation

matiascabello
Copy link
Owner

No description provided.

Copy link

github-actions bot commented Jul 3, 2024

<style> .markdown-body table {min-width: 100%;width: 100%;display: table;} thead {min-width: 100%;width: 100%;} th {min-width: 60%;width: 60%;} th:last-child {min-width: 20%;width: 20%;} th:first-child {min-width: 20%;width: 20%;} </style>

Scout Report - Example 1 - 2024-07-11

Summary

Executed on:

Issues found:

Arithmetic

Overflow Check

Impact: Critical

Issue: Use overflow-checks = true in Cargo.toml profile

Description: An overflow/underflow is typically caught and generates an error. When it is not caught, the operation will result in an inexact result which could lead to serious problems.

Learn More

Findings

ID Package File Location
0 src lib.rs:1:1 - 1:1

Divide Before Multiply

Impact: Medium

Issue: Division before multiplication might result in a loss of precision

Description: Performing a division operation before a multiplication can lead to a loss of precision. This issue becomes significant in programs like smart contracts where numerical precision is crucial.

Learn More

Findings

ID Package File Location
2 src lib.rs:10:9 - 10:27

Best practices

Check Soroban version

Impact: Enhancement

Issue: Use the latest version of Soroban

Description: Using a older version of Soroban can be dangerous, as it may have bugs or security issues. Use the latest version available.

Learn More

Findings

ID Package File Location
1 src lib.rs:1:1 - 1:1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant