QA Report

[c4-bot-4]

See the markdown file with the details of this report here.

[141345]

330 0xA5DF
l r nc
4 2 0

L 1 d dup of #443
L 2 l
L 3 d dup of #111
L 4 d dup of #396
L 5 d dup of #238
L 6 d dup of #409
L 7 l
L 8 l
L 9 l
L 10 r
L 11 r

[c4-sponsor]

Alec1017 (sponsor) confirmed

[Alec1017]

L1- Acknowledged
L2- Acknowledged
L3- Disputed, considered out of scope because it doesnt adhere to standard ERC721 spec
L4- Disputed, this is expected behavior
L5- Confirmed
L6- Confirmed
L7- Disputed, stop module will not be whitelisted
L8- Acknowledged, hook implementations considered out of scope
L9- Disputed, policies can only be enabled or disabled. in this instance, a second stop policy would be created for users to opt-in to

R1- Acknowleged
R2- Disputed, this is intended behavior. To disable the hook, updateHookStatus would be used here instead. the bitmap would be set to 0. It does not matter what the path is set to if the status is 0

[c4-judge]

0xean marked the issue as grade-a

[c4-judge]

0xean marked the issue as selected for report

[0xean]

L3- OOS
L7 - invalid
R2 - invalid

others lgtm.

[0xA5DF]

L1 is dupe of #443
L5 is dupe of #43

[0xean]

#443 is QA, upgraded L5 / #43