Submitted by Daniel_eth, also found by Rhaydden (1, 2), hl_, LSHFGJ, iam_emptyset, Fitro, dontonka, TopStar, Abdessamed, 0xDazai, Agontuk, 0x3b, Shubham, 0xb0k0, Ruhum, avoloder, 0xlemon, AvantGard, Tigerfrake, 0xAleko, ABAIKUNANBAEV, FastChecker, 0xHash, anonymousjoe, y4y, JustUzair, samuraii77, onthehunt11, Autosaida, Trident-Audits, yaioxy, LeFy, EaglesSecurity, MrValioBg, PASCAL, shikhar229169, pfapostol, Tonchi, AuditGuy, 3n0ch, 0x0bserver, dobrevaleri, inzinko, kartik_giri_47538, dyoff, Topmark, DemoreX, abdulsamijay, rbserver, JanuaryPersimmon2024, KupiaSec, Kalogerone, 0xJoyBoy03, Zac, ilchovski, ke1caM, nnez, 0xrex, and shaka
In TraitForgeNft there should be a maximum number of generations (it should be capped at 10), but instead, users can mint infinite generations and there is not a limit.
Foundry:
In TraiForgeNft::\_incrementGeneration, add a check in order to do not allow the the mint of the NFTs above the maximum generation.
Instead of doing this:
Consider doing this:
TForge1 (TraitForge) confirmed
