NFT Dealers

First Flight #58
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

Public `calculateFees` helper

  • Description: calculateFees expone públicamente una operación puramente determinística sobre parámetros y constantes ya visibles en bytecode/código fuente.

  • Why this matters: La función no revela secretos; sin embargo, amplía superficie pública innecesaria si no está justificada por producto/integración.

  • Security impact: Nulo o marginal. Un atacante puede calcular exactamente lo mismo off-chain sin llamar al contrato.

  • Design impact: Puede generar ruido de API y comentarios contradictorios (el código indica “remove before production” pero la función sigue presente).

  • Recommended mitigation: Mantenerla solo si aporta utilidad externa real (SDK/UI/testing on-chain). En caso contrario, eliminarla o mover la lógica a tooling off-chain.

Updates

Lead Judging Commences

rubik0n Lead Judge 16 days ago
Submission Judgement Published
Invalidated
Reason: Design choice
Assigned finding tags:

Invalid

Support

FAQs

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

Give us feedback!