DeFiHardhatFoundry
250,000 USDC
View results
Submission Details
Severity: low
Invalid

`executePipelineConvert` uses regular `transfer` instead of `safeTransfer`

Summary

executePipelineConvert uses regular transfer instead of safeTransfer

Vulnerability Details

Although currently PipelineConvert can only be executed with a Well token or Bean and none of them require using safeTransfer, Wells are allowed to implement custom logic and some future instances might not work properly with regular transfer.

Consider using safeTransfer instead

Impact

Lack of operability

Tools Used

Manual review

Recommendations

Use safeTransfer

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational/Gas

Invalid as per docs https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.