[Bug]: Edition #70 was minted for a master edition with max supply #69

[arcticmatt]

Which package is this bug report for?

token-metadata

Which Type of Package is this bug report for?

Rust Contract

Issue description

For https://formfunction.xyz/@degenpoet/F5smPCKim7iDQQGL3kGA2mtJDaA5wd2E18Vqa5nbJbHD?width=3000&height=4000, you can see that edition #70 was successfully minted, even though the max supply is 69

• Master edition https://explorer.solana.com/address/F5smPCKim7iDQQGL3kGA2mtJDaA5wd2E18Vqa5nbJbHD?cluster=mainnet-beta. You can see that the max supply is 69
• Edition Merge master #70 https://explorer.solana.com/address/dDgJjQDcTxeRmiyda7zn4xNTzS6YD1wowt8rhDLVRuu?cluster=mainnet-beta

AFAICT it's a bug with this code https://github.com/metaplex-foundation/metaplex-program-library/blame/master/token-metadata/program/src/utils.rs#L554-L572

Let's say max_supply = 69 and current_supply = 69

Then, new_supply = 69, which is not > max, and so MaxEditionsMintedAlready will never be hit

Relevant log output

No response

Priority this issue should have

High (immediate attention needed)

[roederw]

Resolved. Thank you for bringing this to our attention. Follow up here if the issue seems to NOT be resolved!

[arcticmatt]

@roederw thanks! Do you mind linking to the PR/commit that fixed this?

[samuelvanderwaal]

@roederw thanks! Do you mind linking to the PR/commit that fixed this?

#779